Change in ovirt-engine[master]: core: decide whether to process domain monitoring in IrsProx...
Liron Aravot 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
Signed-off-by: Liron Aravot <laravot(a)redhat.com>
---
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:
Liron Aravot: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/45359
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2c87698477d6fbb7023b3e7f39eac4af62e91f64
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>