Liron Aravot has submitted this change and it was merged. Change subject: core: determine whether domain monitoring result is actual ...................................................................... core: determine whether domain monitoring result is actual When the domain monitoring results are reported back from vdsm, the first monitor run may haven't finished yet. In that case the returned status from vdsm is a dummy status and not an actual result. The non actual status could lead to a wrong decision making on the engine status (for example - decide that an inactive domain is now active). This patch uses the logic added on vdsm side (change I1fea5) to inspect whether the reported status is actual or not. For earlier version that doesn't return that info, each returned report will be considered as actual to leave the behavior as it was. Change-Id: Iddbba94d81694bd6cb1ce3648872eb81182a50ab Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1183977 Signed-off-by: laravot@redhat.com <laravot@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSDomainsData.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 9 files changed, 61 insertions(+), 21 deletions(-) Approvals: Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38039 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddbba94d81694bd6cb1ce3648872eb81182a50ab Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server