Change in vdsm[master]: vm: event: status is an enumeration, not as tuple

fromani at redhat.com fromani at redhat.com
Fri Jul 10 09:13:14 UTC 2015


Francesco Romani has uploaded a new change for review.

Change subject: vm: event: status is an enumeration, not as tuple
......................................................................

vm: event: status is an enumeration, not as tuple

The current send status event code sends the vm status
as tuple (translated to array) not as single value enumeration.

I can't see why this is needed, so this patch simplifies the code.

Change-Id: I48b24c95209a22747e30367c1ef03377dac08b78
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/13/43413/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index e3aef9e..709a5b4 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -385,7 +385,7 @@
             if vm_status != self._evaluatedStatus:
                 self._evaluatedStatus = vm_status
                 current_status = self.lastStatus
-                stats['status'] = vm_status,
+                stats['status'] = vm_status
                 stats['hash'] = str(hash((self._domain.devices_hash,
                                     self.guestAgent.diskMappingHash)))
 


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

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