Arik Hadas has uploaded a new change for review.
Change subject: vm: add missing vm id to get stats of down vm
......................................................................
vm: add missing vm id to get stats of down vm
The stats that are returned for a VM which is down does not contains the
id of the VM. That causes a regression since the engine uses this id,
without it hibernated VM is not destroyed, but remains down on the host.
Thus, adding the vm-id back.
Change-Id: I9534f2105b9a32ebd4b28dae42e265094a3655b9
Signed-off-by: Arik Hadas <aha...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/94/37694/1
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index dc513cf..6ff94b3 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -1756,6 +1756,7 @@
def _getExitedVmStats(self):
stats = {
+ 'vmId': self.conf['vmId'],
'exitCode': self.conf['exitCode'],
'status': self.lastStatus,
'exitMessage': self.conf['exitMessage'],
--
To view, visit http://gerrit.ovirt.org/37694
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9534f2105b9a32ebd4b28dae42e265094a3655b9
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
