[libvirt] [PATCH 4/5] virsh: Use consistent naming for blkdeviotune options
John Ferlan
jferlan at redhat.com
Wed Jan 25 12:10:59 UTC 2017
On 01/25/2017 04:16 AM, Martin Kletzander wrote:
> All options started with underscores, but we switched them to dashes
> later on, making the style consistent. The latest addition, however,
> did not respect that, so let's change that as well. It is tempting to
> just change the name instead of adding alias, especially since nobody
> ever used it, which we know thanks to the fact that it didn't work.
>
> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> ---
> tools/virsh-domain.c | 6 +++++-
> tools/virsh.pod | 4 ++--
> 2 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
> index 4360306989d8..4f7f23f8a18e 100644
> --- a/tools/virsh-domain.c
> +++ b/tools/virsh-domain.c
> @@ -1264,6 +1264,10 @@ static const vshCmdOptDef opts_blkdeviotune[] = {
> .help = N_("I/O size in bytes")
> },
> {.name = "group_name",
> + .type = VSH_OT_ALIAS,
> + .help = "group-name"
> + },
> + {.name = "group-name",
> .type = VSH_OT_STRING,
> .help = N_("group name to share I/O quota between multiple drives")
> },
> @@ -1398,7 +1402,7 @@ cmdBlkdeviotune(vshControl *ctl, const vshCmd *cmd)
> VSH_ADD_IOTUNE(write-iops-sec-max-length, WRITE_IOPS_SEC_MAX_LENGTH);
> #undef VSH_ADD_IOTUNE
>
> - if (vshCommandOptStringReq(ctl, cmd, "group_name", &group_name) < 0) {
> + if (vshCommandOptStringReq(ctl, cmd, "group-name", &group_name) < 0) {
> vshError(ctl, "%s", _("Unable to parse group parameter"));
Perhaps here's where the error message can change to be 'group-name'
instead of 'group'
ACK - (whether you adjust the error message or not)
John
> goto cleanup;
> }
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index 290f5083d99c..8512ff136265 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -1139,7 +1139,7 @@ command.
> [I<read-bytes-sec-max-length>] [I<write-bytes-sec-max-length>]]
> [[I<total-iops-sec-max-length>] |
> [I<read-iops-sec-max-length>] [I<write-iops-sec-max-length>]]
> -[I<size-iops-sec>] [I<group_name>]
> +[I<size-iops-sec>] [I<group-name>]
>
> Set or query the block disk io parameters for a block device of I<domain>.
> I<device> specifies a unique target name (<target dev='name'/>) or source
> @@ -1179,7 +1179,7 @@ read I/O operations limit.
> I<--write-iops-sec-max-length> specifies duration in seconds to allow maximum
> write I/O operations limit.
> I<--size-iops-sec> specifies size I/O operations limit per second.
> -I<--group_name> specifies group name to share I/O quota between multiple drives.
> +I<--group-name> specifies group name to share I/O quota between multiple drives.
> For a qemu domain, if no name is provided, then the default is to have a single
> group for each I<device>.
>
More information about the libvir-list
mailing list