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

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

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. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org

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
participants (2)
-
Eric Blake
-
Guido Günther