On 07/25/2012 03:59 AM, Yu Mingfei wrote:
> This patch adds virsh_domname() to libvirt_vm.py.
>
> Signed-off-by: Yu Mingfei <yuming...@cn.fujitsu.com>
> ---
>   client/virt/libvirt_vm.py |    9 +++++++++
>   1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/client/virt/libvirt_vm.py b/client/virt/libvirt_vm.py
> index 9a06f05..af7eb6d 100644
> --- a/client/virt/libvirt_vm.py
> +++ b/client/virt/libvirt_vm.py
> @@ -116,6 +116,15 @@ def virsh_cmd(cmd, uri="", ignore_status=False,
> print_info=False):
>       return ret
>
>
> +def virsh_domname(id, uri="", ignore_status=False, print_info=False):
> +    """
> +    Convert a domain id or UUID to domain name
> +
> +    @param id: a domain id or UUID.
> +    """
> +    return virsh_cmd("domname --domain %s" % id, uri, ignore_status,
> print_info)
> +
> +
>   def virsh_freecell(uri = "", ignore_status=False, extra = ""):
>       """
>       Prints the available amount of memory on the machine or within a
> NUMA cell.
> --
> 1.7.1
>
>
> --
> Best Regards
> Yu Mingfei
>

This looks fine to me, nice and simple and style is consistent.

-- 
Chris Evich, RHCA, RHCE, RHCDS, RHCSS
Quality Assurance Engineer
e-mail: cevich + `@' + redhat.com o: 1-888-RED-HAT1 x44214
_______________________________________________
Autotest mailing list
Autotest@test.kernel.org
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to