Change in vdsm[master]: BZ#756693 The statsThread shouldn't take actions on VMs

danken at redhat.com danken at redhat.com
Sun Nov 27 15:11:56 UTC 2011


Dan Kenigsberg has posted comments on this change.

Change subject: BZ#756693 The statsThread shouldn't take actions on VMs
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

It is very sad that commit 7b51c3c, which worked around a libvirt bug, caused a vdsm regression :-(

Reverting it makes sense. However, events are evasive creatures by nature. Could we somehow handle the case of a lost event, without introducing regressions?

--
To view, visit http://gerrit.ovirt.org/392
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0c1a9abe69c61702ff21f41ce60079998b02d165
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list