On 05/22/2015 09:18 PM, Maxim Nestratov wrote:
This new driver 'vz' will be used as a substitution for parallels driver. New domains will be created with 'vz' domain type. Old 'parallels' domains remain supported. Connection to the driver can be made either way: vz:///system or parallels:///system. New URI works fine (together with the old one). But the driver returns 'vz' as domain type in xml and doesn't allow parallels in input xml for *DefineXML function.
[root@dmserv libvirt]# virsh -c parallels+unix:///system dumpxml instance-00000007 | head <domain type='vz'> <name>instance-00000007</name> <uuid>3967ae06-9dc9-4717-990a-414f695c0734</uuid> <memory unit='KiB'>524288</memory> <currentMemory unit='KiB'>524288</currentMemory> <vcpu placement='static'>1</vcpu> <os> <type arch='i686'>hvm</type> </os> <clock offset='utc'/> Could, you, please, fix it? Daniel told we have to continue returning parallels.
Maxim Nestratov (8): parallels: introduce vz driver constant and string parallels: use newly introduced VIR_DOMAIN_VIRT_VZ parallels: add new guest capabilities assigned to vz driver parallels: accept vz as a driver uri and name parallels: add a new vz connection driver and hypervisor structures parallels: increment the number of connection drivers parallels: recommend to connect to vz:///system when connection fails parallels: set VIR_DOMAIN_VIRT_VZ virtType for new domains
src/conf/domain_conf.c | 19 +++++++++----- src/conf/domain_conf.h | 1 + src/libvirt.c | 2 +- src/parallels/parallels_driver.c | 52 ++++++++++++++++++++++++++++++++++----- src/parallels/parallels_network.c | 3 ++- src/parallels/parallels_sdk.c | 2 +- src/parallels/parallels_storage.c | 3 ++- 7 files changed, 66 insertions(+), 16 deletions(-)