Tal Nisan has submitted this change and it was merged. Change subject: core: decide whether to process domain monitoring in IrsProxyData ...................................................................... core: decide whether to process domain monitoring in IrsProxyData Currently the decision on whether it's needed to process the recevied monitoring data is done in IrsBrokerCommand, as this method is supposed to be used only as a proxy to the actual processing method - it makes more sense to have the logic on when we should consider the report that, otherwise each caller will need to copy that logic. Change-Id: I2c87698477d6fbb7023b3e7f39eac4af62e91f64 Bug-Url: https://bugzilla.redhat.com/1256841 Signed-off-by: Liron Aravot <laravot(a)redhat.com&gt; --- 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 2 files changed, 20 insertions(+), 11 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/48924 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c87698477d6fbb7023b3e7f39eac4af62e91f64 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;