* terms of the Eclipse Public License v1.0 which accompanies this distribution,
* and is available at http://www.eclipse.org/legal/epl-v10.html
*/
-
package org.opendaylight.protocol.bgp.rib.impl;
import static org.junit.Assert.assertEquals;
import static org.opendaylight.protocol.util.CheckUtil.checkReceivedMessages;
import static org.opendaylight.protocol.util.CheckUtil.waitFutureSuccess;
-import com.google.common.collect.ImmutableMap;
import io.netty.channel.Channel;
import io.netty.channel.ChannelFuture;
import java.net.InetSocketAddress;
import java.util.Map;
-import java.util.concurrent.ExecutionException;
import org.junit.After;
import org.junit.Before;
import org.junit.Test;
import org.opendaylight.protocol.bgp.mode.api.PathSelectionMode;
import org.opendaylight.protocol.bgp.mode.impl.add.n.paths.AddPathBestNPathSelection;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev180329.open.message.BgpParameters;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev200120.open.message.BgpParameters;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.rib.rev180329.PeerRole;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.rib.rev180329.RibId;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.rib.rev180329.rib.TablesKey;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev180329.BgpId;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev180329.Ipv4AddressFamily;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev180329.UnicastSubsequentAddressFamily;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev200120.BgpId;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev200120.Ipv4AddressFamily;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev200120.UnicastSubsequentAddressFamily;
public class AddPathNPathsTest extends AbstractAddPathTest {
private RIBImpl ribImpl;
private Channel serverChannel;
+ @Override
@Before
public void setUp() throws Exception {
super.setUp();
- final TablesKey tk = new TablesKey(Ipv4AddressFamily.class, UnicastSubsequentAddressFamily.class);
- final Map<TablesKey, PathSelectionMode> pathTables = ImmutableMap.of(tk,
- new AddPathBestNPathSelection(2L, this.peerTracker));
+ final TablesKey tk = new TablesKey(Ipv4AddressFamily.VALUE, UnicastSubsequentAddressFamily.VALUE);
+ final Map<TablesKey, PathSelectionMode> pathTables = Map.of(tk, new AddPathBestNPathSelection(2));
- this.ribImpl = new RIBImpl(new RibId("test-rib"), AS_NUMBER, new BgpId(RIB_ID), this.ribExtension,
- this.serverDispatcher, this.codecsRegistry, getDomBroker(), getDataBroker(), this.policies,
- this.peerTracker, TABLES_TYPE, pathTables);
+ ribImpl = new RIBImpl(tableRegistry, new RibId("test-rib"), AS_NUMBER, new BgpId(RIB_ID), ribExtension,
+ serverDispatcher, codecsRegistry, getDomBroker(), policies, TABLES_TYPE, pathTables);
- this.ribImpl.instantiateServiceInstance();
- this.ribImpl.onGlobalContextUpdated(this.schemaService.getGlobalContext());
- final ChannelFuture channelFuture = this.serverDispatcher.createServer(new InetSocketAddress(RIB_ID, PORT));
+ ribImpl.instantiateServiceInstance();
+ final ChannelFuture channelFuture = serverDispatcher.createServer(
+ new InetSocketAddress(RIB_ID, PORT.toJava()));
waitFutureSuccess(channelFuture);
- this.serverChannel = channelFuture.channel();
+ serverChannel = channelFuture.channel();
}
+ @Override
@After
- public void tearDown() throws ExecutionException, InterruptedException {
- waitFutureSuccess(this.serverChannel.close());
+ public void tearDown() throws Exception {
+ waitFutureSuccess(serverChannel.close());
super.tearDown();
}
+
/*
* N-Paths
* ___________________
* | ODL BGP 127.0.0.1 |
- * [peer://127.0.0.2; p1, lp100] --(iBGP)--> | | --(RR-client, non add-path) --> [Peer://127.0.0.5; (p1, lp100), (p1, lp1200)]
- * [peer://127.0.0.3; p1, lp200] --(iBGP)--> | |
- * [peer://127.0.0.4; p1, lp50] --(iBGP)--> | | --(RR-client, add-path) --> [Peer://127.0.0.6; (p1, path-id1, lp100), (p1, path-id2, pl50), (p1, path-id3, pl200)]
- * [peer://127.0.0.2; p1, lp20] --(iBGP)--> |___________________|
- * p1 = 1.1.1.1/32
+ * [peer://127.0.0.2; p1, lp100] --(iBGP)--> | | --(RR-client, non add-path) -->
+ * [peer://127.0.0.3; p1, lp200] --(iBGP)--> | | [Peer://127.0.0.5; (p1, lp100), (p1, lp1200)]
+ * [peer://127.0.0.4; p1, lp50] --(iBGP)--> | | --(RR-client, add-path) -->
+ * [peer://127.0.0.2; p1, lp20] --(iBGP)--> |___________________| [Peer://127.0.0.6; (p1, path-id1, lp100),
+ * p1 = 1.1.1.1/32 (p1, path-id2, pl50), (p1, path-id3, pl200)]
*/
@Test
public void testUseCase1() throws Exception {
final BgpParameters nonAddPathParams = createParameter(false);
final BgpParameters addPathParams = createParameter(true);
- configurePeer(PEER1, this.ribImpl, nonAddPathParams, PeerRole.Ibgp, this.serverRegistry);
+ configurePeer(tableRegistry, PEER1, ribImpl, nonAddPathParams, PeerRole.Ibgp, serverRegistry);
final BGPSessionImpl session1 = createPeerSession(PEER1, nonAddPathParams, new SimpleSessionListener());
- configurePeer(PEER2, this.ribImpl, nonAddPathParams, PeerRole.Ibgp, this.serverRegistry);
- final BGPSessionImpl session2 = createPeerSession(PEER2, nonAddPathParams, new SimpleSessionListener());
+ configurePeer(tableRegistry, PEER2, ribImpl, nonAddPathParams, PeerRole.Ibgp, serverRegistry);
+ final BGPSessionImpl session2 = createPeerSession(PEER2, nonAddPathParams, new SimpleSessionListener());
- configurePeer(PEER3, this.ribImpl, nonAddPathParams, PeerRole.Ibgp, this.serverRegistry);
+ configurePeer(tableRegistry, PEER3, ribImpl, nonAddPathParams, PeerRole.Ibgp, serverRegistry);
final BGPSessionImpl session3 = createPeerSession(PEER3, nonAddPathParams, new SimpleSessionListener());
final SimpleSessionListener listener4 = new SimpleSessionListener();
- configurePeer(PEER4, this.ribImpl, nonAddPathParams, PeerRole.RrClient, this.serverRegistry);
+ configurePeer(tableRegistry, PEER4, ribImpl, nonAddPathParams, PeerRole.RrClient,
+ serverRegistry);
final BGPSessionImpl session4 = createPeerSession(PEER4, nonAddPathParams, listener4);
final SimpleSessionListener listener5 = new SimpleSessionListener();
- configurePeer(PEER5, this.ribImpl, addPathParams, PeerRole.RrClient, this.serverRegistry);
+ configurePeer(tableRegistry, PEER5, ribImpl, addPathParams, PeerRole.RrClient, serverRegistry);
final BGPSessionImpl session5 = createPeerSession(PEER5, addPathParams, listener5);
checkPeersPresentOnDataStore(5);
//new best route so far
sendRouteAndCheckIsOnLocRib(session1, PREFIX1, 100, 1);
- checkReceivedMessages(listener4, 1);
- checkReceivedMessages(listener5, 1);
- assertEquals(UPD_100, listener5.getListMsg().get(0));
+ checkReceivedMessages(listener4, 2);
+ checkReceivedMessages(listener5, 2);
+ assertEquals(UPD_100, listener5.getListMsg().get(1));
final SimpleSessionListener listener6 = new SimpleSessionListener();
- configurePeer(PEER6, this.ribImpl, nonAddPathParams, PeerRole.RrClient, this.serverRegistry);
+ configurePeer(tableRegistry, PEER6, ribImpl, nonAddPathParams, PeerRole.RrClient,
+ serverRegistry);
final BGPSessionImpl session6 = createPeerSession(PEER6, nonAddPathParams, listener6);
checkPeersPresentOnDataStore(6);
- checkReceivedMessages(listener6, 1);
- assertEquals(UPD_NA_100, listener6.getListMsg().get(0));
+ checkReceivedMessages(listener6, 2);
+ assertEquals(UPD_NA_100, listener6.getListMsg().get(1));
session6.close();
checkPeersPresentOnDataStore(5);
//the second best route
sendRouteAndCheckIsOnLocRib(session2, PREFIX1, 50, 2);
- checkReceivedMessages(listener4, 1);
- checkReceivedMessages(listener5, 2);
- assertEquals(UPD_50, listener5.getListMsg().get(1));
+ checkReceivedMessages(listener4, 2);
+ checkReceivedMessages(listener5, 3);
+ assertEquals(UPD_50, listener5.getListMsg().get(2));
//new best route
sendRouteAndCheckIsOnLocRib(session3, PREFIX1, 200, 2);
- checkReceivedMessages(listener4, 2);
- checkReceivedMessages(listener5, 3);
- assertEquals(UPD_200, listener5.getListMsg().get(2));
+ checkReceivedMessages(listener4, 3);
+ checkReceivedMessages(listener5, 4);
+ assertEquals(UPD_200, listener5.getListMsg().get(3));
//the worst prefix, no changes
sendRouteAndCheckIsOnLocRib(session2, PREFIX1, 20, 2);
- checkReceivedMessages(listener4, 2);
- checkReceivedMessages(listener5, 3);
+ checkReceivedMessages(listener4, 3);
+ checkReceivedMessages(listener5, 4);
//withdraw second best route, 2 advertisement (1 withdrawal) for add-path supported, none for non add path
sendWithdrawalRouteAndCheckIsOnLocRib(session1, PREFIX1, 100, 2);
- checkReceivedMessages(listener4, 2);
- checkReceivedMessages(listener5, 5);
+ checkReceivedMessages(listener4, 3);
+ checkReceivedMessages(listener5, 6);
//we advertise again to try new test
sendRouteAndCheckIsOnLocRib(session1, PREFIX1, 100, 2);
- checkReceivedMessages(listener4, 2);
- checkReceivedMessages(listener5, 6);
+ checkReceivedMessages(listener4, 3);
+ checkReceivedMessages(listener5, 7);
- //withdraw second best route, 2 advertisement (1 withdrawal) for add-path supported, 1 withdrawal for non add path
+ //withdraw second best route, 2 advertisement (1 withdrawal) for add-path supported,
+ // 1 withdrawal for non add path
sendWithdrawalRouteAndCheckIsOnLocRib(session3, PREFIX1, 200, 2);
- checkReceivedMessages(listener4, 3);
- checkReceivedMessages(listener5, 8);
+ checkReceivedMessages(listener4, 4);
+ checkReceivedMessages(listener5, 9);
session1.close();
session2.close();
Code Review / bgpcep.git / blobdiff