Add and improve logs in ACS bootstrap process (#6890)

* Add and improve logs

* Fix unnecessary calls

---------

Co-authored-by: Stephan Krug <[email protected]>

Author: stephankruggg

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<ModuleDefinition> 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<ModuleDefinition> 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);
                 }
             }
         });