Francesco Romani has posted comments on this change.

Change subject: vm: add missing vm id to get stats of down vm
......................................................................


Patch Set 2: Code-Review+1

I don't see how it could be a regression, because in ovirt 3.4, before all the 
stats gathering refactoring, vmId was not here as well for Exited VMs.

However that's very minor, I'm fine with this patch.

-- 
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: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to