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.


Change-Id: I87ca49c91f1705abe55e044980442faf61b0ed22
Signed-off-by: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Bug-Url: https://bugzilla.redhat.com/1350763
---
M lib/yajsonrpc/__init__.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/48/59948/1

diff --git a/lib/yajsonrpc/__init__.py b/lib/yajsonrpc/__init__.py
index 87e7507..4399ab3 100644
--- a/lib/yajsonrpc/__init__.py
+++ b/lib/yajsonrpc/__init__.py
@@ -525,7 +525,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
-            return JsonRpcResponse(errCode['recovery'], None, req.id)
+            return JsonRpcResponse(None, errCode['recovery'], req.id)
 
         self.log.log(logLevel, "Calling '%s' in bridge with %s",
                      req.method, req.params)


-- 
To view, visit https://gerrit.ovirt.org/59948
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I87ca49c91f1705abe55e044980442faf61b0ed22
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczew...@gmail.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to