Piotr Kliczewski has uploaded a new change for review. Change subject: jsonrpc: recovery error passed as response ......................................................................
jsonrpc: recovery error passed as response When we notice that vdsm is in recovery mode we return information about recovery as result instead of returning error. Backport-To: 4.0 Backport-To: 3.6 Change-Id: I87ca49c91f1705abe55e044980442faf61b0ed22 Signed-off-by: Piotr Kliczewski <[email protected]> Bug-Url: https://bugzilla.redhat.com/1350763 Reviewed-on: https://gerrit.ovirt.org/59948 Reviewed-by: Oved Ourfali <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani <[email protected]> --- M lib/yajsonrpc/__init__.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/60008/1 diff --git a/lib/yajsonrpc/__init__.py b/lib/yajsonrpc/__init__.py index 83683cc..f824211 100644 --- a/lib/yajsonrpc/__init__.py +++ b/lib/yajsonrpc/__init__.py @@ -508,7 +508,7 @@ self.log.info("In recovery, ignoring '%s' in bridge with %s", req.method, req.params) # TODO: take the response from the exception instead of via errCode - ctx.requestDone(JsonRpcResponse(errCode['recovery'], None, req.id)) + ctx.requestDone(JsonRpcResponse(None, errCode['recovery'], req.id)) return self.log.log(logLevel, "Calling '%s' in bridge with %s", -- To view, visit https://gerrit.ovirt.org/60008 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I87ca49c91f1705abe55e044980442faf61b0ed22 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Piotr Kliczewski <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
