Re: [libvirt] [PATCH 10/34] conf: Assume at least 1 maximum and current vCPU for every conf

2015-11-27 Thread Peter Krempa
On Mon, Nov 23, 2015 at 08:59:06 -0500, John Ferlan wrote: > On 11/20/2015 10:22 AM, Peter Krempa wrote: > > Set new domain configs to contain at least 1 vCPU add a check that > > maximum vCPU count isn't set to 0 and remove unnecesary checks. > > > > The openvz test suite change is necessary sinc

Re: [libvirt] [PATCH 10/34] conf: Assume at least 1 maximum and current vCPU for every conf

2015-11-23 Thread John Ferlan
On 11/20/2015 10:22 AM, Peter Krempa wrote: > Set new domain configs to contain at least 1 vCPU add a check that > maximum vCPU count isn't set to 0 and remove unnecesary checks. > > The openvz test suite change is necessary since the test case generates > the config via virDomainDefNew but does

[libvirt] [PATCH 10/34] conf: Assume at least 1 maximum and current vCPU for every conf

2015-11-20 Thread Peter Krempa
Set new domain configs to contain at least 1 vCPU add a check that maximum vCPU count isn't set to 0 and remove unnecesary checks. The openvz test suite change is necessary since the test case generates the config via virDomainDefNew but does not set the vCPU info. With the change to virDomainDefN