Dan Kenigsberg has submitted this change and it was merged. 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. This was broken by 7f577ab94b35f2cb2560d3ae49d6b0f753588474, which failed to put the vmId in the 'Exited Vm Stats', but only in the 'Running Vm Stats' 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 Bug-Url: https://bugzilla.redhat.com/1191405 Signed-off-by: Arik Hadas <[email protected]> Reviewed-on: http://gerrit.ovirt.org/37694 Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/37694 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9534f2105b9a32ebd4b28dae42e265094a3655b9 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
