Francesco Romani has uploaded a new change for review. Change subject: api: make usage of doneCode uniform ......................................................................
api: make usage of doneCode uniform The usage of doneCode in VDSM is a little ugly, but this patch makes it uniformly ugly, avoiding pointless (and not yielding better results) diversions from the common practice. Change-Id: Ib782500fe24a4dcc948bf52558bcd7b0d93b6f9e Signed-off-by: Francesco Romani <from...@redhat.com> --- M vdsm/API.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/23/38423/1 diff --git a/vdsm/API.py b/vdsm/API.py index 6b8da3a..5cbda50 100644 --- a/vdsm/API.py +++ b/vdsm/API.py @@ -1375,7 +1375,7 @@ isinstance(h, logging.FileHandler)] fileHandler.setLevel(int(level)) - return dict(status=doneCode) + return {'status': doneCode} # VM-related functions def getVMList(self, fullStatus=False, vmList=()): -- To view, visit https://gerrit.ovirt.org/38423 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib782500fe24a4dcc948bf52558bcd7b0d93b6f9e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <from...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches