Merge "Report (TCP) port number for switches"
[openflowplugin.git] / openflowplugin-impl / src / main / java / org / opendaylight / openflowplugin / impl / OpenFlowPluginProviderImpl.java
index 759e6ee67007b6af8380af2e62dedcddab6ebe1b..421c60ae8c31d54970c644b237dc34f887d58cc4 100644 (file)
@@ -8,20 +8,27 @@
 
 package org.opendaylight.openflowplugin.impl;
 
-
 import com.google.common.base.Preconditions;
 import com.google.common.util.concurrent.FutureCallback;
 import com.google.common.util.concurrent.Futures;
+import com.google.common.util.concurrent.ListenableFuture;
+import io.netty.util.HashedWheelTimer;
+import io.netty.util.Timer;
 import java.lang.management.ManagementFactory;
 import java.util.Collection;
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
+import java.util.concurrent.ExecutionException;
 import java.util.concurrent.SynchronousQueue;
 import java.util.concurrent.ThreadPoolExecutor;
 import java.util.concurrent.TimeUnit;
+import java.util.concurrent.TimeoutException;
+import java.util.function.Consumer;
 import java.util.stream.Collectors;
 import javax.annotation.Nonnull;
 import javax.management.InstanceAlreadyExistsException;
+import javax.management.InstanceNotFoundException;
 import javax.management.MBeanRegistrationException;
 import javax.management.MBeanServer;
 import javax.management.MalformedObjectNameException;
@@ -29,13 +36,15 @@ import javax.management.NotCompliantMBeanException;
 import javax.management.ObjectName;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.NotificationPublishService;
-import org.opendaylight.controller.md.sal.binding.api.NotificationService;
+import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipService;
 import org.opendaylight.controller.sal.binding.api.RpcProviderRegistry;
 import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider;
 import org.opendaylight.openflowjava.protocol.spi.connection.SwitchConnectionProvider;
+import org.opendaylight.openflowplugin.api.openflow.OpenFlowPluginConfigurationService;
 import org.opendaylight.openflowplugin.api.openflow.OpenFlowPluginProvider;
 import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionManager;
 import org.opendaylight.openflowplugin.api.openflow.device.DeviceManager;
+import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainHolder;
 import org.opendaylight.openflowplugin.api.openflow.rpc.RpcManager;
 import org.opendaylight.openflowplugin.api.openflow.statistics.StatisticsManager;
 import org.opendaylight.openflowplugin.api.openflow.statistics.ofpspecific.MessageIntelligenceAgency;
@@ -47,6 +56,7 @@ import org.opendaylight.openflowplugin.impl.connection.ConnectionManagerImpl;
 import org.opendaylight.openflowplugin.impl.device.DeviceManagerImpl;
 import org.opendaylight.openflowplugin.impl.device.initialization.DeviceInitializerProvider;
 import org.opendaylight.openflowplugin.impl.device.initialization.DeviceInitializerProviderFactory;
+import org.opendaylight.openflowplugin.impl.lifecycle.ContextChainHolderImpl;
 import org.opendaylight.openflowplugin.impl.protocol.deserialization.DeserializerInjector;
 import org.opendaylight.openflowplugin.impl.protocol.serialization.SerializerInjector;
 import org.opendaylight.openflowplugin.impl.rpc.RpcManagerImpl;
@@ -61,88 +71,85 @@ import org.opendaylight.openflowplugin.openflow.md.core.sal.convertor.ConvertorM
 import org.opendaylight.openflowplugin.openflow.md.core.session.OFSessionUtil;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import io.netty.util.HashedWheelTimer;
 
