BUG-353: add remote-as awareness
[bgpcep.git] / bgp / rib-impl / src / main / java / org / opendaylight / protocol / bgp / rib / impl / BGPSessionImpl.java
index 65bb948121bb48332b68a3c77b2c3e475b50d235..e40f60f3d3d086f3e4dbf7a421b464f072d6f799 100644 (file)
@@ -19,18 +19,27 @@ import java.util.concurrent.TimeUnit;
 
 import javax.annotation.concurrent.GuardedBy;
 
-import org.opendaylight.protocol.bgp.concepts.BGPTableType;
+import org.opendaylight.protocol.bgp.parser.AsNumberUtil;
 import org.opendaylight.protocol.bgp.parser.BGPError;
-import org.opendaylight.protocol.bgp.parser.BGPMessage;
-import org.opendaylight.protocol.bgp.parser.BGPParameter;
 import org.opendaylight.protocol.bgp.parser.BGPSession;
 import org.opendaylight.protocol.bgp.parser.BGPSessionListener;
 import org.opendaylight.protocol.bgp.parser.BGPTerminationReason;
-import org.opendaylight.protocol.bgp.parser.message.BGPKeepAliveMessage;
-import org.opendaylight.protocol.bgp.parser.message.BGPNotificationMessage;
-import org.opendaylight.protocol.bgp.parser.message.BGPOpenMessage;
-import org.opendaylight.protocol.bgp.parser.parameter.MultiprotocolCapability;
+import org.opendaylight.protocol.bgp.parser.BgpTableTypeImpl;
 import org.opendaylight.protocol.framework.AbstractProtocolSession;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev100924.AsNumber;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev100924.Ipv4Address;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.Keepalive;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.KeepaliveBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.Notify;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.NotifyBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.Open;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.Update;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.open.BgpParameters;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev130919.open.bgp.parameters.CParameters;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.multiprotocol.rev130919.BgpTableType;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.multiprotocol.rev130919.open.bgp.parameters.c.parameters.MultiprotocolCase;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.rib.rev130925.rib.TablesKey;
+import org.opendaylight.yangtools.yang.binding.Notification;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -41,24 +50,25 @@ import com.google.common.base.Preconditions;
 import com.google.common.collect.Sets;
 
 @VisibleForTesting
