[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