Bump MRI upstreams
[openflowplugin.git] / openflowplugin-impl / src / main / java / org / opendaylight / openflowplugin / impl / lifecycle / ContextChainImpl.java
index 275d0f25e0be97b7507f75aef6e63f7c4efd4a87..2cc9aea0527234c53984ea71ca81fbe37cbb4e08 100644 (file)
@@ -13,11 +13,11 @@ import com.google.common.util.concurrent.ListenableFuture;
 import java.util.List;
 import java.util.Objects;
 import java.util.concurrent.CopyOnWriteArrayList;
-import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executor;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicReference;
 import java.util.stream.Collectors;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider;
 import org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier;
 import org.opendaylight.openflowplugin.api.openflow.OFPContext;
@@ -29,6 +29,7 @@ import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainMaster
 import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainMastershipWatcher;
 import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainState;
 import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainStateListener;
+import org.opendaylight.openflowplugin.api.openflow.lifecycle.DeviceInitializationContext;
 import org.opendaylight.openflowplugin.api.openflow.lifecycle.GuardedContext;
 import org.opendaylight.openflowplugin.api.openflow.lifecycle.ReconciliationFrameworkStep;
 import org.slf4j.Logger;
@@ -36,16 +37,15 @@ import org.slf4j.LoggerFactory;
 
 public class ContextChainImpl implements ContextChain {
     private static final Logger LOG = LoggerFactory.getLogger(ContextChainImpl.class);
+    private static final Logger OF_EVENT_LOG = LoggerFactory.getLogger("OfEventLog");
 
     private final AtomicBoolean masterStateOnDevice = new AtomicBoolean(false);
-    private final AtomicBoolean initialGathering = new AtomicBoolean(false);
     private final AtomicBoolean initialSubmitting = new AtomicBoolean(false);
-    private final AtomicBoolean registryFilling = new AtomicBoolean(false);
     private final AtomicBoolean rpcRegistration = new AtomicBoolean(false);
     private final List<DeviceRemovedHandler> deviceRemovedHandlers = new CopyOnWriteArrayList<>();
     private final List<GuardedContext> contexts = new CopyOnWriteArrayList<>();
     private final List<ConnectionContext> auxiliaryConnections = new CopyOnWriteArrayList<>();
-    private final ExecutorService executorService;
+    private final Executor executor;
     private final ContextChainMastershipWatcher contextChainMastershipWatcher;
     private final DeviceInfo deviceInfo;
     private final ConnectionContext primaryConnection;
@@ -53,29 +53,30 @@ public class ContextChainImpl implements ContextChain {
             new AtomicReference<>(ContextChainState.UNDEFINED);
     private AutoCloseable registration;
 
-    ContextChainImpl(@Nonnull final ContextChainMastershipWatcher contextChainMastershipWatcher,
-                     @Nonnull final ConnectionContext connectionContext,
-                     @Nonnull final ExecutorService executorService) {
+    ContextChainImpl(@NonNull final ContextChainMastershipWatcher contextChainMastershipWatcher,
+                     @NonNull final ConnectionContext connectionContext,
+                     @NonNull final Executor executor) {
         this.contextChainMastershipWatcher = contextChainMastershipWatcher;
         this.primaryConnection = connectionContext;
         this.deviceInfo = connectionContext.getDeviceInfo();
-        this.executorService = executorService;
+        this.executor = executor;
     }
 
     @Override
-    public <T extends OFPContext> void addContext(@Nonnull final T context) {
+    public <T extends OFPContext> void addContext(@NonNull final T context) {
         contexts.add(new GuardedContextImpl(context));
     }
 
     @Override
     @SuppressWarnings("checkstyle:IllegalCatch")
     public void instantiateServiceInstance() {
+        OF_EVENT_LOG.debug("Clustering Service Invocation, Node: {}", deviceInfo);
         try {
             contexts.forEach(OFPContext::instantiateServiceInstance);
             LOG.info("Started clustering services for node {}", deviceInfo);
         } catch (final Exception ex) {
-            LOG.warn("Not able to start clustering services for node {}", deviceInfo);
-            executorService.execute(() -> contextChainMastershipWatcher
+            LOG.error("Not able to start clustering services for node {}", deviceInfo);
+            executor.execute(() -> contextChainMastershipWatcher
                     .onNotAbleToStartMastershipMandatory(deviceInfo, ex.toString()));
         }
     }
@@ -89,13 +90,14 @@ public class ContextChainImpl implements ContextChain {
             .map(OFPContext::closeServiceInstance)
             .collect(Collectors.toList()));
 
-        return Futures.transform(servicesToBeClosed, (input) -> {
+        return Futures.transform(servicesToBeClosed, input -> {
+            OF_EVENT_LOG.debug("Closing clustering Services, Node: {}", deviceInfo);
             LOG.info("Closed clustering services for node {}", deviceInfo);
             return null;
-        }, executorService);
+        }, executor);
     }
 
-    @Nonnull
+    @NonNull
     @Override
     public ServiceGroupIdentifier getIdentifier() {
         return deviceInfo.getServiceIdentifier();
@@ -122,6 +124,7 @@ public class ContextChainImpl implements ContextChain {
                 registration.close();
                 registration = null;
                 LOG.info("Closed clustering services registration for node {}", deviceInfo);
+                OF_EVENT_LOG.debug("Closed clustering services registration for node {}", deviceInfo);
             } catch (final Exception e) {
                 LOG.warn("Failed to close clustering services registration for node {} with exception: ",
                         deviceInfo, e);
@@ -152,33 +155,28 @@ public class ContextChainImpl implements ContextChain {
         registration = Objects.requireNonNull(clusterSingletonServiceProvider
                 .registerClusterSingletonService(this));
         LOG.debug("Registered clustering services for node {}", deviceInfo);
+        OF_EVENT_LOG.debug("Registered Clustering Services, Node: {}", deviceInfo);
     }
 
     @Override
-    public boolean isMastered(@Nonnull ContextChainMastershipState mastershipState,
-                              boolean inReconciliationFrameworkStep) {
+    public boolean isMastered(@NonNull final ContextChainMastershipState mastershipState,
+                              final boolean inReconciliationFrameworkStep) {
         switch (mastershipState) {
             case INITIAL_SUBMIT:
                 LOG.debug("Device {}, initial submit OK.", deviceInfo);
+                OF_EVENT_LOG.debug("Device {}, initial submit OK.", deviceInfo);
                 this.initialSubmitting.set(true);
                 break;
             case MASTER_ON_DEVICE:
                 LOG.debug("Device {}, master state OK.", deviceInfo);
+                OF_EVENT_LOG.debug("Device {}, master state OK.", deviceInfo);
                 this.masterStateOnDevice.set(true);
                 break;
-            case INITIAL_GATHERING:
-                LOG.debug("Device {}, initial gathering OK.", deviceInfo);
-                this.initialGathering.set(true);
-                break;
             case RPC_REGISTRATION:
                 LOG.debug("Device {}, RPC registration OK.", deviceInfo);
+                OF_EVENT_LOG.debug("Device {}, RPC registration OK.", deviceInfo);
                 this.rpcRegistration.set(true);
                 break;
-            case INITIAL_FLOW_REGISTRY_FILL:
-                // Flow registry fill is not mandatory to work as a master
-                LOG.debug("Device {}, initial registry filling OK.", deviceInfo);
-                this.registryFilling.set(true);
-                break;
             case CHECK:
                 // no operation
                 break;
@@ -187,12 +185,11 @@ public class ContextChainImpl implements ContextChain {
                 break;
         }
 
-        final boolean result = initialGathering.get() && masterStateOnDevice.get() && rpcRegistration.get()
+        final boolean result = masterStateOnDevice.get() && rpcRegistration.get()
                 && inReconciliationFrameworkStep || initialSubmitting.get();
 
         if (!inReconciliationFrameworkStep && result && mastershipState != ContextChainMastershipState.CHECK) {
-            LOG.info("Device {} is able to work as master{}", deviceInfo,
-                     registryFilling.get() ? "." : " WITHOUT flow registry !!!");
+            LOG.info("Device {} is able to work as master", deviceInfo);
             changeMastershipState(ContextChainState.WORKING_MASTER);
         }
 
@@ -214,19 +211,28 @@ public class ContextChainImpl implements ContextChain {
     }
 
     @Override
-    public boolean addAuxiliaryConnection(@Nonnull ConnectionContext connectionContext) {
-        return connectionContext.getFeatures().getAuxiliaryId() != 0
+    public void initializeDevice() {
+        contexts.forEach(context -> {
+            if (context.map(DeviceInitializationContext.class::isInstance)) {
+                context.map(DeviceInitializationContext.class::cast).initializeDevice();
+            }
+        });
+    }
+
+    @Override
+    public boolean addAuxiliaryConnection(@NonNull final ConnectionContext connectionContext) {
+        return connectionContext.getFeatures().getAuxiliaryId().toJava() != 0
                 && !ConnectionContext.CONNECTION_STATE.RIP.equals(primaryConnection.getConnectionState())
                 && auxiliaryConnections.add(connectionContext);
     }
 
     @Override
-    public boolean auxiliaryConnectionDropped(@Nonnull ConnectionContext connectionContext) {
+    public boolean auxiliaryConnectionDropped(@NonNull final ConnectionContext connectionContext) {
         return auxiliaryConnections.remove(connectionContext);
     }
 
     @Override
-    public void registerDeviceRemovedHandler(@Nonnull final DeviceRemovedHandler deviceRemovedHandler) {
+    public void registerDeviceRemovedHandler(@NonNull final DeviceRemovedHandler deviceRemovedHandler) {
         deviceRemovedHandlers.add(deviceRemovedHandler);
     }
 
@@ -248,9 +254,7 @@ public class ContextChainImpl implements ContextChain {
     }
 
     private void unMasterMe() {
-        registryFilling.set(false);
         initialSubmitting.set(false);
-        initialGathering.set(false);
         masterStateOnDevice.set(false);
         rpcRegistration.set(false);
     }