Change in vdsm[master]: domainMonitor: Separate change detection from lastCheck value

amureini at redhat.com amureini at redhat.com
Tue Dec 3 18:10:20 UTC 2013


Allon Mureinik has posted comments on this change.

Change subject: domainMonitor: Separate change detection from lastCheck value
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

....................................................
Commit Message
Line 18: behavior is required for engine monitoring logic.
Line 19: 
Line 20: This patch restore the previous lastCheck semantics and use a new
Line 21: "checked" flag for detecting monitor status changes.
Line 22: 
Don't we have a Bug-Url for this?
Line 23: Change-Id: I8e0df2ee0333354146b5d9429fd83a4f914ff751


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8e0df2ee0333354146b5d9429fd83a4f914ff751
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list