[Libvirt-cim] [PATCH 2 of 3] Add raise_indication() to CSIndication

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Tue Jun 30 20:19:25 UTC 2009


Sharad Mishra wrote:
> 
>  > +
>  > +        /* FIXME:  This is a Pegasus work around. Pegsus loses the 
> namespace
>  > +                   when an ObjectPath is pulled from an instance */
>  > +        if (STREQ(NAMESPACE(ref), ""))
>  > +                CMSetNameSpace(ref, "root/virt");
>  > +
> 
> Is it always "root/virt"? Seems like we are hard coding here.
> 
> -Sharad
> 

Agreed... this is an unfortunate work around for a Pegasus bug.  We 
should never need to hard code the namespace.  The namespace should be 
embedded in the ObjectPath appropriately.

However, Pegasus has a bug in which the namespace is striped from the 
ObjectPath whenever you pull it from an instance.  So until that issue 
is fixed, we don't have any other way to set the namespace appropriately.

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list