[Libvir] Re: [PATCH] Python bindings now generate exceptions for libvirt errors (third version)

[Daniel Veillard]

On Wed, Mar 28, 2007 at 03:13:28PM +0100, Daniel P. Berrange wrote:
> On Wed, Mar 28, 2007 at 07:07:32AM -0400, Daniel Veillard wrote:
> > If the fact of being inactive is only represented internally by having
> > id == -1 then we probably made a mistake, we should have
> >    domain->flags & DOMAIN_IS_DEFINED 
> > bit set.
> 
> We already have a state flag for it
> 
>   VIR_DOMAIN_SHUTOFF
> 
> which is set in virDomainInfoPtr, we can fix virt-manager not to call get_id
> if the domain is shutdown.

  It would be nice though if each time you look whether the domain is
defined or not, libvirt could give teh answer directly instead of going
though the potentially expensive Info call. This would also require
to double check invalidation of the _virDomain cached data at the 
proper places.

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard at redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/