Arik Hadas has uploaded a new change for review. Change subject: core: init os-info before the compensate phase ......................................................................
core: init os-info before the compensate phase Change-Id: I0a66a7d19ad6b2e7d8aaa9341f112a8ed3f0f5da Bug-Url: https://bugzilla.redhat.com/1109749 Signed-off-by: Arik Hadas <aha...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java 1 file changed, 5 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/36/29236/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java index fd9cd00..c73835b 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java @@ -169,6 +169,11 @@ @Override public void initialize() { log.info("Start initializing " + getClass().getSimpleName()); + // initialize configuration utils to use DB + Config.setConfigUtils(new DBConfigUtils()); + // we need to initialize os-info before the compensations take place because of VmPoolCommandBase#osRepository + initOsRepository(); + // When getting a proxy to this bean using JBoss embedded, the initialize method is called for each method // invocation on the proxy, as it is called by setup method which is @PostConstruct - the initialized flag // makes sure that initialization occurs only once per class (which is ok, as this is a @Service) @@ -178,8 +183,6 @@ compensate(); firstInitialization = false; } - // initialize configuration utils to use DB - Config.setConfigUtils(new DBConfigUtils()); log.info("Running ovirt-engine " + Config.<String>getValue(ConfigValues.ProductRPMVersion)); _resourceManger = new VDSBrokerFrontendImpl(); @@ -194,7 +197,6 @@ IsoDomainListSyncronizer.getInstance(); - initOsRepository(); initSearchDependencies(); initHandlers(); -- To view, visit http://gerrit.ovirt.org/29236 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0a66a7d19ad6b2e7d8aaa9341f112a8ed3f0f5da Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Arik Hadas <aha...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
