Change in ovirt-engine[master]: core: ConfigUtils#getPackagesForCheckUpdate()

[amureini at redhat.com]

Allon Mureinik has submitted this change and it was merged.

Change subject: core: ConfigUtils#getPackagesForCheckUpdate()
......................................................................


core: ConfigUtils#getPackagesForCheckUpdate()

getPackagesForCheckUpdate() is part of the HostUpgradeManager's business
logic and is only ever used there. This patch moves it to its rightful
place in order to keep the commons' Config class as minimal as possible,
like common classes should be.

Change-Id: I04250b3ddc55836fd2d323bb4cfcaeb75bbea451
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/HostUpgradeManager.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/Config.java
2 files changed, 25 insertions(+), 29 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/49384
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I04250b3ddc55836fd2d323bb4cfcaeb75bbea451
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>