Arik Hadas has submitted this change and it was merged.
Change subject: core: minor cleanup in VmJobsMonitoring
......................................................................
core: minor cleanup in VmJobsMonitoring
1. Break long chain of calls in getExistingJobsForVm to
multiple lines.
2. Rename vmJobsToRemove and vmJobsToUpdate to jobsToRemove
and jobsToUpdate as in this context it is clear that it is
about VM-jobs.
Change-Id: I182927c2b9a693564b9780caed9cf7912e9702a0
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmJobsMonitoring.java
1 file changed, 10 insertions(+), 7 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit
https://gerrit.ovirt.org/56246
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I182927c2b9a693564b9780caed9cf7912e9702a0
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>