[virt-tools-list] Openvz guests

[Cole Robinson]

On 02/28/2011 03:20 PM, Guido Günther wrote:
> Hi,
> On Wed, Dec 01, 2010 at 04:07:23PM +0100, Guido Günther wrote:
>> On Wed, Nov 24, 2010 at 03:17:41PM -0500, Cleber Rosa wrote:
>>> Rich, 
>>>
>>> Are you sure there's GUI code for this on virt-manager? 
>>>
>>> I remember that not long ago there wasn't even code on virt-install to support this kind of thing... 
>>
>> With the attached patch you can at least start/stop OpenVZ containers.
>> The wizard needs more work since it doesn't allow for numeric vm names
>> and we need an option to not require a disk image. 
> 
> I wonder if this patch can be applied? It'd help to move OpenVZ
> integration forward.
> Cheers,
>  -- Guido
> 

I'm wary of listing hypervisor options that are poorly supported in the app,
for example I assume the new VM wizard doesn't work for openvz and may even
blow up in spectacular ways or create nonfunctional VMs.

At the very least major functionality that isn't implemented should show an
error message to that effect.

Another option might be to allow entering a manual URI in the connection
wizard, which is a bit more discoverable than the command line.

- Cole

>> Cheers
>>  -- Guido
>>
> 
>> # HG changeset patch
>> # Parent dedbf8d0d5a34399a2baa8422b0f154a1c243bb0
>> # User Guido Günther <agx at sigxcpu.org>
>> Allow to select OpenVZ connections in connection dialog.
>>
>> This allows to start/stop local/remote OpenVZ containers.
>>
>>
>> diff --git a/src/virtManager/connect.py b/src/virtManager/connect.py
>> --- a/src/virtManager/connect.py
>> +++ b/src/virtManager/connect.py
>> @@ -29,6 +29,7 @@
>>  
>>  HV_XEN = 0
>>  HV_QEMU = 1
>> +HV_OPENVZ = 2
>>  
>>  CONN_SSH = 0
>>  CONN_TCP = 1
>> @@ -290,6 +291,8 @@
>>          hvstr = ""
>>          if hv == HV_XEN:
>>              hvstr = "xen"
>> +        elif hv == HV_OPENVZ:
>> +            hvstr = "openvz"
>>          else:
>>              hvstr = "qemu"
>>  
>> @@ -311,7 +314,7 @@
>>              hoststr += addrstr + "/"
>>  
>>          uri = hvstr + hoststr
>> -        if hv == HV_QEMU:
>> +        if hv in [ HV_QEMU, HV_OPENVZ]:
>>              uri += "system"
>>  
>>          return uri
>> diff --git a/src/vmm-open-connection.glade b/src/vmm-open-connection.glade
>> --- a/src/vmm-open-connection.glade
>> +++ b/src/vmm-open-connection.glade
>> @@ -24,7 +24,8 @@
>>                <widget class="GtkComboBox" id="hypervisor">
>>                  <property name="visible">True</property>
>>                  <property name="items" translatable="yes">Xen
>> -QEMU/KVM</property>
>> +QEMU/KVM
>> +OpenVZ</property>
>>                  <accessibility>
>>                    <atkproperty name="AtkObject::accessible-name" translatable="yes">Hypervisor Select</atkproperty>
>>                  </accessibility>
> 
>> _______________________________________________
>> virt-tools-list mailing list
>> virt-tools-list at redhat.com
>> https://www.redhat.com/mailman/listinfo/virt-tools-list
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list