Change in vdsm[master]: api: make usage of doneCode uniform

fromani at redhat.com fromani at redhat.com
Thu Mar 5 15:50:56 UTC 2015


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 <fromani at 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 <fromani at redhat.com>


More information about the vdsm-patches mailing list