-public class OpenFlowPluginProviderImpl implements OpenFlowPluginProvider, OpenFlowPluginExtensionRegistratorProvider {
+public class OpenFlowPluginProviderImpl implements OpenFlowPluginProvider, OpenFlowPluginConfigurationService, OpenFlowPluginExtensionRegistratorProvider {
 
     private static final Logger LOG = LoggerFactory.getLogger(OpenFlowPluginProviderImpl.class);
-    private static final MessageIntelligenceAgency messageIntelligenceAgency = new MessageIntelligenceAgencyImpl();
-    private static final int TICKS_PER_WHEEL = 500;
-    // 0.5 sec.
+
+    private static final int TICKS_PER_WHEEL = 500; // 0.5 sec.
     private static final long TICK_DURATION = 10;
-    private static final Integer DEFAULT_BARRIER_COUNT = 25600;
-    private static final Long DEFAULT_ECHO_TIMEOUT = 2000L;
-    private static final Long DEFAULT_BARRIER_TIMEOUT = 500L;
+    private static final String POOL_NAME = "ofppool";
 
-    private final HashedWheelTimer hashedWheelTimer = new HashedWheelTimer(TICK_DURATION, TimeUnit.MILLISECONDS, TICKS_PER_WHEEL);
+    private static final MessageIntelligenceAgency MESSAGE_INTELLIGENCE_AGENCY = new MessageIntelligenceAgencyImpl();
+    private static final String MESSAGE_INTELLIGENCE_AGENCY_MX_BEAN_NAME = String
+            .format("%s:type=%s",
+                    MessageIntelligenceAgencyMXBean.class.getPackage().getName(),
+                    MessageIntelligenceAgencyMXBean.class.getSimpleName());
 
-    private final int rpcRequestsQuota;
-    private final long globalNotificationQuota;
+    private final HashedWheelTimer hashedWheelTimer = new HashedWheelTimer(TICK_DURATION, TimeUnit.MILLISECONDS, TICKS_PER_WHEEL);
+    private final NotificationPublishService notificationPublishService;
+    private final ExtensionConverterManager extensionConverterManager;
+    private final DataBroker dataBroker;
+    private final Collection<SwitchConnectionProvider> switchConnectionProviders;
+    private final DeviceInitializerProvider deviceInitializerProvider;
     private final ConvertorManager convertorManager;
+    private final RpcProviderRegistry rpcProviderRegistry;
+    private final ClusterSingletonServiceProvider singletonServicesProvider;
+    private final EntityOwnershipService entityOwnershipService;
+    private ContextChainHolder contextChainHolder;
+    private int rpcRequestsQuota;
+    private long globalNotificationQuota;
     private long barrierInterval;
     private int barrierCountLimit;
     private long echoReplyTimeout;
     private DeviceManager deviceManager;
     private RpcManager rpcManager;
-    private RpcProviderRegistry rpcProviderRegistry;
     private StatisticsManager statisticsManager;
     private ConnectionManager connectionManager;
-    private NotificationService notificationProviderService;
-    private NotificationPublishService notificationPublishService;
-    private ExtensionConverterManager extensionConverterManager;
-    private DataBroker dataBroker;
-    private Collection<SwitchConnectionProvider> switchConnectionProviders;
-    private boolean switchFeaturesMandatory = false;
-    private boolean isStatisticsPollingOn = true;
+    private boolean switchFeaturesMandatory;
+    private boolean isStatisticsPollingOn;
     private boolean isStatisticsRpcEnabled;
-    private boolean isFlowRemovedNotificationOn = true;
-    private boolean skipTableFeatures = true;
+    private boolean isFlowRemovedNotificationOn;
+    private boolean skipTableFeatures;
     private long basicTimerDelay;
     private long maximumTimerDelay;
-    private boolean useSingleLayerSerialization = false;
-    private final DeviceInitializerProvider deviceInitializerProvider;
+    private boolean useSingleLayerSerialization;
+    private ThreadPoolExecutor threadPool;
+    private int threadPoolMinThreads;
+    private int threadPoolMaxThreads;
+    private long threadPoolTimeout;
+    private boolean initialized = false;
 
-    private final ThreadPoolExecutor threadPool;
-    private ClusterSingletonServiceProvider singletonServicesProvider;
-
-    public OpenFlowPluginProviderImpl(final long rpcRequestsQuota,
-                                      final long globalNotificationQuota,
-                                      final int threadPoolMinThreads,
-                                      final int threadPoolMaxThreads,
-                                      final long threadPoolTimeout) {
-        Preconditions.checkArgument(rpcRequestsQuota > 0 && rpcRequestsQuota <= Integer.MAX_VALUE, "rpcRequestQuota has to be in range <1,%s>", Integer.MAX_VALUE);
-        this.rpcRequestsQuota = (int) rpcRequestsQuota;
-        this.globalNotificationQuota = Preconditions.checkNotNull(globalNotificationQuota);
-
-        // Creates a thread pool that creates new threads as needed, but will reuse previously
-        // constructed threads when they are available.
-        // Threads that have not been used for x seconds are terminated and removed from the cache.
-        threadPool = new ThreadPoolLoggingExecutor(
-                Preconditions.checkNotNull(threadPoolMinThreads),
-                Preconditions.checkNotNull(threadPoolMaxThreads),
-                Preconditions.checkNotNull(threadPoolTimeout), TimeUnit.SECONDS,
-                new SynchronousQueue<>(), "ofppool");
+    public static MessageIntelligenceAgency getMessageIntelligenceAgency() {
+        return MESSAGE_INTELLIGENCE_AGENCY;
+    }
 
+    OpenFlowPluginProviderImpl(final List<SwitchConnectionProvider> switchConnectionProviders,
+                               final DataBroker dataBroker,
+                               final RpcProviderRegistry rpcProviderRegistry,
+                               final NotificationPublishService notificationPublishService,
+                               final ClusterSingletonServiceProvider singletonServiceProvider,
+                               final EntityOwnershipService entityOwnershipService) {
+        this.switchConnectionProviders = switchConnectionProviders;
+        this.dataBroker = dataBroker;
+        this.rpcProviderRegistry = rpcProviderRegistry;
+        this.notificationPublishService = notificationPublishService;
+        this.singletonServicesProvider = singletonServiceProvider;
+        this.entityOwnershipService = entityOwnershipService;
         convertorManager = ConvertorManagerFactory.createDefaultManager();
+        extensionConverterManager = new ExtensionConverterManagerImpl();
         deviceInitializerProvider = DeviceInitializerProviderFactory.createDefaultProvider();
     }
 
-    @Override
-    public boolean isStatisticsPollingOn() {
-        return isStatisticsPollingOn;
-    }
-
-    @Override
-    public void setStatisticsPollingOn(final boolean isStatisticsPollingOn) {
-        this.isStatisticsPollingOn = isStatisticsPollingOn;
-    }
 
     private void startSwitchConnections() {
         Futures.addCallback(Futures.allAsList(switchConnectionProviders.stream().map(switchConnectionProvider -> {
-            // Inject OpenflowPlugin custom serializers and deserializers into OpenflowJava
+            // Inject OpenFlowPlugin custom serializers and deserializers into OpenFlowJava
             if (useSingleLayerSerialization) {
                 SerializerInjector.injectSerializers(switchConnectionProvider);
                 DeserializerInjector.injectDeserializers(switchConnectionProvider);
+            } else {
+                DeserializerInjector.revertDeserializers(switchConnectionProvider);
             }
 
             // Set handler of incoming connections and start switch connection provider
@@ -155,221 +162,405 @@ public class OpenFlowPluginProviderImpl implements OpenFlowPluginProvider, OpenF
             }
 
             @Override
-            public void onFailure(@Nonnull final Throwable t) {
-                LOG.warn("Some switchConnectionProviders failed to start.", t);
+            public void onFailure(@Nonnull final Throwable throwable) {
+                LOG.warn("Some switchConnectionProviders failed to start.", throwable);
             }
         });
     }
 
-    @Override
-    public boolean isSwitchFeaturesMandatory() {
-        return switchFeaturesMandatory;
-    }
-
-    @Override
-    public void setBarrierCountLimit(final int barrierCountLimit) {
-        this.barrierCountLimit = barrierCountLimit;
-    }
-
-    @Override
-    public void setBarrierInterval(final long barrierTimeoutLimit) {
-        this.barrierInterval = barrierTimeoutLimit;
-    }
-
-    @Override
-    public void setEchoReplyTimeout(final long echoReplyTimeout) {
-        this.echoReplyTimeout = echoReplyTimeout;
-    }
-
-    @Override
-    public void setFlowRemovedNotification(boolean isFlowRemovedNotificationOn) {
-        this.isFlowRemovedNotificationOn = isFlowRemovedNotificationOn;
-    }
-
-    @Override
-    public void setClusteringSingletonServicesProvider(ClusterSingletonServiceProvider singletonServicesProvider) {
-        this.singletonServicesProvider = singletonServicesProvider;
-    }
-
-    @Override
-    public void setSkipTableFeatures(final boolean skipTableFeatures){
-        this.skipTableFeatures = skipTableFeatures;
-    }
-
-    @Override
-    public void setBasicTimerDelay(long basicTimerDelay) {
-        this.basicTimerDelay = basicTimerDelay;
-    }
-
-    @Override
-    public void setMaximumTimerDelay(long maximumTimerDelay) {
-        this.maximumTimerDelay = maximumTimerDelay;
-    }
-
-    @Override
-    public void setSwitchFeaturesMandatory(final boolean switchFeaturesMandatory) {
-        this.switchFeaturesMandatory = switchFeaturesMandatory;
-    }
+    private ListenableFuture<List<Boolean>> shutdownSwitchConnections() {
+        final ListenableFuture<List<Boolean>> listListenableFuture = Futures.allAsList(switchConnectionProviders.stream().map(switchConnectionProvider -> {
+            // Revert deserializers to their original state
+            if (useSingleLayerSerialization) {
+                DeserializerInjector.revertDeserializers(switchConnectionProvider);
+            }
 
-    public static MessageIntelligenceAgency getMessageIntelligenceAgency() {
-        return OpenFlowPluginProviderImpl.messageIntelligenceAgency;
-    }
+            // Shutdown switch connection provider
+            return switchConnectionProvider.shutdown();
+        }).collect(Collectors.toSet()));
 
-    @Override
-    public void setSwitchConnectionProviders(final Collection<SwitchConnectionProvider> switchConnectionProviders) {
-        this.switchConnectionProviders = switchConnectionProviders;
-    }
+        Futures.addCallback(listListenableFuture, new FutureCallback<List<Boolean>>() {
+            @Override
+            public void onSuccess(final List<Boolean> result) {
+                LOG.info("All switchConnectionProviders were successfully shut down ({}).", result.size());
+            }
 
-    @Override
-    public void setDataBroker(final DataBroker dataBroker) {
-        this.dataBroker = dataBroker;
-    }
+            @Override
+            public void onFailure(@Nonnull final Throwable throwable) {
+                LOG.warn("Some switchConnectionProviders failed to shutdown.", throwable);
+            }
+        });
 
-    @Override
-    public void setRpcProviderRegistry(final RpcProviderRegistry rpcProviderRegistry) {
-        this.rpcProviderRegistry = rpcProviderRegistry;
+        return listListenableFuture;
     }
 
     @Override
     public void initialize() {
-        Preconditions.checkNotNull(dataBroker, "missing data broker");
-        Preconditions.checkNotNull(rpcProviderRegistry, "missing RPC provider registry");
-        Preconditions.checkNotNull(notificationProviderService, "missing notification provider service");
-        Preconditions.checkNotNull(singletonServicesProvider, "missing singleton services provider");
-
-        extensionConverterManager = new ExtensionConverterManagerImpl();
         // TODO: copied from OpenFlowPluginProvider (Helium) misusesing the old way of distributing extension converters
         // TODO: rewrite later!
         OFSessionUtil.getSessionManager().setExtensionConverterProvider(extensionConverterManager);
 
-        connectionManager = new ConnectionManagerImpl(echoReplyTimeout, threadPool);
+        // Creates a thread pool that creates new threads as needed, but will reuse previously
+        // constructed threads when they are available.
+        // Threads that have not been used for x seconds are terminated and removed from the cache.
+        threadPool = new ThreadPoolLoggingExecutor(
+                Preconditions.checkNotNull(threadPoolMinThreads),
+                Preconditions.checkNotNull(threadPoolMaxThreads),
+                Preconditions.checkNotNull(threadPoolTimeout),
+                TimeUnit.SECONDS, new SynchronousQueue<>(), POOL_NAME);
 
-        registerMXBean(messageIntelligenceAgency);
 
-        deviceManager = new DeviceManagerImpl(dataBroker,
-                globalNotificationQuota,
-                switchFeaturesMandatory,
-                barrierInterval,
-                barrierCountLimit,
+        contextChainHolder = new ContextChainHolderImpl(hashedWheelTimer, threadPool);
+        contextChainHolder.changeEntityOwnershipService(entityOwnershipService);
+
+        connectionManager = new ConnectionManagerImpl(threadPool);
+        connectionManager.setEchoReplyTimeout(echoReplyTimeout);
+
+        registerMXBean(MESSAGE_INTELLIGENCE_AGENCY, MESSAGE_INTELLIGENCE_AGENCY_MX_BEAN_NAME);
+
+        contextChainHolder.addSingletonServicesProvider(singletonServicesProvider);
+
+        deviceManager = new DeviceManagerImpl(
+                dataBroker,
                 getMessageIntelligenceAgency(),
-                isFlowRemovedNotificationOn,
-                singletonServicesProvider,
                 notificationPublishService,
                 hashedWheelTimer,
                 convertorManager,
-                skipTableFeatures,
-                useSingleLayerSerialization,
-                deviceInitializerProvider);
+                deviceInitializerProvider,
+                useSingleLayerSerialization);
+
+        deviceManager.setGlobalNotificationQuota(globalNotificationQuota);
+        deviceManager.setSwitchFeaturesMandatory(switchFeaturesMandatory);
+        deviceManager.setBarrierInterval(barrierInterval);
+        deviceManager.setBarrierCountLimit(barrierCountLimit);
+        deviceManager.setFlowRemovedNotificationOn(isFlowRemovedNotificationOn);
+        deviceManager.setSkipTableFeatures(skipTableFeatures);
 
         ((ExtensionConverterProviderKeeper) deviceManager).setExtensionConverterProvider(extensionConverterManager);
 
-        rpcManager = new RpcManagerImpl(rpcProviderRegistry, rpcRequestsQuota, extensionConverterManager, convertorManager, notificationPublishService);
-        statisticsManager = new StatisticsManagerImpl(rpcProviderRegistry, isStatisticsPollingOn, hashedWheelTimer,
-                convertorManager,basicTimerDelay,maximumTimerDelay);
+        rpcManager = new RpcManagerImpl(rpcProviderRegistry, extensionConverterManager, convertorManager, notificationPublishService);
+        rpcManager.setRpcRequestQuota(rpcRequestsQuota);
 
-        /* Initialization Phase ordering - OFP Device Context suite */
-        // CM -> DM -> SM -> RPC -> Role -> DM
-        connectionManager.setDeviceConnectedHandler(deviceManager);
-        deviceManager.setDeviceInitializationPhaseHandler(statisticsManager);
-        statisticsManager.setDeviceInitializationPhaseHandler(rpcManager);
-        rpcManager.setDeviceInitializationPhaseHandler(deviceManager);
+        statisticsManager = new StatisticsManagerImpl(rpcProviderRegistry, hashedWheelTimer, convertorManager);
+        statisticsManager.setBasicTimerDelay(basicTimerDelay);
+        statisticsManager.setMaximumTimerDelay(maximumTimerDelay);
+        statisticsManager.setIsStatisticsPollingOn(isStatisticsPollingOn);
+
+        // Device connection handler moved from device manager to context holder
+        connectionManager.setDeviceConnectedHandler(contextChainHolder);
 
         /* Termination Phase ordering - OFP Device Context suite */
-        deviceManager.setDeviceTerminationPhaseHandler(rpcManager);
-        rpcManager.setDeviceTerminationPhaseHandler(statisticsManager);
-        statisticsManager.setDeviceTerminationPhaseHandler(deviceManager);
+        connectionManager.setDeviceDisconnectedHandler(contextChainHolder);
 
         rpcManager.setStatisticsRpcEnabled(isStatisticsRpcEnabled);
 
         TranslatorLibraryUtil.injectBasicTranslatorLibrary(deviceManager, convertorManager);
         deviceManager.initialize();
 
+        contextChainHolder.addManager(deviceManager);
+        contextChainHolder.addManager(statisticsManager);
+        contextChainHolder.addManager(rpcManager);
+
         startSwitchConnections();
+        initialized = true;
     }
 
+
     @Override
-    public void update(Map<String,Object> props) {
-        LOG.debug("Update managed properties = {}", props.toString());
+    public void update(@Nonnull final Map<String, Object> properties) {
+        properties.forEach((key, value) -> {
+            final PropertyType propertyType = PropertyType.forValue(key);
 
-        final boolean containsUseSingleLayer = props.containsKey("use-single-layer-serialization");
+            if (Objects.nonNull(propertyType)) {
+                updateProperty(propertyType, value);
+            }
+        });
+    }
 
-        if (containsUseSingleLayer) {
-            final Boolean useSingleLayer = Boolean.valueOf(props.get("use-single-layer-serialization").toString());
+    private void doPropertyUpdate(final PropertyType propertyType,
+                                  final boolean modifiable,
+                                  final Object origValue,
+                                  final Object newValue,
+                                  final Consumer<Object> successCallback) {
+        if (initialized) {
+            if (Objects.equals(origValue, newValue)) {
+                LOG.debug("{} config parameter is already set to {})", propertyType, origValue);
+                return;
+            } else if (!modifiable) {
+                LOG.warn("{} update ({} -> {}) is not allowed after controller start", propertyType, origValue, newValue);
+                return;
+            }
+        }
 
-            if (useSingleLayer != useSingleLayerSerialization) {
-                useSingleLayerSerialization = useSingleLayer;
+        successCallback.accept(newValue);
+        LOG.info("{} config parameter is updated ({} -> {})", propertyType, origValue, newValue);
+    }
 
-                if (useSingleLayer) {
-                    switchConnectionProviders.forEach(switchConnectionProvider -> {
-                        SerializerInjector.injectSerializers(switchConnectionProvider);
-                        DeserializerInjector.injectDeserializers(switchConnectionProvider);
-                    });
-                } else {
-                    switchConnectionProviders.forEach(DeserializerInjector::revertDeserializers);
-                }
+    @Override
+    public void updateProperty(@Nonnull final PropertyType key, @Nonnull final Object value) {
+        try {
+            final String sValue = value.toString();
+            final Consumer<Object> successCallback;
+            final boolean modifiable;
+            final Object oldValue;
+            final Object newValue;
+
+            switch (key) {
+                case RPC_REQUESTS_QUOTA:
+                    successCallback = (result) -> {
+                        rpcRequestsQuota = (int) result;
+
+                        if (initialized) {
+                            rpcManager.setRpcRequestQuota(rpcRequestsQuota);
+                        }
+                    };
+
+                    oldValue = rpcRequestsQuota;
+                    newValue = Integer.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case SWITCH_FEATURES_MANDATORY:
+                    successCallback = (result) -> {
+                        switchFeaturesMandatory = (boolean) result;
+
+                        if (initialized) {
+                            deviceManager.setSwitchFeaturesMandatory(switchFeaturesMandatory);
+                        }
+                    };
+
+                    oldValue = switchFeaturesMandatory;
+                    newValue = Boolean.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case GLOBAL_NOTIFICATION_QUOTA:
+                    successCallback = (result) -> {
+                        globalNotificationQuota = (long) result;
+
+                        if (initialized) {
+                            deviceManager.setGlobalNotificationQuota(globalNotificationQuota);
+                        }
+                    };
+
+                    oldValue = globalNotificationQuota;
+                    newValue = Long.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case IS_STATISTICS_POLLING_ON:
+                    successCallback = (result) -> {
+                        isStatisticsPollingOn = (boolean) result;
+
+                        if (initialized) {
+                            statisticsManager.setIsStatisticsPollingOn(isStatisticsPollingOn);
+                        }
+                    };
+
+                    oldValue = isStatisticsPollingOn;
+                    newValue = Boolean.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case IS_STATISTICS_RPC_ENABLED:
+                    successCallback = (result) -> {
+                        isStatisticsRpcEnabled = (boolean) result;
+
+                        if (initialized) {
+                            rpcManager.setStatisticsRpcEnabled(isStatisticsRpcEnabled);
+                        }
+                    };
+
+                    oldValue = isStatisticsRpcEnabled;
+                    newValue = Boolean.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case BARRIER_INTERVAL_TIMEOUT_LIMIT:
+                    successCallback = (result) -> {
+                        barrierInterval = (long) result;
+
+                        if (initialized) {
+                            deviceManager.setBarrierInterval(barrierInterval);
+                        }
+                    };
+
+                    oldValue = barrierInterval;
+                    newValue = Long.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case BARRIER_COUNT_LIMIT:
+                    successCallback = (result) -> {
+                        barrierCountLimit = (int) result;
+
+                        if (initialized) {
+                            deviceManager.setBarrierCountLimit(barrierCountLimit);
+                        }
+                    };
+
+                    oldValue = barrierCountLimit;
+                    newValue = Integer.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case ECHO_REPLY_TIMEOUT:
+                    successCallback = (result) -> {
+                        echoReplyTimeout = (long) result;
+
+                        if (initialized) {
+                            connectionManager.setEchoReplyTimeout(echoReplyTimeout);
+                        }
+                    };
+
+                    oldValue = echoReplyTimeout;
+                    newValue = Long.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case THREAD_POOL_MIN_THREADS:
+                    successCallback = (result) -> threadPoolMinThreads = (int) result;
+                    oldValue = threadPoolMinThreads;
+                    newValue = Integer.valueOf(sValue);
+                    modifiable = false;
+                    break;
+                case THREAD_POOL_MAX_THREADS:
+                    successCallback = (result) -> threadPoolMaxThreads = (int) result;
+                    oldValue = threadPoolMaxThreads;
+                    newValue = Integer.valueOf(sValue);
+                    modifiable = false;
+                    break;
+                case THREAD_POOL_TIMEOUT:
+                    successCallback = (result) -> threadPoolTimeout = (long) result;
+                    oldValue = threadPoolTimeout;
+                    newValue = Long.valueOf(sValue);
+                    modifiable = false;
+                    break;
+                case ENABLE_FLOW_REMOVED_NOTIFICATION:
+                    successCallback = (result) -> {
+                        isFlowRemovedNotificationOn = (boolean) result;
+
+                        if (initialized) {
+                            deviceManager.setFlowRemovedNotificationOn(isFlowRemovedNotificationOn);
+                        }
+                    };
+
+                    oldValue = isFlowRemovedNotificationOn;
+                    newValue = Boolean.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case SKIP_TABLE_FEATURES:
+                    successCallback = (result) -> {
+                        skipTableFeatures = (boolean) result;
+
+                        if (initialized) {
+                            deviceManager.setSkipTableFeatures(skipTableFeatures);
+                        }
+                    };
+
+                    oldValue = skipTableFeatures;
+                    newValue = Boolean.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case BASIC_TIMER_DELAY:
+                    successCallback = (result) -> {
+                        basicTimerDelay = (long) result;
+
+                        if (initialized) {
+                            statisticsManager.setBasicTimerDelay(basicTimerDelay);
+                        }
+                    };
+
+                    oldValue = basicTimerDelay;
+                    newValue = Long.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case MAXIMUM_TIMER_DELAY:
+                    successCallback = (result) -> {
+                        maximumTimerDelay = (long) result;
+
+                        if (initialized) {
+                            statisticsManager.setMaximumTimerDelay(maximumTimerDelay);
+                        }
+                    };
+
+                    oldValue = maximumTimerDelay;
+                    newValue = Long.valueOf(sValue);
+                    modifiable = true;
+                    break;
+                case USE_SINGLE_LAYER_SERIALIZATION:
+                    successCallback = (result) -> useSingleLayerSerialization = (boolean) result;
+                    oldValue = useSingleLayerSerialization;
+                    newValue = Boolean.valueOf(sValue);
+                    modifiable = false;
+                    break;
+                default:
+                    return;
             }
+
+            doPropertyUpdate(key, modifiable, oldValue, newValue, successCallback);
+        } catch (final Exception ex) {
+            LOG.warn("Failed to read configuration property '{}={}', error: {}", key, value, ex);
         }
+    }
 
-        if(deviceManager != null) {
-            if (containsUseSingleLayer) {
-                deviceManager.setUseSingleLayerSerialization(Boolean.valueOf(props.get("use-single-layer-serialization").toString()));
-            }
+    @Override
+    public ExtensionConverterRegistrator getExtensionConverterRegistrator() {
+        return extensionConverterManager;
+    }
 
-            if (props.containsKey("notification-flow-removed-off")) {
-                deviceManager.setFlowRemovedNotificationOn(Boolean.valueOf(props.get("enable-flow-removed-notification").toString()));
-            }
-            if (props.containsKey("skip-table-features")) {
-                deviceManager.setSkipTableFeatures(Boolean.valueOf(props.get("skip-table-features").toString()));
-            }
-            if (props.containsKey("barrier-count-limit")) {
-                try {
-                    deviceManager.setBarrierCountLimit(Integer.valueOf(props.get("barrier-count-limit").toString()));
-                } catch (NumberFormatException ex) {
-                    deviceManager.setBarrierCountLimit(DEFAULT_BARRIER_COUNT);
-                }
-            }
-            if (props.containsKey("barrier-interval-timeout-limit")){
-                try {
-                    deviceManager.setBarrierInterval(Long.valueOf(props.get("barrier-interval-timeout-limit").toString()));
-                } catch (NumberFormatException ex) {
-                    deviceManager.setBarrierInterval(DEFAULT_BARRIER_TIMEOUT);
-                }
-            }
+    @Override
+    public void close() {
+        initialized = false;
+
+        try {
+            shutdownSwitchConnections().get(10, TimeUnit.SECONDS);
+        } catch (InterruptedException | ExecutionException | TimeoutException e) {
+            LOG.warn("Failed to shut down switch connections in time {}s, error: {}", 10, e);
         }
 
-        if(rpcManager != null && props.containsKey("is-statistics-rpc-enabled")){
-            rpcManager.setStatisticsRpcEnabled(Boolean.valueOf((props.get("is-statistics-rpc-enabled").toString())));
+        gracefulShutdown(contextChainHolder);
+        gracefulShutdown(deviceManager);
+        gracefulShutdown(rpcManager);
+        gracefulShutdown(statisticsManager);
+        gracefulShutdown(threadPool);
+        gracefulShutdown(hashedWheelTimer);
+        unregisterMXBean(MESSAGE_INTELLIGENCE_AGENCY_MX_BEAN_NAME);
+    }
+
+    private static void gracefulShutdown(final AutoCloseable closeable) {
+        if (Objects.isNull(closeable)) {
+            return;
         }
 
-        if (connectionManager != null && props.containsKey("echo-reply-timeout") ){
-            try {
-                connectionManager.setEchoReplyTimeout(Long.valueOf(props.get("echo-reply-timeout").toString()));
-            }catch (NumberFormatException ex){
-                connectionManager.setEchoReplyTimeout(DEFAULT_ECHO_TIMEOUT);
-            }
+        try {
+            closeable.close();
+        } catch (Exception e) {
+            LOG.warn("Failed to shutdown {} gracefully.", closeable);
         }
+    }
 
-        if(statisticsManager != null && props.containsKey("is-statistics-polling-on")){
-            statisticsManager.setIsStatisticsPollingOn(Boolean.valueOf(props.get("is-statistics-polling-on").toString()));
+    private static void gracefulShutdown(final Timer timer) {
+        if (Objects.isNull(timer)) {
+            return;
         }
 
-        if(statisticsManager != null && props.containsKey("basic-timer-delay")){
-            statisticsManager.setBasicTimerDelay(Long.valueOf(props.get("basic-timer-delay").toString()));
+        try {
+            timer.stop();
+        } catch (Exception e) {
+            LOG.warn("Failed to shutdown {} gracefully.", timer);
         }
+    }
 
-        if(statisticsManager != null && props.containsKey("maximum-timer-delay")){
-            statisticsManager.setMaximumTimerDelay(Long.valueOf(props.get("maximum-timer-delay").toString()));
+    private static void gracefulShutdown(final ThreadPoolExecutor threadPoolExecutor) {
+        if (Objects.isNull(threadPoolExecutor)) {
+            return;
+        }
+
+        try {
+            threadPoolExecutor.shutdownNow();
+        } catch (Exception e) {
+            LOG.warn("Failed to shutdown {} gracefully.", threadPoolExecutor);
         }
     }
 
-    private static void registerMXBean(final MessageIntelligenceAgency messageIntelligenceAgency) {
+    private static void registerMXBean(final Object bean, final String beanName) {
         final MBeanServer mbs = ManagementFactory.getPlatformMBeanServer();
+
         try {
-            final String pathToMxBean = String.format("%s:type=%s",
-                    MessageIntelligenceAgencyMXBean.class.getPackage().getName(),
-                    MessageIntelligenceAgencyMXBean.class.getSimpleName());
-            final ObjectName name = new ObjectName(pathToMxBean);
-            mbs.registerMBean(messageIntelligenceAgency, name);
+            mbs.registerMBean(bean, new ObjectName(beanName));
         } catch (MalformedObjectNameException
                 | NotCompliantMBeanException
                 | MBeanRegistrationException
@@ -378,39 +569,15 @@ public class OpenFlowPluginProviderImpl implements OpenFlowPluginProvider, OpenF
         }
     }
 
-    @Override
-    public void setNotificationProviderService(final NotificationService notificationProviderService) {
-        this.notificationProviderService = notificationProviderService;
-    }
-
-    @Override
-    public void setNotificationPublishService(final NotificationPublishService notificationPublishProviderService) {
-        this.notificationPublishService = notificationPublishProviderService;
-    }
-
-    @Override
-    public ExtensionConverterRegistrator getExtensionConverterRegistrator() {
-        return extensionConverterManager;
-    }
-
-    @Override
-    public void setIsStatisticsRpcEnabled(final boolean isStatisticsRpcEnabled) {
-        this.isStatisticsRpcEnabled = isStatisticsRpcEnabled;
-    }
-
-    @Override
-    public void close() throws Exception {
-        //TODO: consider wrapping each manager into try-catch
-        deviceManager.close();
-        rpcManager.close();
-        statisticsManager.close();
-
-        // Manually shutdown all remaining running threads in pool
-        threadPool.shutdown();
-    }
+    private static void unregisterMXBean(final String beanName) {
+        final MBeanServer mbs = ManagementFactory.getPlatformMBeanServer();
 
-    @Override
-    public void setIsUseSingleLayerSerialization(Boolean useSingleLayerSerialization) {
-        this.useSingleLayerSerialization = useSingleLayerSerialization;
+        try {
+            mbs.unregisterMBean(new ObjectName(beanName));
+        } catch (InstanceNotFoundException
+                | MBeanRegistrationException
+                | MalformedObjectNameException e) {
+            LOG.warn("Error unregistering MBean {}", e);
+        }
     }
 }