]> git.proxmox.com Git - proxmox-backup.git/commit
fix #5622: backup client: properly handle rate/burst parameters
authorDominik Csapak <d.csapak@proxmox.com>
Fri, 9 Aug 2024 08:20:32 +0000 (10:20 +0200)
committerWolfgang Bumiller <w.bumiller@proxmox.com>
Fri, 30 Aug 2024 11:21:29 +0000 (13:21 +0200)
commitff485aa3200d9eb52330728038c8e71bda741706
tree8e3b7ec86a3d42687e4d5f09a58c09b7cc44fcea
parent8f27262d422b08349ab29a25c220508caeefb3c9
fix #5622: backup client: properly handle rate/burst parameters

The rate and burst parameters are integers, so the mapping from value
with `.as_str()` will always return `None` effectively never
applying any rate limit at all.

Fix it by turning them into a HumanByte instead of an integer.

To not crowd the parameter section so much, create a
ClientRateLimitConfig struct that gets flattened into the parameter list
of the backup client.

To adapt the description of the parameters, add new schemas that copy
the `HumanByte` schema but change the description.

With this, the rate limit actually works, and there is no lower limit
any more.

The old TRAFFIC_CONTROL_RATE/BURST_SCHEMAs can be deleted since the
client was the only user of them.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
pbs-api-types/src/traffic_control.rs
proxmox-backup-client/src/main.rs