Irit Goihman has uploaded a new change for review.
Change subject: jsonrpc: fix result dictionary
......................................................................
jsonrpc: fix result dictionary
Until now, jsonrpc command result held the data itself (not inside a
dict), which caused creating a faulty response representation that
wasn't consistent with xmlrpc commands returned results.
This patch fixes the command returned value and now it is
represented as a member in a dictionary with a corresponding key.
Change-Id: I985db6344fa7e34d78d38d7c6fe96a6b91839604
Signed-off-by: Irit Goihman <igoih...@redhat.com>
---
M lib/vdsm/rpc/Bridge.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/52/63052/1
diff --git a/lib/vdsm/rpc/Bridge.py b/lib/vdsm/rpc/Bridge.py
index bd180e0..62db38f 100644
--- a/lib/vdsm/rpc/Bridge.py
+++ b/lib/vdsm/rpc/Bridge.py
@@ -102,7 +102,7 @@
if member is None:
return None
try:
- return response[member]
+ return {member: response[member]}
except KeyError:
raise VdsmError(5, "Response is missing '%s' member" % member)
--
To view, visit https://gerrit.ovirt.org/63052
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I985db6344fa7e34d78d38d7c6fe96a6b91839604
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoih...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
