package org.opendaylight.openflowplugin.impl.lifecycle;
import com.google.common.annotations.VisibleForTesting;
-import com.google.common.base.Verify;
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 java.util.Collections;
+import com.google.common.util.concurrent.MoreExecutors;
+import com.google.common.util.concurrent.ThreadFactoryBuilder;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import java.util.HashMap;
+import java.util.List;
import java.util.Map;
import java.util.Objects;
import java.util.Optional;
+import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.ExecutionException;
import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.ScheduledExecutorService;
+import java.util.concurrent.ThreadFactory;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.TimeoutException;
-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipChange;
-import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipListenerRegistration;
-import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipService;
-import org.opendaylight.controller.md.sal.common.api.data.TransactionCommitFailedException;
+import java.util.stream.Collectors;
+import org.eclipse.jdt.annotation.NonNull;
+import org.opendaylight.mdsal.eos.binding.api.EntityOwnershipChange;
+import org.opendaylight.mdsal.eos.binding.api.EntityOwnershipListenerRegistration;
+import org.opendaylight.mdsal.eos.binding.api.EntityOwnershipService;
+import org.opendaylight.mdsal.eos.common.api.EntityOwnershipState;
import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider;
import org.opendaylight.openflowplugin.api.openflow.OFPManager;
import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionContext;
import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChain;
import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainHolder;
import org.opendaylight.openflowplugin.api.openflow.lifecycle.ContextChainMastershipState;
-import org.opendaylight.openflowplugin.api.openflow.lifecycle.LifecycleService;
+import org.opendaylight.openflowplugin.api.openflow.lifecycle.MasterChecker;
+import org.opendaylight.openflowplugin.api.openflow.lifecycle.OwnershipChangeListener;
+import org.opendaylight.openflowplugin.api.openflow.role.RoleContext;
+import org.opendaylight.openflowplugin.api.openflow.role.RoleManager;
import org.opendaylight.openflowplugin.api.openflow.rpc.RpcContext;
import org.opendaylight.openflowplugin.api.openflow.rpc.RpcManager;
import org.opendaylight.openflowplugin.api.openflow.statistics.StatisticsContext;
import org.opendaylight.openflowplugin.api.openflow.statistics.StatisticsManager;
import org.opendaylight.openflowplugin.impl.util.DeviceStateUtil;
-import org.opendaylight.openflowplugin.impl.util.ItemScheduler;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeId;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.NodeKey;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.FeaturesReply;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.mdsal.core.general.entity.rev150930.Entity;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflow.provider.config.rev160510.OpenflowProviderConfig;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflowplugin.rf.state.rev170713.ResultState;
import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
+import org.opendaylight.yangtools.yang.common.Uint8;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-public class ContextChainHolderImpl implements ContextChainHolder {
+public class ContextChainHolderImpl implements ContextChainHolder, MasterChecker {
private static final Logger LOG = LoggerFactory.getLogger(ContextChainHolderImpl.class);
+ private static final Logger OF_EVENT_LOG = LoggerFactory.getLogger("OfEventLog");
private static final String CONTEXT_CREATED_FOR_CONNECTION = " context created for connection: {}";
- private static final long CHECK_ROLE_MASTER_TIMEOUT = 20000L;
- private static final long CHECK_ROLE_MASTER_TOLERANCE = CHECK_ROLE_MASTER_TIMEOUT / 2;
private static final long REMOVE_DEVICE_FROM_DS_TIMEOUT = 5000L;
private static final String ASYNC_SERVICE_ENTITY_TYPE = "org.opendaylight.mdsal.AsyncServiceCloseEntityType";
-
- private final Map<DeviceInfo, ContextChain> contextChainMap = Collections.synchronizedMap(new HashMap<>());
+ private static final String SERVICE_ENTITY_TYPE = "org.opendaylight.mdsal.ServiceEntityType";
+ private static final String SEPARATOR = ":";
+ private final Map<DeviceInfo, ContextChain> contextChainMap = new ConcurrentHashMap<>();
+ private final Map<DeviceInfo, ? super ConnectionContext> connectingDevices = new ConcurrentHashMap<>();
+ private final EntityOwnershipListenerRegistration eosListenerRegistration;
+ private final ClusterSingletonServiceProvider singletonServiceProvider;
+ private final ExecutorService executorService;
+ private final OwnershipChangeListener ownershipChangeListener;
+ private final ThreadFactory threadFactory = new ThreadFactoryBuilder().setDaemon(true)
+ .setNameFormat("node-cleaner-%d").setUncaughtExceptionHandler((thread, throwable) -> {
+ LOG.warn("Uncaught exception while removing node data from operational datastore.", throwable);
+ }).build();
+ private final ScheduledExecutorService nodeCleanerExecutor = Executors.newScheduledThreadPool(
+ Runtime.getRuntime().availableProcessors() , threadFactory);
+ private final EntityOwnershipService entityOwnershipService;
+ private final OpenflowProviderConfig config;
private DeviceManager deviceManager;
private RpcManager rpcManager;
private StatisticsManager statisticsManager;
- private EntityOwnershipListenerRegistration eosListenerRegistration;
- private ClusterSingletonServiceProvider singletonServicesProvider;
- private final ItemScheduler<DeviceInfo, ContextChain> scheduler;
- private final ExecutorService executorService;
-
- public ContextChainHolderImpl(final HashedWheelTimer timer, final ExecutorService executorService) {
- this.scheduler = new ItemScheduler<>(
- timer,
- CHECK_ROLE_MASTER_TIMEOUT,
- CHECK_ROLE_MASTER_TOLERANCE,
- ContextChain::makeDeviceSlave);
-
+ private RoleManager roleManager;
+
+ public ContextChainHolderImpl(final ExecutorService executorService,
+ final ClusterSingletonServiceProvider singletonServiceProvider,
+ final EntityOwnershipService entityOwnershipService,
+ final OwnershipChangeListener ownershipChangeListener,
+ final OpenflowProviderConfig config) {
+ this.singletonServiceProvider = singletonServiceProvider;
this.executorService = executorService;
+ this.ownershipChangeListener = ownershipChangeListener;
+ this.ownershipChangeListener.setMasterChecker(this);
+ this.entityOwnershipService = entityOwnershipService;
+ this.config = config;
+ this.eosListenerRegistration = Objects
+ .requireNonNull(entityOwnershipService.registerListener(ASYNC_SERVICE_ENTITY_TYPE, this));
}
@Override
public <T extends OFPManager> void addManager(final T manager) {
- if (Objects.isNull(deviceManager) && manager instanceof DeviceManager) {
+ if (deviceManager == null && manager instanceof DeviceManager) {
LOG.trace("Context chain holder: Device manager OK.");
deviceManager = (DeviceManager) manager;
- } else if (Objects.isNull(rpcManager) && manager instanceof RpcManager) {
+ } else if (rpcManager == null && manager instanceof RpcManager) {
LOG.trace("Context chain holder: RPC manager OK.");
rpcManager = (RpcManager) manager;
- } else if (Objects.isNull(statisticsManager) && manager instanceof StatisticsManager) {
+ } else if (statisticsManager == null && manager instanceof StatisticsManager) {
LOG.trace("Context chain holder: Statistics manager OK.");
statisticsManager = (StatisticsManager) manager;
+ } else if (roleManager == null && manager instanceof RoleManager) {
+ LOG.trace("Context chain holder: Role manager OK.");
+ roleManager = (RoleManager) manager;
}
}
- @Override
- public ContextChain createContextChain(final ConnectionContext connectionContext) {
+ @VisibleForTesting
+ void createContextChain(final ConnectionContext connectionContext) {
final DeviceInfo deviceInfo = connectionContext.getDeviceInfo();
- final String deviceInfoLOGValue = deviceInfo.getLOGValue();
- final ContextChain contextChain = new ContextChainImpl(connectionContext);
-
- if (LOG.isDebugEnabled()) {
- LOG.debug("Context chain" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfoLOGValue);
- }
-
- final LifecycleService lifecycleService = new LifecycleServiceImpl(this, executorService);
- lifecycleService.registerDeviceRemovedHandler(deviceManager);
- lifecycleService.registerDeviceRemovedHandler(rpcManager);
- lifecycleService.registerDeviceRemovedHandler(statisticsManager);
-
- if (LOG.isDebugEnabled()) {
- LOG.debug("Lifecycle services" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfoLOGValue);
- }
final DeviceContext deviceContext = deviceManager.createContext(connectionContext);
-
- if (LOG.isDebugEnabled()) {
- LOG.debug("Device" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfoLOGValue);
- }
-
- final RpcContext rpcContext = rpcManager.createContext(connectionContext.getDeviceInfo(), deviceContext);
-
- if (LOG.isDebugEnabled()) {
- LOG.debug("RPC" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfoLOGValue);
- }
-
- final StatisticsContext statisticsContext = statisticsManager.createContext(deviceContext);
-
- if (LOG.isDebugEnabled()) {
- LOG.debug("Statistics" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfoLOGValue);
- }
-
- deviceContext.setLifecycleInitializationPhaseHandler(statisticsContext);
- statisticsContext.setLifecycleInitializationPhaseHandler(rpcContext);
- statisticsContext.setInitialSubmitHandler(deviceContext);
-
- contextChain.addLifecycleService(lifecycleService);
+ deviceContext.registerMastershipWatcher(this);
+ LOG.debug("Device" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfo);
+
+ final RpcContext rpcContext = rpcManager.createContext(deviceContext);
+ rpcContext.registerMastershipWatcher(this);
+ LOG.debug("RPC" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfo);
+
+ final StatisticsContext statisticsContext = statisticsManager
+ .createContext(deviceContext, ownershipChangeListener.isReconciliationFrameworkRegistered());
+ statisticsContext.registerMastershipWatcher(this);
+ LOG.debug("Statistics" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfo);
+
+ final RoleContext roleContext = roleManager.createContext(deviceContext);
+ roleContext.registerMastershipWatcher(this);
+ LOG.debug("Role" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfo);
+
+ final ContextChain contextChain = new ContextChainImpl(this, connectionContext, executorService);
+ contextChain.registerDeviceRemovedHandler(deviceManager);
+ contextChain.registerDeviceRemovedHandler(rpcManager);
+ contextChain.registerDeviceRemovedHandler(statisticsManager);
+ contextChain.registerDeviceRemovedHandler(roleManager);
+ contextChain.registerDeviceRemovedHandler(this);
contextChain.addContext(deviceContext);
contextChain.addContext(rpcContext);
contextChain.addContext(statisticsContext);
+ contextChain.addContext(roleContext);
+ contextChainMap.put(deviceInfo, contextChain);
+ connectingDevices.remove(deviceInfo);
+ LOG.debug("Context chain" + CONTEXT_CREATED_FOR_CONNECTION, deviceInfo);
- LOG.info("Starting timer for setting SLAVE role on node {} if no role will be set in {}s.",
- deviceInfo.getLOGValue(), CHECK_ROLE_MASTER_TIMEOUT / 1000L);
- scheduler.add(deviceInfo, contextChain);
- scheduler.startIfNotRunning();
deviceContext.onPublished();
- contextChain.registerServices(this.singletonServicesProvider);
- return contextChain;
- }
-
- @Override
- public synchronized void destroyContextChain(final DeviceInfo deviceInfo) {
- Optional.ofNullable(contextChainMap.remove(deviceInfo)).ifPresent(contextChain -> {
- deviceManager.sendNodeRemovedNotification(deviceInfo.getNodeInstanceIdentifier());
- contextChain.close();
- });
+ contextChain.registerServices(singletonServiceProvider);
}
@Override
- public ConnectionStatus deviceConnected(final ConnectionContext connectionContext) throws Exception {
+ public ConnectionStatus deviceConnected(final ConnectionContext connectionContext) {
final DeviceInfo deviceInfo = connectionContext.getDeviceInfo();
final ContextChain contextChain = contextChainMap.get(deviceInfo);
- LOG.info("Device {} connected.", deviceInfo.getLOGValue());
-
- if (contextChain != null) {
- if (contextChain.addAuxiliaryConnection(connectionContext)) {
- LOG.info("An auxiliary connection was added to device: {}", deviceInfo.getLOGValue());
- return ConnectionStatus.MAY_CONTINUE;
+ final FeaturesReply featuresReply = connectionContext.getFeatures();
+ final Uint8 auxiliaryId = featuresReply != null ? featuresReply.getAuxiliaryId() : null;
+
+ if (auxiliaryId != null && auxiliaryId.toJava() != 0) {
+ if (contextChain == null) {
+ LOG.warn("An auxiliary connection for device {}, but no primary connection. Refusing connection.",
+ deviceInfo);
+ return ConnectionStatus.REFUSING_AUXILIARY_CONNECTION;
} else {
- LOG.warn("Device {} already connected. Closing all connection to the device.", deviceInfo.getLOGValue());
- destroyContextChain(deviceInfo);
- return ConnectionStatus.ALREADY_CONNECTED;
+ if (contextChain.addAuxiliaryConnection(connectionContext)) {
+ LOG.info("An auxiliary connection was added to device: {}", deviceInfo);
+ return ConnectionStatus.MAY_CONTINUE;
+ } else {
+ LOG.warn("Not able to add auxiliary connection to the device {}", deviceInfo);
+ return ConnectionStatus.REFUSING_AUXILIARY_CONNECTION;
+ }
}
} else {
- if (LOG.isDebugEnabled()) {
- LOG.debug("No context chain found for device: {}, creating new.", deviceInfo.getLOGValue());
+ LOG.info("Device {} connected.", deviceInfo);
+ final boolean contextExists = contextChain != null;
+ final boolean isClosing = contextExists && contextChain.isClosing();
+
+ if (!isClosing && connectingDevices.putIfAbsent(deviceInfo, connectionContext) != null) {
+ LOG.warn("Device {} is already trying to connect, wait until succeeded or disconnected.", deviceInfo);
+ return ConnectionStatus.ALREADY_CONNECTED;
}
- contextChainMap.put(deviceInfo, createContextChain(connectionContext));
- }
- return ConnectionStatus.MAY_CONTINUE;
- }
+ if (contextExists) {
+ if (isClosing) {
+ LOG.warn("Device {} is already in termination state, closing all incoming connections.",
+ deviceInfo);
+ return ConnectionStatus.CLOSING;
+ }
+
+ LOG.warn("Device {} already connected. Closing previous connection", deviceInfo);
+ destroyContextChain(deviceInfo);
+ LOG.info("Old connection dropped, creating new context chain for device {}", deviceInfo);
+ createContextChain(connectionContext);
+ } else {
+ LOG.info("No context chain found for device: {}, creating new.", deviceInfo);
+ createContextChain(connectionContext);
+ }
+
+ return ConnectionStatus.MAY_CONTINUE;
+ }
- @Override
- public void addSingletonServicesProvider(final ClusterSingletonServiceProvider singletonServicesProvider) {
- this.singletonServicesProvider = singletonServicesProvider;
}
@Override
- public void onNotAbleToStartMastership(final DeviceInfo deviceInfo, @Nonnull final String reason, final boolean mandatory) {
- LOG.warn("Not able to set MASTER role on device {}, reason: {}", deviceInfo.getLOGValue(), reason);
+ public void onNotAbleToStartMastership(final DeviceInfo deviceInfo, final String reason, final boolean mandatory) {
+ LOG.warn("Not able to set MASTER role on device {}, reason: {}", deviceInfo, reason);
if (!mandatory) {
return;
}
Optional.ofNullable(contextChainMap.get(deviceInfo)).ifPresent(contextChain -> {
- LOG.warn("This mastering is mandatory, destroying context chain and closing connection for device {}.", deviceInfo.getLOGValue());
- addDestroyChainCallback(contextChain.stopChain(), deviceInfo);
+ LOG.warn("This mastering is mandatory, destroying context chain and closing connection for device {}.",
+ deviceInfo);
+ destroyContextChain(deviceInfo);
});
}
@Override
- public void onMasterRoleAcquired(final DeviceInfo deviceInfo, @Nonnull final ContextChainMastershipState mastershipState) {
- scheduler.remove(deviceInfo);
-
+ public void onMasterRoleAcquired(final DeviceInfo deviceInfo, final ContextChainMastershipState mastershipState) {
Optional.ofNullable(contextChainMap.get(deviceInfo)).ifPresent(contextChain -> {
- if (contextChain.isMastered(mastershipState)) {
- LOG.info("Role MASTER was granted to device {}", deviceInfo.getLOGValue());
+ if (!ContextChainMastershipState.INITIAL_SUBMIT.equals(mastershipState)) {
+ if (contextChain.isMastered(mastershipState, true)) {
+ Futures.addCallback(ownershipChangeListener.becomeMasterBeforeSubmittedDS(deviceInfo),
+ reconciliationFrameworkCallback(deviceInfo, contextChain, mastershipState),
+ MoreExecutors.directExecutor());
+ }
+ } else if (contextChain.isMastered(mastershipState, false)) {
+ LOG.info("Role MASTER was granted to device {}", deviceInfo);
deviceManager.sendNodeAddedNotification(deviceInfo.getNodeInstanceIdentifier());
}
});
@Override
public void onSlaveRoleAcquired(final DeviceInfo deviceInfo) {
+ ownershipChangeListener.becomeSlaveOrDisconnect(deviceInfo);
+ LOG.info("Role SLAVE was granted to device {}", deviceInfo);
Optional.ofNullable(contextChainMap.get(deviceInfo)).ifPresent(ContextChain::makeContextChainStateSlave);
}
@Override
- public void onSlaveRoleNotAcquired(final DeviceInfo deviceInfo) {
+ public void onSlaveRoleNotAcquired(final DeviceInfo deviceInfo, final String reason) {
+ LOG.warn("Not able to set SLAVE role on device {}, reason: {}", deviceInfo, reason);
Optional.ofNullable(contextChainMap.get(deviceInfo)).ifPresent(contextChain -> destroyContextChain(deviceInfo));
}
public void onDeviceDisconnected(final ConnectionContext connectionContext) {
final DeviceInfo deviceInfo = connectionContext.getDeviceInfo();
- if (Objects.isNull(deviceInfo)) {
- return;
- }
-
- Optional.ofNullable(contextChainMap.get(deviceInfo)).ifPresent(contextChain -> {
+ Optional.ofNullable(connectionContext.getDeviceInfo()).map(contextChainMap::get).ifPresent(contextChain -> {
if (contextChain.auxiliaryConnectionDropped(connectionContext)) {
- LOG.info("Auxiliary connection from device {} disconnected.", deviceInfo.getLOGValue());
+ LOG.info("Auxiliary connection from device {} disconnected.", deviceInfo);
} else {
- LOG.info("Device {} disconnected.", deviceInfo.getLOGValue());
- addDestroyChainCallback(contextChain.connectionDropped(), deviceInfo);
+ LOG.info("Device {} disconnected.", deviceInfo);
+ destroyContextChain(deviceInfo);
}
});
}
- @Override
- public void changeEntityOwnershipService(@Nonnull final EntityOwnershipService entityOwnershipService) {
- if (Objects.nonNull(this.eosListenerRegistration)) {
- LOG.warn("Entity ownership service listener is already registered.");
- } else {
- this.eosListenerRegistration = Verify.verifyNotNull(entityOwnershipService.registerListener
- (ASYNC_SERVICE_ENTITY_TYPE, this));
- }
- }
-
@VisibleForTesting
boolean checkAllManagers() {
- return Objects.nonNull(deviceManager) && Objects.nonNull(rpcManager) && Objects.nonNull(statisticsManager);
+ return deviceManager != null && rpcManager != null && statisticsManager != null && roleManager != null;
}
@Override
- public void close() throws Exception {
- scheduler.close();
-
- contextChainMap.forEach((deviceInfo, contextChain) -> {
- if (contextChain.isMastered(ContextChainMastershipState.CHECK)) {
- addDestroyChainCallback(contextChain.stopChain(), deviceInfo);
- } else {
- destroyContextChain(deviceInfo);
- }
- });
+ public ContextChain getContextChain(final DeviceInfo deviceInfo) {
+ return contextChainMap.get(deviceInfo);
+ }
+ @Override
+ public void close() {
+ Map<DeviceInfo, ContextChain> copyOfChains = new HashMap<>(contextChainMap);
+ copyOfChains.keySet().forEach(this::destroyContextChain);
+ copyOfChains.clear();
contextChainMap.clear();
-
-
- if (Objects.nonNull(eosListenerRegistration)) {
- eosListenerRegistration.close();
- eosListenerRegistration = null;
- }
+ eosListenerRegistration.close();
+ OF_EVENT_LOG.debug("EOS registration closed for all devices");
+ nodeCleanerExecutor.shutdownNow();
}
@Override
- public void ownershipChanged(EntityOwnershipChange entityOwnershipChange) {
- if (entityOwnershipChange.hasOwner()) {
+ @SuppressFBWarnings("BC_UNCONFIRMED_CAST_OF_RETURN_VALUE")
+ public void ownershipChanged(final EntityOwnershipChange entityOwnershipChange) {
+ LOG.info("Entity ownership change received for node : {}", entityOwnershipChange);
+ if (entityOwnershipChange.inJeopardy()) {
+ LOG.warn("Controller is in Jeopardy, ignore ownership change notification. {}", entityOwnershipChange);
+ return;
+ }
+ if (entityOwnershipChange.getState().hasOwner()) {
return;
}
- final String entityName = getEntityNameFromOwnershipChange(entityOwnershipChange);
-
- if (Objects.nonNull(entityName)) {
- if (LOG.isDebugEnabled()) {
- LOG.debug("Entity {} has no owner", entityName);
+ // Findbugs flags a false violation for "Unchecked/unconfirmed cast" from GenericEntity to Entity hence the
+ // suppression above. The suppression is temporary until EntityOwnershipChange is modified to eliminate the
+ // violation.
+ final String entityName = entityOwnershipChange
+ .getEntity()
+ .getIdentifier()
+ .firstKeyOf(Entity.class)
+ .getName();
+
+ if (entityName != null && entityName.startsWith("openflow:")) {
+ if (nodeCleanerExecutor.isShutdown()) {
+ LOG.warn("Node cleaner executor thread-pool is down.");
+ return;
}
+ LOG.debug("Device {} will be removed from datastore in {} msec, if it's not transient notification.",
+ entityName, config.getDeviceDatastoreRemovalDelay().getValue());
+ final String dpnId = getDpnIdFromNodeName(entityName);
+ nodeCleanerExecutor.schedule(() -> {
+ try {
+ Optional<EntityOwnershipState> ownershipState = getCurrentOwnershipStatus(entityName);
+ if (!ownershipState.isPresent()
+ || Objects.equals(ownershipState.get(), EntityOwnershipState.NO_OWNER)) {
+ LOG.debug("Entity {} has no owner", entityName);
+ final KeyedInstanceIdentifier<Node, NodeKey> nodeInstanceIdentifier =
+ DeviceStateUtil.createNodeInstanceIdentifier(new NodeId(entityName));
+ deviceManager.sendNodeRemovedNotification(nodeInstanceIdentifier);
+ LOG.info("Try to remove device {} from operational DS", entityName);
+ ListenableFuture<?> future =
+ deviceManager.removeDeviceFromOperationalDS(nodeInstanceIdentifier);
+ Futures.addCallback(future, new FutureCallback<Object>() {
+ @Override
+ public void onSuccess(final Object result) {
+ LOG.debug("Node removed from Oper DS, Node: {}", dpnId);
+ OF_EVENT_LOG.debug("Node removed from Oper DS, Node: {}", dpnId);
+ }
+
+ @Override
+ public void onFailure(final Throwable throwable) {
+ LOG.error("Could not remove device {} from operational DS", dpnId, throwable);
+ }
+ }, MoreExecutors.directExecutor());
+ future.get(REMOVE_DEVICE_FROM_DS_TIMEOUT, TimeUnit.MILLISECONDS);
+ } else {
+ LOG.warn("Seems like device is still owned by other controller instance. Skip deleting {} "
+ + "node from operational datastore.", entityName);
+ }
+ } catch (TimeoutException | ExecutionException | NullPointerException | InterruptedException e) {
+ LOG.warn("Not able to remove device {} from operational DS. ", entityName, e);
+ }
+ }, config.getDeviceDatastoreRemovalDelay().getValue().toJava(), TimeUnit.MILLISECONDS);
+ }
+ }
- final NodeId nodeId = new NodeId(entityName);
-
- try {
- final KeyedInstanceIdentifier<Node, NodeKey> nodeInstanceIdentifier =
- DeviceStateUtil.createNodeInstanceIdentifier(nodeId);
-
- deviceManager.sendNodeRemovedNotification(nodeInstanceIdentifier);
+ private void destroyContextChain(final DeviceInfo deviceInfo) {
+ OF_EVENT_LOG.debug("Destroying context chain for device {}", deviceInfo.getDatapathId());
+ ownershipChangeListener.becomeSlaveOrDisconnect(deviceInfo);
+ Optional.ofNullable(contextChainMap.get(deviceInfo)).ifPresent(contextChain -> {
+ deviceManager.sendNodeRemovedNotification(deviceInfo.getNodeInstanceIdentifier());
+ contextChain.close();
+ connectingDevices.remove(deviceInfo);
+ });
+ }
- LOG.info("Removing device {} from operational DS", nodeId);
- deviceManager
- .removeDeviceFromOperationalDS(nodeInstanceIdentifier)
- .checkedGet(REMOVE_DEVICE_FROM_DS_TIMEOUT, TimeUnit.MILLISECONDS);
- } catch (TimeoutException | TransactionCommitFailedException e) {
- LOG.info("Not able to remove device {} from operational DS. Probably removed by another cluster node.",
- nodeId);
- }
- }
+ @Override
+ public List<DeviceInfo> listOfMasteredDevices() {
+ return contextChainMap.entrySet().stream()
+ .filter(deviceInfoContextChainEntry -> deviceInfoContextChainEntry.getValue()
+ .isMastered(ContextChainMastershipState.CHECK, false)).map(Map.Entry::getKey)
+ .collect(Collectors.toList());
}
- private String getEntityNameFromOwnershipChange(final EntityOwnershipChange entityOwnershipChange) {
- final YangInstanceIdentifier.NodeIdentifierWithPredicates lastIdArgument =
- (YangInstanceIdentifier.NodeIdentifierWithPredicates) entityOwnershipChange
- .getEntity()
- .getId()
- .getLastPathArgument();
-
- return lastIdArgument
- .getKeyValues()
- .values()
- .iterator()
- .next()
- .toString();
+ @Override
+ public boolean isAnyDeviceMastered() {
+ return contextChainMap.entrySet().stream().findAny()
+ .filter(deviceInfoContextChainEntry -> deviceInfoContextChainEntry.getValue()
+ .isMastered(ContextChainMastershipState.CHECK, false)).isPresent();
}
- private void addDestroyChainCallback(final ListenableFuture<Void> future, final DeviceInfo deviceInfo) {
- scheduler.remove(deviceInfo);
+ @Override
+ public void onDeviceRemoved(final DeviceInfo deviceInfo) {
+ contextChainMap.remove(deviceInfo);
+ LOG.debug("Context chain removed for node {}", deviceInfo);
+ }
- Futures.addCallback(future, new FutureCallback<Void>() {
+ private FutureCallback<ResultState> reconciliationFrameworkCallback(@NonNull final DeviceInfo deviceInfo,
+ final ContextChain contextChain, final ContextChainMastershipState mastershipState) {
+ return new FutureCallback<>() {
@Override
- public void onSuccess(@Nullable final Void aVoid) {
- destroyContextChain(deviceInfo);
+ public void onSuccess(final ResultState result) {
+ if (ResultState.DONOTHING == result) {
+ OF_EVENT_LOG.debug("Device {} connection is enabled by reconciliation framework", deviceInfo);
+ LOG.info("Device {} connection is enabled by reconciliation framework.", deviceInfo);
+ if (mastershipState == ContextChainMastershipState.MASTER_ON_DEVICE) {
+ ownershipChangeListener.becomeMaster(deviceInfo);
+ contextChain.initializeDevice();
+ }
+ contextChain.continueInitializationAfterReconciliation();
+ } else {
+ OF_EVENT_LOG.debug("Reconciliation framework failure for device {}", deviceInfo);
+ LOG.warn("Reconciliation framework failure for device {} with result {}", deviceInfo, result);
+ destroyContextChain(deviceInfo);
+ }
}
@Override
- public void onFailure(@Nonnull final Throwable throwable) {
+ public void onFailure(final Throwable throwable) {
+ OF_EVENT_LOG.debug("Reconciliation framework failure for device {} with error {}", deviceInfo,
+ throwable.getMessage());
+ LOG.warn("Reconciliation framework failure.", throwable);
destroyContextChain(deviceInfo);
}
- });
+ };
+ }
+
+ private String getDpnIdFromNodeName(final String nodeName) {
+ return nodeName.substring(nodeName.lastIndexOf(SEPARATOR) + 1);
+ }
+
+ private Optional<EntityOwnershipState> getCurrentOwnershipStatus(final String nodeId) {
+ org.opendaylight.mdsal.eos.binding.api.Entity entity = createNodeEntity(nodeId);
+ Optional<EntityOwnershipState> ownershipStatus
+ = entityOwnershipService.getOwnershipState(entity);
+
+ if (ownershipStatus.isPresent()) {
+ LOG.debug("Current ownership status for node {} is {}", nodeId, ownershipStatus.get());
+ return Optional.of(ownershipStatus.get());
+ } else {
+ LOG.trace("Ownership status is not available for node {}", nodeId);
+ }
+ return Optional.empty();
+ }
+
+ private org.opendaylight.mdsal.eos.binding.api.Entity createNodeEntity(final String nodeId) {
+ return new org.opendaylight.mdsal.eos.binding.api.Entity(ASYNC_SERVICE_ENTITY_TYPE, nodeId);
}
-}
\ No newline at end of file
+}
Code Review / openflowplugin.git / blobdiff