Arik Hadas has posted comments on this change. Change subject: vm: add missing vm id to get stats of down vm ......................................................................
Patch Set 2: @Francesco, I think the old code added the vm id automatically and now with json it is not. Verification: before this patch I hibernated a VM, no destroy call was sent by the engine because of the missing id so the VM remained (in down state) on the host. after applying this patch, destroy call was sent and the VM was removed from the host. In addition, Shahar verified it on his environment in migration flow. -- To view, visit http://gerrit.ovirt.org/37694 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9534f2105b9a32ebd4b28dae42e265094a3655b9 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches