import com.google.common.util.concurrent.ListenableFuture;
import javax.annotation.Nullable;
import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionContext;
+import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionStatus;
import org.opendaylight.openflowplugin.api.openflow.connection.HandshakeContext;
import org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceConnectedHandler;
import org.opendaylight.openflowplugin.api.openflow.md.core.HandshakeListener;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-/**
- *
- */
public class HandshakeListenerImpl implements HandshakeListener {
private static final Logger LOG = LoggerFactory.getLogger(HandshakeListenerImpl.class);
private HandshakeContext handshakeContext;
/**
- * @param connectionContext
- * @param deviceConnectedHandler
+ * Constructor.
+ *
+ * @param connectionContext - connection context
+ * @param deviceConnectedHandler - device connected handler
*/
- public HandshakeListenerImpl(final ConnectionContext connectionContext, final DeviceConnectedHandler deviceConnectedHandler) {
+ public HandshakeListenerImpl(final ConnectionContext connectionContext,
+ final DeviceConnectedHandler deviceConnectedHandler) {
this.connectionContext = connectionContext;
this.deviceConnectedHandler = deviceConnectedHandler;
}
@Override
public void onHandshakeSuccessful(final GetFeaturesOutput featureOutput, final Short version) {
- LOG.debug("handshake succeeded: {}", connectionContext.getConnectionAdapter().getRemoteAddress());
- closeHandshakeContext();
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("handshake succeeded: {}", connectionContext.getConnectionAdapter().getRemoteAddress());
+ }
+ this.handshakeContext.close();
connectionContext.changeStateToWorking();
connectionContext.setFeatures(featureOutput);
connectionContext.setNodeId(InventoryDataServiceUtil.nodeIdFromDatapathId(featureOutput.getDatapathId()));
private FutureCallback<RpcResult<BarrierOutput>> addBarrierCallback() {
return new FutureCallback<RpcResult<BarrierOutput>>() {
@Override
+ @SuppressWarnings("checkstyle:IllegalCatch")
public void onSuccess(@Nullable final RpcResult<BarrierOutput> result) {
- LOG.debug("succeeded by getting sweep barrier after posthandshake for device {}", connectionContext.getNodeId());
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("succeeded by getting sweep barrier after post-handshake for device {}",
+ connectionContext.getDeviceInfo());
+ }
try {
- if (!deviceConnectedHandler.deviceConnected(connectionContext)) {
- connectionContext.closeConnection(true);
+ ConnectionStatus connectionStatusResult = deviceConnectedHandler.deviceConnected(connectionContext);
+ if (connectionStatusResult != ConnectionStatus.MAY_CONTINUE) {
+ connectionContext.closeConnection(false);
}
- SessionStatistics.countEvent(connectionContext.getNodeId().toString(),
+ SessionStatistics.countEvent(connectionContext.getDeviceInfo().toString(),
SessionStatistics.ConnectionStatus.CONNECTION_CREATED);
} catch (final Exception e) {
- LOG.error("ConnectionContext initial processing failed: {}", e.getMessage());
- SessionStatistics.countEvent(connectionContext.getNodeId().toString(),
+ LOG.warn("initial processing failed for device {}", connectionContext.getDeviceInfo(), e);
+ SessionStatistics.countEvent(connectionContext.getDeviceInfo().toString(),
SessionStatistics.ConnectionStatus.CONNECTION_DISCONNECTED_BY_OFP);
connectionContext.closeConnection(true);
}
}
@Override
- public void onFailure(final Throwable t) {
- LOG.error("failed to get sweep barrier after posthandshake for device {}", connectionContext.getNodeId());
+ public void onFailure(final Throwable throwable) {
+ LOG.warn("failed to get sweep barrier after post-handshake for device {}",
+ connectionContext.getDeviceInfo(), throwable);
connectionContext.closeConnection(false);
}
};
}
- protected ListenableFuture<RpcResult<BarrierOutput>> fireBarrier(final Short version, final long xid) {
+ private ListenableFuture<RpcResult<BarrierOutput>> fireBarrier(final Short version, final long xid) {
final BarrierInput barrierInput = new BarrierInputBuilder()
.setXid(xid)
.setVersion(version)
.build();
return JdkFutureAdapters.listenInPoolThread(
- connectionContext.getConnectionAdapter().barrier(barrierInput));
+ this.connectionContext.getConnectionAdapter().barrier(barrierInput));
}
@Override
public void onHandshakeFailure() {
- LOG.debug("handshake failed: {}", connectionContext.getConnectionAdapter().getRemoteAddress());
- closeHandshakeContext();
- connectionContext.closeConnection(false);
- }
-
- private void closeHandshakeContext() {
- try {
- handshakeContext.close();
- } catch (final Exception e) {
- LOG.error("Closing handshake context failed: {}", e.getMessage());
- LOG.debug("Detail in handshake context close: {}", e);
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("handshake failed: {}", this.connectionContext.getConnectionAdapter().getRemoteAddress());
}
+ this.handshakeContext.close();
+ this.connectionContext.closeConnection(false);
}
@Override