-public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implements BGPSession {
+public class BGPSessionImpl extends AbstractProtocolSession<Notification> implements BGPSession {
 
-       private static final Logger logger = LoggerFactory.getLogger(BGPSessionImpl.class);
+       private static final Logger LOG = LoggerFactory.getLogger(BGPSessionImpl.class);
 
+       /*
+        * 240
+        */
        private static final int DEFAULT_HOLD_TIMER_VALUE = 15;
 
-       public static int HOLD_TIMER_VALUE = DEFAULT_HOLD_TIMER_VALUE; // 240
+       private static final Notification KEEP_ALIVE = new KeepaliveBuilder().build();
 
        /**
         * Internal session state.
         */
        public enum State {
                /**
-                * The session object is created by the negotiator in OpenConfirm state.
-                * While in this state, the session object is half-alive, e.g. the timers
-                * are running, but the session is not completely up, e.g. it has not been
-                * announced to the listener. If the session is torn down in this state,
-                * we do not inform the listener.
+                * The session object is created by the negotiator in OpenConfirm state. While in this state, the session object
+                * is half-alive, e.g. the timers are running, but the session is not completely up, e.g. it has not been
+                * announced to the listener. If the session is torn down in this state, we do not inform the listener.
                 */
                OpenConfirm,
                /**
@@ -74,6 +84,7 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
        /**
         * System.nanoTime value about when was sent the last message Protected to be updated also in tests.
         */
+       @VisibleForTesting
        protected long lastMessageSentAt;
 
        /**
@@ -97,36 +108,46 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
        @GuardedBy("this")
        private State state = State.OpenConfirm;
 
-       private final int keepAlive;
 
-       private final Set<BGPTableType> tableTypes;
+       private final Set<BgpTableType> tableTypes;
+       private final int holdTimerValue;
+       private final int keepAlive;
+       private final AsNumber asNumber;
+       private final Ipv4Address bgpId;
 
-       BGPSessionImpl(final Timer timer, final BGPSessionListener listener, final Channel channel, final BGPOpenMessage remoteOpen) {
+       BGPSessionImpl(final Timer timer, final BGPSessionListener listener, final Channel channel, final Open remoteOpen) {
                this.listener = Preconditions.checkNotNull(listener);
                this.stateTimer = Preconditions.checkNotNull(timer);
                this.channel = Preconditions.checkNotNull(channel);
-               this.keepAlive = remoteOpen.getHoldTime() / 3;
-
-               final Set<BGPTableType> tts = Sets.newHashSet();
-               if (remoteOpen.getOptParams() != null) {
-                       for (final BGPParameter param : remoteOpen.getOptParams()) {
-                               if (param instanceof MultiprotocolCapability) {
-                                       tts.add(((MultiprotocolCapability) param).getTableType());
+               this.keepAlive = remoteOpen.getHoldTimer() / 3;
+               this.holdTimerValue = remoteOpen.getHoldTimer();
+               this.asNumber = AsNumberUtil.advertizedAsNumber(remoteOpen);
+
+               final Set<TablesKey> tts = Sets.newHashSet();
+               final Set<BgpTableType> tats = Sets.newHashSet();
+               if (remoteOpen.getBgpParameters() != null) {
+                       for (final BgpParameters param : remoteOpen.getBgpParameters()) {
+                               final CParameters cp = param.getCParameters();
+                               if (cp instanceof MultiprotocolCase) {
+                                       final TablesKey tt = new TablesKey(((MultiprotocolCase) cp).getMultiprotocolCapability().getAfi(), ((MultiprotocolCase) cp).getMultiprotocolCapability().getSafi());
+                                       LOG.trace("Added table type to sync {}", tt);
+                                       tts.add(tt);
+                                       tats.add(new BgpTableTypeImpl(tt.getAfi(), tt.getSafi()));
                                }
                        }
                }
 
                this.sync = new BGPSynchronization(this, this.listener, tts);
-               this.tableTypes = tts;
+               this.tableTypes = tats;
 
-               if (remoteOpen.getHoldTime() != 0) {
+               if (remoteOpen.getHoldTimer() != 0) {
                        this.stateTimer.newTimeout(new TimerTask() {
 
                                @Override
                                public void run(final Timeout timeout) throws Exception {
                                        handleHoldTimer();
                                }
-                       }, remoteOpen.getHoldTime(), TimeUnit.SECONDS);
+                       }, remoteOpen.getHoldTimer(), TimeUnit.SECONDS);
 
                        this.stateTimer.newTimeout(new TimerTask() {
                                @Override
@@ -135,13 +156,14 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
                                }
                        }, this.keepAlive, TimeUnit.SECONDS);
                }
+               this.bgpId = remoteOpen.getBgpIdentifier();
        }
 
        @Override
        public synchronized void close() {
-               logger.debug("Closing session: {}", this);
+               LOG.debug("Closing session: {}", this);
                if (this.state != State.Idle) {
-                       this.sendMessage(new BGPNotificationMessage(BGPError.CEASE));
+                       this.sendMessage(new NotifyBuilder().setErrorCode(BGPError.CEASE.getCode()).build());
                        this.channel.close();
                        this.state = State.Idle;
                }
@@ -153,21 +175,23 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
         * @param msg incoming message
         */
        @Override
-       public void handleMessage(final BGPMessage msg) {
+       public void handleMessage(final Notification msg) {
                // Update last reception time
                this.lastMessageReceivedAt = System.nanoTime();
 
-               if (msg instanceof BGPOpenMessage) {
+               if (msg instanceof Open) {
                        // Open messages should not be present here
                        this.terminate(BGPError.FSM_ERROR);
-               } else if (msg instanceof BGPNotificationMessage) {
+               } else if (msg instanceof Notify) {
                        // Notifications are handled internally
-                       logger.info("Session closed because Notification message received: {}", ((BGPNotificationMessage) msg).getError());
+                       LOG.info("Session closed because Notification message received: {} / {}", ((Notify) msg).getErrorCode(),
+                                       ((Notify) msg).getErrorSubcode());
                        this.closeWithoutMessage();
-                       this.listener.onSessionTerminated(this, new BGPTerminationReason(((BGPNotificationMessage) msg).getError()));
-               } else if (msg instanceof BGPKeepAliveMessage) {
+                       this.listener.onSessionTerminated(this,
+                                       new BGPTerminationReason(BGPError.forValue(((Notify) msg).getErrorCode(), ((Notify) msg).getErrorSubcode())));
+               } else if (msg instanceof Keepalive) {
                        // Keepalives are handled internally
-                       logger.debug("Received KeepAlive messsage.");
+                       LOG.debug("Received KeepAlive messsage.");
                        this.kaCounter++;
                        if (this.kaCounter >= 2) {
                                this.sync.kaReceived();
@@ -175,6 +199,7 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
                } else {
                        // All others are passed up
                        this.listener.onMessage(this, msg);
+                       this.sync.updReceived((Update) msg);
                }
        }
 
@@ -185,18 +210,18 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
                }
        }
 
-       void sendMessage(final BGPMessage msg) {
+       void sendMessage(final Notification msg) {
                try {
                        this.channel.writeAndFlush(msg);
                        this.lastMessageSentAt = System.nanoTime();
-                       logger.debug("Sent message: {}", msg);
+                       LOG.debug("Sent message: {}", msg);
                } catch (final Exception e) {
-                       logger.warn("Message {} was not sent.", msg, e);
+                       LOG.warn("Message {} was not sent.", msg, e);
                }
        }
 
        private synchronized void closeWithoutMessage() {
-               logger.debug("Closing session: {}", this);
+               LOG.debug("Closing session: {}", this);
                this.channel.close();
                this.state = State.Idle;
        }
@@ -208,7 +233,7 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
         * @param closeObject
         */
        private void terminate(final BGPError error) {
-               this.sendMessage(new BGPNotificationMessage(error));
+               this.sendMessage(new NotifyBuilder().setErrorCode(error.getCode()).setErrorSubcode(error.getSubcode()).build());
                this.closeWithoutMessage();
 
                this.listener.onSessionTerminated(this, new BGPTerminationReason(error));
@@ -226,10 +251,10 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
                }
 
                final long ct = System.nanoTime();
-               final long nextHold = this.lastMessageReceivedAt + TimeUnit.SECONDS.toNanos(HOLD_TIMER_VALUE);
+               final long nextHold = this.lastMessageReceivedAt + TimeUnit.SECONDS.toNanos(holdTimerValue);
 
                if (ct >= nextHold) {
-                       logger.debug("HoldTimer expired. " + new Date());
+                       LOG.debug("HoldTimer expired. " + new Date());
                        this.terminate(BGPError.HOLD_TIMER_EXPIRED);
                } else {
                        this.stateTimer.newTimeout(new TimerTask() {
@@ -256,7 +281,7 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
                long nextKeepalive = this.lastMessageSentAt + TimeUnit.SECONDS.toNanos(this.keepAlive);
 
                if (ct >= nextKeepalive) {
-                       this.sendMessage(new BGPKeepAliveMessage());
+                       this.sendMessage(KEEP_ALIVE);
                        nextKeepalive = this.lastMessageSentAt + TimeUnit.SECONDS.toNanos(this.keepAlive);
                }
                this.stateTimer.newTimeout(new TimerTask() {
@@ -268,7 +293,7 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
        }
 
        @Override
-       final public String toString() {
+       public final String toString() {
                return addToStringAttributes(Objects.toStringHelper(this)).toString();
        }
 
@@ -279,7 +304,7 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
        }
 
        @Override
-       public Set<BGPTableType> getAdvertisedTableTypes() {
+       public Set<BgpTableType> getAdvertisedTableTypes() {
                return this.tableTypes;
        }
 
@@ -292,4 +317,14 @@ public class BGPSessionImpl extends AbstractProtocolSession<BGPMessage> implemen
        public synchronized State getState() {
                return this.state;
        }
+
+       @Override
+       public final byte[] getBgpId() {
+               return this.bgpId.getValue().getBytes();
+       }
+
+       @Override
+       public final AsNumber getAsNumber() {
+               return this.asNumber;
+       }
 }