Re: [Libvirt-cim] [PATCH 2 of 3] Add raise_indication() to CSIndication
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(a)linux.vnet.ibm.com