X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=openflowplugin-impl%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fopenflowplugin%2Fimpl%2Fdevice%2FDeviceContextImpl.java;h=cb01c5cabe2e0dd7ed805ba548e6791c0d68a9c4;hb=refs%2Fchanges%2F19%2F50819%2F10;hp=96acf077a969620cd8ccaeef7d571fdc33f3a83c;hpb=597d6bb29686e4b574966813cb476a5bc9b3ec53;p=openflowplugin.git diff --git a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java index 96acf077a9..cb01c5cabe 100644 --- a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java +++ b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/DeviceContextImpl.java @@ -8,26 +8,34 @@ package org.opendaylight.openflowplugin.impl.device; import com.google.common.annotations.VisibleForTesting; +import com.google.common.base.Function; import com.google.common.base.Preconditions; import com.google.common.base.Verify; +import com.google.common.util.concurrent.AsyncFunction; import com.google.common.util.concurrent.FutureCallback; import com.google.common.util.concurrent.Futures; +import com.google.common.util.concurrent.JdkFutureAdapters; import com.google.common.util.concurrent.ListenableFuture; +import io.netty.util.HashedWheelTimer; import io.netty.util.Timeout; +import io.netty.util.TimerTask; import java.math.BigInteger; import java.util.HashMap; import java.util.List; import java.util.Map; import java.util.concurrent.ExecutionException; +import java.util.concurrent.Future; +import java.util.concurrent.TimeUnit; import javax.annotation.Nonnull; +import javax.annotation.Nullable; 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.ReadOnlyTransaction; import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; -import org.opendaylight.controller.md.sal.common.api.data.TransactionChainClosedException; import org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier; import org.opendaylight.openflowjava.protocol.api.connection.ConnectionAdapter; import org.opendaylight.openflowjava.protocol.api.keys.MessageTypeKey; +import org.opendaylight.openflowplugin.api.OFConstants; import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionContext; import org.opendaylight.openflowplugin.api.openflow.device.DeviceContext; import org.opendaylight.openflowplugin.api.openflow.device.DeviceInfo; @@ -37,6 +45,7 @@ import org.opendaylight.openflowplugin.api.openflow.device.MessageTranslator; import org.opendaylight.openflowplugin.api.openflow.device.RequestContext; import org.opendaylight.openflowplugin.api.openflow.device.TranslatorLibrary; import org.opendaylight.openflowplugin.api.openflow.device.Xid; +import org.opendaylight.openflowplugin.api.openflow.device.handlers.ClusterInitializationPhaseHandler; import org.opendaylight.openflowplugin.api.openflow.device.handlers.MultiMsgCollector; import org.opendaylight.openflowplugin.api.openflow.lifecycle.LifecycleService; import org.opendaylight.openflowplugin.api.openflow.md.core.SwitchConnectionDistinguisher; @@ -62,6 +71,7 @@ import org.opendaylight.openflowplugin.impl.registry.flow.DeviceFlowRegistryImpl import org.opendaylight.openflowplugin.impl.registry.flow.FlowRegistryKeyFactory; import org.opendaylight.openflowplugin.impl.registry.group.DeviceGroupRegistryImpl; import org.opendaylight.openflowplugin.impl.registry.meter.DeviceMeterRegistryImpl; +import org.opendaylight.openflowplugin.impl.rpc.AbstractRequestContext; import org.opendaylight.openflowplugin.impl.util.DeviceInitializationUtils; import org.opendaylight.openflowplugin.openflow.md.core.sal.convertor.ConvertorExecutor; import org.opendaylight.openflowplugin.openflow.md.core.session.SwitchConnectionCookieOFImpl; @@ -92,16 +102,19 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.experimenter import org.opendaylight.yang.gen.v1.urn.opendaylight.packet.service.rev130709.PacketReceived; import org.opendaylight.yang.gen.v1.urn.opendaylight.port.statistics.rev131214.FlowCapableNodeConnectorStatisticsData; import org.opendaylight.yang.gen.v1.urn.opendaylight.port.statistics.rev131214.FlowCapableNodeConnectorStatisticsDataBuilder; +import org.opendaylight.yang.gen.v1.urn.opendaylight.role.service.rev150727.OfpRole; +import org.opendaylight.yang.gen.v1.urn.opendaylight.role.service.rev150727.SalRoleService; +import org.opendaylight.yang.gen.v1.urn.opendaylight.role.service.rev150727.SetRoleInput; +import org.opendaylight.yang.gen.v1.urn.opendaylight.role.service.rev150727.SetRoleInputBuilder; +import org.opendaylight.yang.gen.v1.urn.opendaylight.role.service.rev150727.SetRoleOutput; import org.opendaylight.yangtools.yang.binding.DataObject; import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier; +import org.opendaylight.yangtools.yang.common.RpcResult; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -/** - * - */ -public class DeviceContextImpl implements DeviceContext, ExtensionConverterProviderKeeper{ +public class DeviceContextImpl implements DeviceContext, ExtensionConverterProviderKeeper { private static final Logger LOG = LoggerFactory.getLogger(DeviceContextImpl.class); @@ -112,14 +125,21 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi // TODO: high water mark factor should be parametrized private static final float HIGH_WATERMARK_FACTOR = 0.95f; + // Timeout in seconds after what we will give up on propagating role + private static final int SET_ROLE_TIMEOUT = 10; + + private boolean initialized; + + private SalRoleService salRoleService = null; + private final HashedWheelTimer hashedWheelTimer; private ConnectionContext primaryConnectionContext; private final DeviceState deviceState; private final DataBroker dataBroker; private final Map auxiliaryConnectionContexts; - private final TransactionChainManager transactionChainManager; - private final DeviceFlowRegistry deviceFlowRegistry; - private final DeviceGroupRegistry deviceGroupRegistry; - private final DeviceMeterRegistry deviceMeterRegistry; + private TransactionChainManager transactionChainManager; + private DeviceFlowRegistry deviceFlowRegistry; + private DeviceGroupRegistry deviceGroupRegistry; + private DeviceMeterRegistry deviceMeterRegistry; private final PacketInRateLimiter packetInLimiter; private final MessageSpy messageSpy; private final ItemLifeCycleKeeper flowLifeCycleKeeper; @@ -132,56 +152,62 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi private final ItemLifeCycleRegistry itemLifeCycleSourceRegistry; private ExtensionConverterProvider extensionConverterProvider; private final DeviceManager deviceManager; - + private boolean skipTableFeatures; private boolean switchFeaturesMandatory; private final DeviceInfo deviceInfo; private final ConvertorExecutor convertorExecutor; private volatile CONTEXT_STATE state; + private ClusterInitializationPhaseHandler clusterInitializationPhaseHandler; + private final DeviceManager myManager; + private final boolean useSingleLayerSerialization; - public DeviceContextImpl( + DeviceContextImpl( @Nonnull final ConnectionContext primaryConnectionContext, @Nonnull final DataBroker dataBroker, @Nonnull final MessageSpy messageSpy, @Nonnull final TranslatorLibrary translatorLibrary, @Nonnull final DeviceManager manager, - final ConvertorExecutor convertorExecutor) { - this.primaryConnectionContext = Preconditions.checkNotNull(primaryConnectionContext); + final ConvertorExecutor convertorExecutor, + final boolean skipTableFeatures, + final HashedWheelTimer hashedWheelTimer, + final DeviceManager myManager, + final boolean useSingleLayerSerialization) { + this.primaryConnectionContext = primaryConnectionContext; this.deviceInfo = primaryConnectionContext.getDeviceInfo(); + this.hashedWheelTimer = hashedWheelTimer; + this.myManager = myManager; this.deviceState = new DeviceStateImpl(); - this.dataBroker = Preconditions.checkNotNull(dataBroker); - this.transactionChainManager = new TransactionChainManager(dataBroker, deviceInfo); - auxiliaryConnectionContexts = new HashMap<>(); - deviceFlowRegistry = new DeviceFlowRegistryImpl(dataBroker, deviceInfo.getNodeInstanceIdentifier()); - deviceGroupRegistry = new DeviceGroupRegistryImpl(); - deviceMeterRegistry = new DeviceMeterRegistryImpl(); + this.dataBroker = dataBroker; + this.auxiliaryConnectionContexts = new HashMap<>(); this.messageSpy = Preconditions.checkNotNull(messageSpy); - this.deviceManager = Preconditions.checkNotNull(manager); + this.deviceManager = manager; - packetInLimiter = new PacketInRateLimiter(primaryConnectionContext.getConnectionAdapter(), + this.packetInLimiter = new PacketInRateLimiter(primaryConnectionContext.getConnectionAdapter(), /*initial*/ 1000, /*initial*/2000, this.messageSpy, REJECTED_DRAIN_FACTOR); this.translatorLibrary = translatorLibrary; - portStatusTranslator = translatorLibrary.lookupTranslator( + this.portStatusTranslator = translatorLibrary.lookupTranslator( new TranslatorKey(deviceInfo.getVersion(), PortGrouping.class.getName())); - packetInTranslator = translatorLibrary.lookupTranslator( + this.packetInTranslator = translatorLibrary.lookupTranslator( new TranslatorKey(deviceInfo.getVersion(), PacketIn.class.getName())); - flowRemovedTranslator = translatorLibrary.lookupTranslator( + this.flowRemovedTranslator = translatorLibrary.lookupTranslator( new TranslatorKey(deviceInfo.getVersion(), FlowRemoved.class.getName())); - itemLifeCycleSourceRegistry = new ItemLifeCycleRegistryImpl(); - flowLifeCycleKeeper = new ItemLifeCycleSourceImpl(); - itemLifeCycleSourceRegistry.registerLifeCycleSource(flowLifeCycleKeeper); + this.itemLifeCycleSourceRegistry = new ItemLifeCycleRegistryImpl(); + this.flowLifeCycleKeeper = new ItemLifeCycleSourceImpl(); + this.itemLifeCycleSourceRegistry.registerLifeCycleSource(flowLifeCycleKeeper); this.state = CONTEXT_STATE.INITIALIZATION; this.convertorExecutor = convertorExecutor; + this.skipTableFeatures = skipTableFeatures; + this.useSingleLayerSerialization = useSingleLayerSerialization; + this.initialized = false; } - /** - * This method is called from {@link DeviceManagerImpl} only. So we could say "posthandshake process finish" - * and we are able to set a scheduler for an automatic transaction submitting by time (0,5sec). - */ @Override public void initialSubmitTransaction() { - transactionChainManager.initialSubmitWriteTransaction(); + if (initialized) { + transactionChainManager.initialSubmitWriteTransaction(); + } } @Override @@ -216,24 +242,30 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi public void writeToTransaction(final LogicalDatastoreType store, final InstanceIdentifier path, final T data){ - transactionChainManager.writeToTransaction(store, path, data, false); + if (initialized) { + transactionChainManager.writeToTransaction(store, path, data, false); + } } @Override public void writeToTransactionWithParentsSlow(final LogicalDatastoreType store, final InstanceIdentifier path, final T data){ - transactionChainManager.writeToTransaction(store, path, data, true); + if (initialized) { + transactionChainManager.writeToTransaction(store, path, data, true); + } } @Override - public void addDeleteToTxChain(final LogicalDatastoreType store, final InstanceIdentifier path) throws TransactionChainClosedException { - transactionChainManager.addDeleteOperationTotTxChain(store, path); + public void addDeleteToTxChain(final LogicalDatastoreType store, final InstanceIdentifier path) { + if (initialized) { + transactionChainManager.addDeleteOperationTotTxChain(store, path); + } } @Override public boolean submitTransaction() { - return transactionChainManager.submitWriteTransaction(); + return initialized && transactionChainManager.submitWriteTransaction(); } @Override @@ -242,7 +274,7 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi } @Override - public ConnectionContext getAuxiliaryConnectiobContexts(final BigInteger cookie) { + public ConnectionContext getAuxiliaryConnectionContexts(final BigInteger cookie) { return auxiliaryConnectionContexts.get(new SwitchConnectionCookieOFImpl(cookie.longValue())); } @@ -283,17 +315,17 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi final org.opendaylight.yang.gen.v1.urn.opendaylight.flow.service.rev130819.FlowRemoved flowRemovedNotification = flowRemovedTranslator.translate(flowRemoved, deviceInfo, null); - if(!deviceManager.getIsNotificationFlowRemovedOff()) { + if(deviceManager.isFlowRemovedNotificationOn()) { // Trigger off a notification notificationPublishService.offerNotification(flowRemovedNotification); } else if(LOG.isDebugEnabled()) { - LOG.debug("For nodeId={} isNotificationFlowRemovedOff={}", getDeviceInfo().getLOGValue(), deviceManager.getIsNotificationFlowRemovedOff()); + LOG.debug("For nodeId={} isFlowRemovedNotificationOn={}", getDeviceInfo().getLOGValue(), deviceManager.isFlowRemovedNotificationOn()); } final ItemLifecycleListener itemLifecycleListener = flowLifeCycleKeeper.getItemLifecycleListener(); if (itemLifecycleListener != null) { //2. create registry key - final FlowRegistryKey flowRegKey = FlowRegistryKeyFactory.create(flowRemovedNotification); + final FlowRegistryKey flowRegKey = FlowRegistryKeyFactory.create(getDeviceInfo().getVersion(), flowRemovedNotification); //3. lookup flowId final FlowDescriptor flowDescriptor = deviceFlowRegistry.retrieveIdForFlow(flowRegKey); //4. if flowId present: @@ -330,7 +362,8 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi } submitTransaction(); } catch (final Exception e) { - LOG.warn("Error processing port status message: ", e); + LOG.warn("Error processing port status message for port {} on device {} : {}", portStatus.getPortNo(), + getDeviceInfo().getNodeId().toString(), e); } } @@ -421,15 +454,6 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi return translatorLibrary; } - @Override - public synchronized void close() { - LOG.debug("closing deviceContext: {}, nodeId:{}", - getPrimaryConnectionContext().getConnectionAdapter().getRemoteAddress(), - getDeviceInfo().getLOGValue()); - // NOOP - throw new UnsupportedOperationException("Autocloseble.close will be removed soon"); - } - @Override public void setCurrentBarrierTimeout(final Timeout timeout) { barrierTaskTimeout = timeout; @@ -453,7 +477,7 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi @Override public void onPublished() { Verify.verify(CONTEXT_STATE.INITIALIZATION.equals(getState())); - setState(CONTEXT_STATE.WORKING); + this.state = CONTEXT_STATE.WORKING; primaryConnectionContext.getConnectionAdapter().setPacketInFiltering(false); for (final ConnectionContext switchAuxConnectionContext : auxiliaryConnectionContexts.values()) { switchAuxConnectionContext.getConnectionAdapter().setPacketInFiltering(false); @@ -487,33 +511,41 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi @Override public synchronized void shutdownConnection() { - LOG.debug("Shutdown method for node {}", getDeviceInfo().getLOGValue()); + if (LOG.isDebugEnabled()) { + LOG.debug("Shutdown method for node {}", getDeviceInfo().getLOGValue()); + } if (CONTEXT_STATE.TERMINATION.equals(getState())) { LOG.debug("DeviceCtx for Node {} is in termination process.", getDeviceInfo().getLOGValue()); return; } - setState(CONTEXT_STATE.TERMINATION); if (ConnectionContext.CONNECTION_STATE.RIP.equals(getPrimaryConnectionContext().getConnectionState())) { LOG.debug("ConnectionCtx for Node {} is in RIP state.", getDeviceInfo().getLOGValue()); return; } - /* Terminate Auxiliary Connection */ + + // Terminate Auxiliary Connection for (final ConnectionContext connectionContext : auxiliaryConnectionContexts.values()) { LOG.debug("Closing auxiliary connection {}", connectionContext.getNodeId()); connectionContext.closeConnection(false); } - /* Terminate Primary Connection */ + + // Terminate Primary Connection getPrimaryConnectionContext().closeConnection(true); - /* Close all Group Registry */ - deviceGroupRegistry.close(); - deviceFlowRegistry.close(); - deviceMeterRegistry.close(); + + // Close all datastore registries + if (initialized) { + deviceGroupRegistry.close(); + deviceFlowRegistry.close(); + deviceMeterRegistry.close(); + } } @Override public ListenableFuture shuttingDownDataStoreTransactions() { - return transactionChainManager.shuttingDown(); + return initialized + ? this.transactionChainManager.shuttingDown() + : Futures.immediateFuture(null); } @VisibleForTesting @@ -532,21 +564,51 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi } @Override - public void setState(CONTEXT_STATE state) { - this.state = state; - } - - @Override - public void startupClusterServices() throws ExecutionException, InterruptedException { - LOG.debug("Initializing transaction chain manager for node {}", getDeviceInfo().getLOGValue()); - this.transactionChainManager.activateTransactionManager(); - LOG.debug("Waiting to get node {} information", getDeviceInfo().getLOGValue()); - DeviceInitializationUtils.initializeNodeInformation(this, switchFeaturesMandatory, this.convertorExecutor); - } - - @Override - public ListenableFuture stopClusterServices(boolean deviceDisconnected) { - return this.transactionChainManager.deactivateTransactionManager(); + public ListenableFuture stopClusterServices(boolean connectionInterrupted) { + final ListenableFuture deactivateTxManagerFuture = initialized + ? transactionChainManager.deactivateTransactionManager() + : Futures.immediateFuture(null); + + if (!connectionInterrupted) { + final ListenableFuture makeSlaveFuture = Futures.transform(makeDeviceSlave(), new Function, Void>() { + @Nullable + @Override + public Void apply(@Nullable RpcResult setRoleOutputRpcResult) { + return null; + } + }); + + Futures.addCallback(makeSlaveFuture, new FutureCallback() { + @Override + public void onSuccess(@Nullable Void aVoid) { + if (LOG.isDebugEnabled()) { + LOG.debug("Role SLAVE was successfully propagated on device, node {}", deviceInfo.getLOGValue()); + } + } + + @Override + public void onFailure(final Throwable throwable) { + LOG.warn("Was not able to set role SLAVE to device on node {} ", deviceInfo.getLOGValue()); + LOG.trace("Error occurred on device role setting, probably connection loss: ", throwable); + } + }); + + return Futures.transform(deactivateTxManagerFuture, new AsyncFunction() { + @Override + public ListenableFuture apply(Void aVoid) throws Exception { + // Add fallback to remove device from operational DS if setting slave fails + return Futures.withFallback(makeSlaveFuture, t -> + myManager.removeDeviceFromOperationalDS(deviceInfo)); + } + }); + } else { + return Futures.transform(deactivateTxManagerFuture, new AsyncFunction() { + @Override + public ListenableFuture apply(Void aVoid) throws Exception { + return myManager.removeDeviceFromOperationalDS(deviceInfo); + } + }); + } } @Override @@ -559,16 +621,149 @@ public class DeviceContextImpl implements DeviceContext, ExtensionConverterProvi return this.deviceInfo; } + @Override + public void close() { + if (CONTEXT_STATE.TERMINATION.equals(getState())){ + if (LOG.isDebugEnabled()) { + LOG.debug("DeviceContext for node {} is already in TERMINATION state.", getDeviceInfo().getLOGValue()); + } + } else { + this.state = CONTEXT_STATE.TERMINATION; + } + } + @Override public void putLifecycleServiceIntoTxChainManager(final LifecycleService lifecycleService){ - this.transactionChainManager.setLifecycleService(lifecycleService); + if (initialized) { + this.transactionChainManager.setLifecycleService(lifecycleService); + } } @Override public void replaceConnectionContext(final ConnectionContext connectionContext){ // Act like we are initializing the context - setState(CONTEXT_STATE.INITIALIZATION); + this.state = CONTEXT_STATE.INITIALIZATION; this.primaryConnectionContext = connectionContext; this.onPublished(); } + + @Override + public boolean isUseSingleLayerSerialization() { + return useSingleLayerSerialization; + } + + @Override + public boolean isSkipTableFeatures() { + return this.skipTableFeatures; + } + + @Override + public void setSalRoleService(@Nonnull SalRoleService salRoleService) { + this.salRoleService = salRoleService; + } + + @Override + public void setLifecycleInitializationPhaseHandler(final ClusterInitializationPhaseHandler handler) { + this.clusterInitializationPhaseHandler = handler; + } + + @Override + public boolean onContextInstantiateService(final ConnectionContext connectionContext) { + + if (getPrimaryConnectionContext().getConnectionState().equals(ConnectionContext.CONNECTION_STATE.RIP)) { + LOG.warn("Connection on device {} was interrupted, will stop starting master services.", deviceInfo.getLOGValue()); + return false; + } + + LOG.info("Starting device context cluster services for node {}", deviceInfo.getLOGValue()); + + lazyTransactionManagerInitialization(); + + this.transactionChainManager.activateTransactionManager(); + + try { + DeviceInitializationUtils.initializeNodeInformation(this, switchFeaturesMandatory, this.convertorExecutor); + } catch (ExecutionException | InterruptedException e) { + LOG.warn("Device {} cannot be initialized: ", deviceInfo.getLOGValue(), e); + return false; + } + + Futures.addCallback(sendRoleChangeToDevice(OfpRole.BECOMEMASTER), new RpcResultFutureCallback()); + + return this.clusterInitializationPhaseHandler.onContextInstantiateService(getPrimaryConnectionContext()); + } + + @VisibleForTesting + void lazyTransactionManagerInitialization() { + if (!this.initialized) { + if (LOG.isDebugEnabled()) { + LOG.debug("Transaction chain manager for node {} created", deviceInfo.getLOGValue()); + } + this.transactionChainManager = new TransactionChainManager(dataBroker, deviceInfo); + this.deviceFlowRegistry = new DeviceFlowRegistryImpl(deviceInfo.getVersion(), dataBroker, deviceInfo.getNodeInstanceIdentifier()); + this.deviceGroupRegistry = new DeviceGroupRegistryImpl(); + this.deviceMeterRegistry = new DeviceMeterRegistryImpl(); + this.initialized = true; + } + } + + @Nullable + @Override + public RequestContext createRequestContext() { + return new AbstractRequestContext(deviceInfo.reserveXidForDeviceMessage()) { + @Override + public void close() { + } + }; + + } + + ListenableFuture> sendRoleChangeToDevice(final OfpRole newRole) { + if (LOG.isDebugEnabled()) { + LOG.debug("Sending new role {} to device {}", newRole, deviceInfo.getNodeId()); + } + + final Future> setRoleOutputFuture; + + if (deviceInfo.getVersion() >= OFConstants.OFP_VERSION_1_3) { + final SetRoleInput setRoleInput = (new SetRoleInputBuilder()).setControllerRole(newRole) + .setNode(new NodeRef(deviceInfo.getNodeInstanceIdentifier())).build(); + + setRoleOutputFuture = this.salRoleService.setRole(setRoleInput); + + final TimerTask timerTask = timeout -> { + if (!setRoleOutputFuture.isDone()) { + LOG.warn("New role {} was not propagated to device {} during {} sec", newRole, deviceInfo.getLOGValue(), SET_ROLE_TIMEOUT); + setRoleOutputFuture.cancel(true); + } + }; + + hashedWheelTimer.newTimeout(timerTask, SET_ROLE_TIMEOUT, TimeUnit.SECONDS); + } else { + LOG.info("Device: {} with version: {} does not support role", deviceInfo.getLOGValue(), deviceInfo.getVersion()); + return Futures.immediateFuture(null); + } + + return JdkFutureAdapters.listenInPoolThread(setRoleOutputFuture); + } + + @Override + public ListenableFuture> makeDeviceSlave() { + return sendRoleChangeToDevice(OfpRole.BECOMESLAVE); + } + + private class RpcResultFutureCallback implements FutureCallback> { + @Override + public void onSuccess(@Nullable RpcResult setRoleOutputRpcResult) { + if (LOG.isDebugEnabled()) { + LOG.debug("Role MASTER was successfully set on device, node {}", deviceInfo.getLOGValue()); + } + } + + @Override + public void onFailure(final Throwable throwable) { + LOG.warn("Was not able to set MASTER role on device, node {}", deviceInfo.getLOGValue()); + shutdownConnection(); + } + } }