Bug 8342: Add info logging to ConfigManagerActivator 69/56469/3
authorTom Pantelis <tompantelis@gmail.com>
Tue, 2 May 2017 19:16:44 +0000 (15:16 -0400)
committerTom Pantelis <tompantelis@gmail.com>
Tue, 9 May 2017 12:24:54 +0000 (12:24 +0000)
Change-Id: I7b01961910dd2ba7ed9a421ee52e0aec29c68ade
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
(cherry picked from commit 1635f00212f8a583c6160cc3ff153e2c62af5092)

opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/osgi/ConfigManagerActivator.java

index 9d8694280b901cbb2f8d5375e33f76494f9e8bcd..eba7ad1a05314b572081fdd41c98b541f11f220c 100644 (file)
@@ -51,6 +51,7 @@ public class ConfigManagerActivator implements BundleActivator, SynchronousBundl
 
     @Override
     public void start(final BundleContext context) {
+        LOG.info("Config manager starting...");
         try {
             // the inner strategy is backed by thread context cl?
             final ModuleInfoBackedContext moduleInfoBackedContext = ModuleInfoBackedContext.create();
@@ -133,17 +134,20 @@ public class ConfigManagerActivator implements BundleActivator, SynchronousBundl
 
             context.addBundleListener(this);
         } catch(final Exception e) {
-            LOG.warn("Error starting config manager", e);
+            LOG.error("Error starting config manager", e);
         } catch(final Error e) {
             // Log JVM Error and re-throw. The OSGi container may silently fail the bundle and not always log
             // the exception. This has been seen on initial feature install.
             LOG.error("Error starting config manager", e);
             throw e;
         }
+
+        LOG.info("Config manager start complete");
     }
 
     @Override
     public void stop(final BundleContext context) throws Exception {
+        LOG.info("Config manager stopping");
         context.removeBundleListener(this);
         this.autoCloseable.close();
     }