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

Re: [libvirt] [PATCH] OpenVZ: take veid from vmdef->name when defining new domains



On Mon, Nov 29, 2010 at 04:23:37PM -0700, Eric Blake wrote:
> On 11/28/2010 05:32 PM, Guido Günther wrote:
> > Hi,
> > We currently use the next free veid although there's one given in the
> > domain xml. This currently breaks defining new domains since vmdef->name
> > and veid don't match leading to the following error later on:
> > 
> >     error: Failed to define domain from 110.xml
> >     error: internal error Could not set UUID
> > 
> > Since silently ignoring vmdef->name is not nice respect it instead. We
> > avoid veid collisions in the upper levels already.
> > 
> > O.k. to apply?
> >  -- Guido
> 
> ACK.
Pushed now along with the (also ACK'ed) other openvz change and the
.gitignore fixup.
Thanks,
 -- Guido


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