[libvirt] [PATCH] repeat lookup by name in LookupByID

[Daniel P. Berrange]

On Wed, Jul 16, 2008 at 03:53:29PM -0400, Daniel Veillard wrote:
> On Wed, Jul 16, 2008 at 08:10:21PM +0100, Daniel P. Berrange wrote:
> > 
> > Yes, the documentation is wrong - all inactive VMs have an ID
> > of -1, and thus lookup-by-ID is nonsensical for inactive VMs. 
> > 
> > If any application did make use of this change which falls back to
> > lookup-by-name, then it would only ever work with OpenVZ and not
> > any of the other libvirt drivers, which isn't useful behaviour.
> [...]
> > >   Then the virLookupById description must be updated, I'm not against it,
> > > but we need to be coherent.
> > 
> > Indeed, the docs need to be clarified.
> 
>   okay, what about
>    * Try to find a domain based on the hypervisor ID number
>    * Note that this won't work for inactive domains which have an ID of -1,
>    * in that case a lookup based on the Name or UUId need to be done instead.
> 
> and then revert that specific part of the patch, as attached. Also 
> I would make a 'make rebuild' in the doc directory and push the doc update

Yep, that gets my vote


Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|