[virt-tools-list] [PATCH v3 1/6] Add GUI to create wizzard for virtuozzo containers

[Mikhail Feoktistov]

On 01.03.2017 23:56, Cole Robinson wrote:
> On 02/21/2017 09:27 AM, Mikhail Feoktistov wrote:
>> Add virtuozzo hypervisor to connection list.
>> Add radio buttons for choosing VM or container virtualization type.
>> New wizzard window for setting template name for containers.
>> ---
>>   ui/create.ui              | 149 +++++++++++++++++++++++++++++++++++++++++++++-
>>   virtManager/connect.py    |   8 ++-
>>   virtManager/connection.py |   1 +
>>   virtManager/create.py     |  58 +++++++++++++++++-
>>   virtinst/connection.py    |   3 +
>>   5 files changed, 212 insertions(+), 7 deletions(-)
>>
> So since I don't have a virtuozzo setup, the only way to test this is to use
> the fake URL magic. So I did for example:
>
> virt-manager --connect
> __virtinst_test__test:////home/crobinso/src/virt-manager/tests/testdriver.xml,predictable,caps=/home/crobinso/src/virt-manager/tests/capabilities-xml/vz.xml,vz
>
> But when I launch the new VM wizard like that, only the 'container' option is
> available, but the wizard is stuck in that state and can't progress.
>
> I think the example capabilities XML tests/capabilities-xml/vz.xml should be
> expanded to list both container and VM install options. Please send a patch
> for that.
Done in patch series v4
> I suspect the UI will need some tweaking but I can't give specifics until I
> have something to play with a bit. But I think the first screen should be
> working similar to the xen UI for picking between paravirt and fullvirt. You
> can use a fake xen URL to play with it (adjust paths for your virt-manager dir):
>
> __virtinst_test__test:///home/crobinso/src/virt-manager/tests/testdriver.xml,xen,caps=/home/crobinso/src/virt-manager/tests/capabilities-xml/xen-rhel5.4.xml
My suggestion is to make two radion buttons like for LXC containers (os 
or app container)
1. This is the main choice and user should see it first
2. In case of container we disable all radio buttons below (local, 
network,pxe, etc) because
there is another step in the wizard where user can choose template name.
Now it's just a text box, but in the future we will add a drop box with 
all template names, which are present on the host.
Also we'll add text box for setting default password for container on 
this step.

If we do it in combo box like for Xen, it will be a bit confused for user.
Because he sees that all installation methods are disabled and at the 
same time
combo box is hidden in "Architecture options"  which are collapsed.
>
> Also have this patch stick to just the ui/create.ui and virtManager/create.py
> changes. If you combine the rest of the changes from this patch with patch #2
> and move it to the beginning of the series I can apply it independently
>
> Thanks,
> Cole