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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>