X-Git-Url: https://code.grnet.gr/git/ganeti-local/blobdiff_plain/858f3d18118d59490cf3c01c0ceba44bfb05b646..a66bd91b7074b62884839ebff79bce9e237b39f1:/lib/rpc.py?ds=sidebyside diff --git a/lib/rpc.py b/lib/rpc.py index 98f0f6e..8101538 100644 --- a/lib/rpc.py +++ b/lib/rpc.py @@ -83,9 +83,6 @@ class RpcResult(object): failed, and therefore we use this class to encapsulate the result. @ivar data: the data payload, for successful results, or None - @type failed: boolean - @ivar failed: whether the operation failed at transport level (not - application level on the remote node) @ivar call: the name of the RPC call @ivar node: the name of the node to which we made the call @ivar offline: whether the operation failed because the node was @@ -97,12 +94,10 @@ class RpcResult(object): """ def __init__(self, data=None, failed=False, offline=False, call=None, node=None): - self.failed = failed self.offline = offline self.call = call self.node = node if offline: - self.failed = True self.fail_msg = "Node is marked offline" self.data = self.payload = None elif failed: @@ -152,14 +147,6 @@ class RpcResult(object): ec = errors.OpExecError raise ec(msg) - def RemoteFailMsg(self): - """Check if the remote procedure failed. - - @return: the fail_msg attribute - - """ - return self.fail_msg - class Client: """RPC Client class. @@ -810,7 +797,7 @@ class RpcRunner(object): """ result = self._SingleNodeCall(node, "blockdev_getmirrorstatus", [dsk.ToDict() for dsk in disks]) - if not result.failed: + if not result.fail_msg: result.payload = [objects.BlockDevStatus.FromDict(i) for i in result.payload] return result @@ -822,7 +809,7 @@ class RpcRunner(object): """ result = self._SingleNodeCall(node, "blockdev_find", [disk.ToDict()]) - if not result.failed and result.payload is not None: + if not result.fail_msg and result.payload is not None: result.payload = objects.BlockDevStatus.FromDict(result.payload) return result @@ -923,7 +910,7 @@ class RpcRunner(object): """ result = self._SingleNodeCall(node, "os_get", [name]) - if not result.failed and isinstance(result.data, dict): + if not result.fail_msg and isinstance(result.data, dict): result.data = objects.OS.FromDict(result.data) return result