Francesco Romani has uploaded a new change for review.

Change subject: vm: remove unused 'output' value in response
......................................................................

vm: remove unused 'output' value in response

The 'output' value is not documented in schema, and apparently
Engine doesn't care about it. This patch removes it.

Change-Id: I0721885876fbe15651737aad4f3b903deb5ef4bb
Signed-off-by: Francesco Romani <from...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/16/44116/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 83d8ad8..164ca29 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -1168,7 +1168,7 @@
                     del self.conf['pauseCode']
             except KeyError:
                 pass
-            return {'status': doneCode, 'output': ['']}
+            return response.success()
         finally:
             if not guestCpuLocked:
                 self._guestCpuLock.release()
@@ -1185,7 +1185,7 @@
                                      guestCpuLocked=True)
             self._logGuestCpuStatus('pause')
             self._lastStatus = afterState
-            return {'status': doneCode, 'output': ['']}
+            return response.success()
         finally:
             if not guestCpuLocked:
                 self._guestCpuLock.release()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0721885876fbe15651737aad4f3b903deb5ef4bb
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

Reply via email to