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
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
