1
0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-20 07:59:00 +00:00
Peter Krempa 39218865f4 virsh: Require option flags for 'blkdeviotune' arguments
Make all of the tunable parameter flags require the option name (don't
parse them positionally).

While techically this would be a breaking change if anyone were to
specify the tunable values positionally this is not the case as the
first two tunables are not compatible with each other:

  $ virsh blkdeviotune cd vda 4 5
  error: Unable to change block I/O throttle
  error: invalid argument: total and read/write of bytes_sec cannot be set at the same time

The above is produced by all implementations of the API (qemu and test
drivers). It is true that the first tunable can be specified
positionally (--total-bytes-sec) but it is misleading and shoud not be
allowed either.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
2024-04-02 14:24:29 +02:00
..
2023-11-24 09:53:14 +01:00
2024-03-13 15:02:52 +01:00
2024-03-13 15:02:52 +01:00
2019-08-09 09:03:53 +02:00
2024-03-13 15:02:52 +01:00
2024-03-13 15:02:52 +01:00
2024-03-13 15:02:52 +01:00
2024-03-13 15:02:52 +01:00
2023-03-13 13:29:07 +01:00
2023-05-05 15:08:25 +02:00