[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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'>
  <memory unit='KiB'>524288</memory>
  <currentMemory unit='KiB'>524288</currentMemory>
  <vcpu placement='static'>1</vcpu>
    <type arch='i686'>hvm</type>
  <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(-)

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]