Re: [PATCH 2/3] virt-manager: show the VM title in the manager window

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

 



On 09/10/2013 10:07 AM, Giuseppe Scrivano wrote:
> If a title is defined for a VM, append it to the name of the VM.
> 
> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
> ---
>  virtManager/manager.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/virtManager/manager.py b/virtManager/manager.py
> index 147c896..c481c7b 100644
> --- a/virtManager/manager.py
> +++ b/virtManager/manager.py
> @@ -747,7 +747,7 @@ class vmmManager(vmmGObjectUI):
>              color = self._build_conn_color(conn)
>              os_icon = None
>          else:
> -            name = vm.get_name()
> +            name = vm.get_descriptive_name()
>              status = vm.run_status()
>              markup = self._build_vm_markup(name, status)
>              status_icon = vm.run_status_icon_name()
> @@ -862,7 +862,7 @@ class vmmManager(vmmGObjectUI):
>              return
>  
>          try:
> -            name = vm.get_name()
> +            name = vm.get_descriptive_name()
>              status = vm.run_status()
>  
>              row[ROW_SORT_KEY] = name
> 

ACK in theory, but obviously this requires patch #1 which I had some comments on.

- 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