Re: [libvirt] [PATCH 0/8] parallels: add vz driver
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(-)