[pve-devel] [PATCH pve-storage 1/1] volume export: explicitly state unit of bandwidth limit

Stefan Hanreich s.hanreich at proxmox.com
Wed Mar 29 14:34:35 CEST 2023

Other places like replication explicitly state whether byte or bit
are used as a unit rather than only the abbreviation.

Signed-off-by: Stefan Hanreich <s.hanreich at proxmox.com>
Could have gone for Bps instead, but I think this is more explicit
and all other places I could find are using the explicit unit as well.
Also, everything else is in lowercase so I decided to stick with that.

 PVE/Storage.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PVE/Storage.pm b/PVE/Storage.pm
index cec3996..b99ed35 100755
--- a/PVE/Storage.pm
+++ b/PVE/Storage.pm
@@ -752,7 +752,7 @@ my $volume_export_prepare = sub {
     my $cstream;
     if (defined($ratelimit_bps)) {
 	$cstream = [ '/usr/bin/cstream', '-t', $ratelimit_bps ];
-	$logfunc->("using a bandwidth limit of $ratelimit_bps bps for transferring '$volid'") if $logfunc;
+	$logfunc->("using a bandwidth limit of $ratelimit_bps bytes per second for transferring '$volid'") if $logfunc;
     volume_snapshot($cfg, $volid, $snapshot) if $migration_snapshot;

More information about the pve-devel mailing list