X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fconfig%2Fconfig-manager%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fconfig%2Fmanager%2Fimpl%2Fosgi%2FConfigManagerActivator.java;h=c075b09df76d5ba0fd770ec9af2e4adfb0c28b7c;hp=d464cb9006af15b24de4e1e517f57883a893fd47;hb=d9de6c2ddfb30eb2eee782c229f6e03cef352bbd;hpb=df2aabb0c61b9be04623c2a660c5a1592ff051f7 diff --git a/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/osgi/ConfigManagerActivator.java b/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/osgi/ConfigManagerActivator.java index d464cb9006..c075b09df7 100644 --- a/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/osgi/ConfigManagerActivator.java +++ b/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/osgi/ConfigManagerActivator.java @@ -18,7 +18,7 @@ import org.opendaylight.controller.config.manager.impl.jmx.ConfigRegistryJMXRegi import org.opendaylight.controller.config.manager.impl.osgi.mapping.ModuleInfoBundleTracker; import org.opendaylight.controller.config.manager.impl.osgi.mapping.RuntimeGeneratedMappingServiceActivator; import org.opendaylight.controller.config.spi.ModuleFactory; -import org.opendaylight.yangtools.concepts.Registration; +import org.opendaylight.yangtools.concepts.ObjectRegistration; import org.opendaylight.yangtools.yang.binding.YangModuleInfo; import org.opendaylight.yangtools.yang.data.impl.codec.CodecRegistry; import org.osgi.framework.BundleActivator; @@ -31,17 +31,15 @@ import org.slf4j.LoggerFactory; public class ConfigManagerActivator implements BundleActivator { private static final Logger logger = LoggerFactory.getLogger(ConfigManagerActivator.class); - private ExtensibleBundleTracker>> bundleTracker; + private final MBeanServer configMBeanServer = ManagementFactory.getPlatformMBeanServer(); + private ExtensibleBundleTracker>> bundleTracker; private ConfigRegistryImpl configRegistry; private ConfigRegistryJMXRegistrator configRegistryJMXRegistrator; - private ServiceRegistration configRegistryServiceRegistration; - - private final MBeanServer configMBeanServer = ManagementFactory.getPlatformMBeanServer(); - + private ServiceRegistration configRegistryServiceRegistration; private RuntimeGeneratedMappingServiceActivator mappingServiceActivator; @Override - public void start(BundleContext context) throws Exception { + public void start(BundleContext context) { // track bundles containing YangModuleInfo ModuleInfoBundleTracker moduleInfoBundleTracker = new ModuleInfoBundleTracker(); @@ -72,7 +70,7 @@ public class ConfigManagerActivator implements BundleActivator { try { configRegistryJMXRegistrator.registerToJMX(configRegistry); } catch (InstanceAlreadyExistsException e) { - throw new RuntimeException("Config Registry was already registered to JMX", e); + throw new IllegalStateException("Config Registry was already registered to JMX", e); } ServiceTracker serviceTracker = new ServiceTracker<>(context, ModuleFactory.class, @@ -81,7 +79,7 @@ public class ConfigManagerActivator implements BundleActivator { } @Override - public void stop(BundleContext context) throws Exception { + public void stop(BundleContext context) { try { configRegistry.close(); } catch (Exception e) {