Re: [virt-manager PATCH] cli: add missing remove_first method for --blkiotune

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/25/2014 01:47 AM, Chen Hanxiao wrote:
> Add missing remove_first method for --blkiotune.
> Update related man pages.
> 
> Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx>
> ---
>  man/virt-install.pod | 2 +-
>  virtinst/cli.py      | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/man/virt-install.pod b/man/virt-install.pod
> index 3ccfa41..259a443 100644
> --- a/man/virt-install.pod
> +++ b/man/virt-install.pod
> @@ -164,7 +164,7 @@ Use --memtune=? to see a list of all available sub options. Complete details at
>  
>  Tune blkio policy for the domain process. Example invocations
>  
> -    --blkiotune weight=100
> +    --blkiotune 100
>      --blkiotune weight=100,device_path=/dev/sdc,device_weight=200
>  
>  Use --blkiotune=? to see a list of all available sub options. Complete details at L<http://libvirt.org/formatdomain.html#elementsBlockTuning>
> diff --git a/virtinst/cli.py b/virtinst/cli.py
> index 5bdc1de..826663a 100644
> --- a/virtinst/cli.py
> +++ b/virtinst/cli.py
> @@ -2026,6 +2026,7 @@ class ParserPanic(VirtCLIParser):
>  class ParserBlkiotune(VirtCLIParser):
>      def _init_params(self):
>          self.clear_attr = "blkiotune"
> +        self.remove_first = "weight"
>  
>          self.set_param("blkiotune.weight", "weight")
>          self.set_param("blkiotune.device_path", "device_path")
> 

ACK

- Cole

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux