[Libguestfs] [PATCH v2v] v2v: Remove extraneous '=' when setting --bandwidth/--bandwidth-file.

Richard W.M. Jones rjones at redhat.com
Thu May 28 10:51:16 UTC 2020


Commit c3a54d6aed6dfc65f9ffa59976bb8d20044c03a8 ("v2v: Add standalone
nbdkit module.") was supposed to be a simple refactoring but it broke
the --bandwidth and --bandwidth-file options (amongst other things).

Because of an extra '=' character which was accidentally left over, it
would add an extra character in the nbdkit-rate-filter command line.
For example:

  virt-v2v .. --bandwidth 200M

would invoke:

  nbdkit .. --filter rate rate==200M

which causes a parse error.  The --bandwidth-file option does not
invoke a parse error but does not work, for similar reasons.

Thanks: Ming Xie
---
 v2v/nbdkit_sources.ml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/v2v/nbdkit_sources.ml b/v2v/nbdkit_sources.ml
index 979c37734d..402dfd0e55 100644
--- a/v2v/nbdkit_sources.ml
+++ b/v2v/nbdkit_sources.ml
@@ -118,11 +118,11 @@ let common_create ?bandwidth ?extra_debug ?extra_env plugin_name plugin_args =
          let args =
            match bandwidth with
            | StaticBandwidth rate ->
-              [ "rate=", rate ]
+              [ "rate", rate ]
            | DynamicBandwidth (None, filename) ->
-              [ "rate-file=", filename ]
+              [ "rate-file", filename ]
            | DynamicBandwidth (Some rate, filename) ->
-              [ "rate=", rate; "rate-file=", filename ] in
+              [ "rate", rate; "rate-file", filename ] in
          cmd, args
     )
     else cmd, [] in
-- 
2.26.2




More information about the Libguestfs mailing list