[Libvirt-cim] [PATCH] Some minor std_association fixes

Dan Smith danms at us.ibm.com
Fri Nov 9 21:42:34 UTC 2007


KR> You mean, instead of returning success, we should set the status
KR> using cu_statusf().  Returning an error makes sense from a client
KR> point of view.  If the client thinks an association is going to
KR> return a specific instance, and nothing is returned, it won't know
KR> the state of the system.  If an error is returned, it's much more
KR> obvious to the client what has happened.

Right, if you return OK, but no instances, then it just looks like
there is nothing on the other end of your association, which would
normally be a transient state.  However, asking for a result class
that we don't support is not a transient error, it's a real one, and
we should signal as such.

Thanks for catching this and bringing it into the foreground for the
requisite beating :)

-- 
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: danms at us.ibm.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 188 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvirt-cim/attachments/20071109/f772b9b6/attachment.sig>


More information about the Libvirt-cim mailing list