Change in vdsm[master]: vdsm: variable name change to avoid naming conflict

mmirecki at redhat.com mmirecki at redhat.com
Fri Jul 3 11:37:45 UTC 2015


Marcin Mirecki has uploaded a new change for review.

Change subject: vdsm: variable name change to avoid naming conflict
......................................................................

vdsm: variable name change to avoid naming conflict

Changes:
'response' conflicts with respons.py for handling errors
replaced by 'result'

Change-Id: I6dce6b9e84edc347e64981b053d53de4091b7f6f
Signed-off-by: mmirecki <mmirecki at redhat.com>
Signed-off-by: Marcin Mirecki <mmirecki at redhat.com>
---
M vdsm/API.py
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/89/43189/1

diff --git a/vdsm/API.py b/vdsm/API.py
index 89f35ac..85d3340 100644
--- a/vdsm/API.py
+++ b/vdsm/API.py
@@ -571,10 +571,10 @@
         self.log.debug('Migration create')
 
         params['vmId'] = self._UUID
-        response = self.create(params)
-        if response['status']['code']:
+        result = self.create(params)
+        if result['status']['code']:
             self.log.debug('Migration create - Failed')
-            return response
+            return result
 
         v = self._cif.vmContainer.get(self._UUID)
 
@@ -583,7 +583,7 @@
 
         self.log.debug('Destination VM creation succeeded')
         return {'status': doneCode, 'migrationPort': 0,
-                'params': response['vmList']}
+                'params': result['vmList']}
 
     def monitorCommand(self, command):
         """


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6dce6b9e84edc347e64981b053d53de4091b7f6f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>


More information about the vdsm-patches mailing list