Merge branch 'add-logs-MS-startup' into '4.16.0.0-scclouds'

Daniel Augusto Veronezi Salvador · Daniel Augusto Veronezi Salvador · commit 41e9bc065d05 · 2022-10-28T14:48:34.000-03:00
Adição de _logs_ no processo de inicialização do MS Closes apache#699 See merge request scclouds/scclouds!321
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
@@ -97,16 +97,17 @@ protected void startContexts() {
             @Override
             public void with(ModuleDefinition def, Stack<ModuleDefinition> parents) {
                 try {
+                    log.debug(String.format("Trying to obtain module [%s] context.", def.getName()));
                     ApplicationContext context = getApplicationContext(def.getName());
                     try {
                         Runnable runnable = context.getBean("moduleStartup", Runnable.class);
-                        log.info("Starting module [" + def.getName() + "]");
+                        log.info(String.format("Starting module [%s].", def.getName()));
                         runnable.run();
                     } catch (BeansException e) {
-                        // Ignore
+                        log.error(String.format("Failed to start module [%s] due to: [%s].", def.getName(), 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 +118,14 @@ protected void loadContexts() {
             @Override
             public void with(ModuleDefinition def, Stack<ModuleDefinition> parents) {
                 try {
+                    log.debug(String.format("Trying to obtain module [%s] context.", def.getName()));
                     ApplicationContext parent = getApplicationContext(parents.peek().getName());
+                    log.debug(String.format("Trying to load module [%s] context.", def.getName()));
                     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);
                 }
             }
         });
