[Libvirt-cim] [PATCH] Some minor std_association fixes
Kaitlin Rupert
kaitlin at linux.vnet.ibm.com
Fri Nov 9 21:37:19 UTC 2007
Dan Smith wrote:
> If they try to pass an invalid reference (i.e. one we don't support an
> association for) we RETURN_UNSUPPORTED(). Shouldn't we do the same
> thing if they ask for a result that we don't support a transition to?
> Rather, shouldn't these two cases behave the same, no matter what that
> behavior should be?
>
You mean, instead of returning success, we should set the status using
cu_statusf(). Returning an error makes sense from a client point of
view. If the client thinks an association is going to return a specific
instance, and nothing is returned, it won't know the state of the
system. If an error is returned, it's much more obvious to the client
what has happened.
--
Kaitlin Rupert
IBM Linux Technology Center
karupert at us.ibm.com
More information about the Libvirt-cim
mailing list