diff --git a/framework/spring/module/src/main/java/org/apache/cloudstack/spring/module/model/impl/DefaultModuleDefinitionSet.java b/framework/spring/module/src/main/java/org/apache/cloudstack/spring/module/model/impl/DefaultModuleDefinitionSet.java index 5dab5d7bd349..ec9bbc0cd972 100644 --- a/framework/spring/module/src/main/java/org/apache/cloudstack/spring/module/model/impl/DefaultModuleDefinitionSet.java +++ b/framework/spring/module/src/main/java/org/apache/cloudstack/spring/module/model/impl/DefaultModuleDefinitionSet.java @@ -97,16 +97,18 @@ protected void startContexts() { @Override public void with(ModuleDefinition def, Stack parents) { try { - ApplicationContext context = getApplicationContext(def.getName()); + String moduleDefinitionName = def.getName(); + log.debug(String.format("Trying to obtain module [%s] context.", moduleDefinitionName)); + ApplicationContext context = getApplicationContext(moduleDefinitionName); try { Runnable runnable = context.getBean("moduleStartup", Runnable.class); - log.info("Starting module [" + def.getName() + "]"); + log.info(String.format("Starting module [%s].", moduleDefinitionName)); runnable.run(); } catch (BeansException e) { - // Ignore + log.error(String.format("Failed to start module [%s] due to: [%s].", moduleDefinitionName, e.getMessage()), e); } } catch (EmptyStackException e) { - // The root context is already loaded, so ignore the exception + log.error(String.format("Failed to obtain module context due to [%s]. Using root context instead.", e.getMessage()), e); } } }); @@ -117,10 +119,15 @@ protected void loadContexts() { @Override public void with(ModuleDefinition def, Stack parents) { try { + String moduleDefinitionName = def.getName(); + log.debug(String.format("Trying to obtain module [%s] context.", moduleDefinitionName)); ApplicationContext parent = getApplicationContext(parents.peek().getName()); + log.debug(String.format("Trying to load module [%s] context.", moduleDefinitionName)); loadContext(def, parent); } catch (EmptyStackException e) { - // The root context is already loaded, so ignore the exception + log.error(String.format("Failed to obtain module context due to [%s]. Using root context instead.", e.getMessage()), e); + } catch (BeansException e) { + log.error(String.format("Failed to load module [%s] due to: [%s].", def.getName(), e.getMessage()), e); } } });