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