On Wed, Jul 16, 2008 at 08:54:25PM +0100, Daniel P. Berrange wrote: > 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 > okay, done :-) Daniel -- Red Hat Virtualization group http://redhat.com/virtualization/ Daniel Veillard | virtualization library http://libvirt.org/ veillard@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list