Re: [Libvir] [PATCH] patch for virsh domname <uuid> command error
On Thu, Mar 01, 2007 at 06:11:44PM +0900, Tatsuro Enokura wrote:
Hi, Dan
Daniel P. Berrange wrote:
>> and shut-off domain's information is got from xend.
>> I make a patch for xend_internal.c
>>
>> The attached patch resolve this issue in the following way:
>>
>> 1) If the domain's name can't be pulled out from
>> the running domains list by uuid, request xend by
>> "/xend/domains/<uuid>".
>> 2) If the domain's data can be teken from xend, pull out
>> the domain's name.
>
> It looks like a reasonable patch - I'll do a little testing and if
> it works on Fedora 7 I'll add it to CVS.
Could you tell me the test result on this patch?
If the test is fine, please commit it.
So, in previous XenD you were not able to request a domain based on UUID,
eg GET /xen/domain/3ef946bb4d5033a8cdba29f405de11ea
was not a valid URL served by XenD. So we basically did a get on /xen/domain/
to build a list of names, and then iterate over them, fetching each one in
turn until we found the one we wanted. This is obviously horribly slow,
but the only option we had on Xen < 3.0.4
Now, that XenD does allow lookups based on UUID, I am changing your patch
so that it *always* does the lookup based on UUID, so that we avoid the
slow path completely on Xen >= 3.0.4
I propose to apply the attached patch version - anyone else with comments ?
Dan.
--
|=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=|
|=- Perl modules: http://search.cpan.org/~danberr/ -=|
|=- Projects: http://freshmeat.net/~danielpb/ -=|
|=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|
Attachments:
- libvirt-uuid-lookup.patch (text/plain — 3.8 KB)