Integrate MRI projects for Neon 06/75006/32
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 9 Aug 2018 13:36:25 +0000 (15:36 +0200)
committerClaudio David Gasparini <claudio.gasparini@pantheon.tech>
Wed, 24 Oct 2018 13:39:09 +0000 (13:39 +0000)
This patch bumps:
- odlparent to 4.0.2
- yangtools to 2.1.2
- mdsal to 3.0.1

It also fixes assumptions around list presence when it is empty.

JIRA: BGPCEP-827
Change-Id: Ifecf9e3a7909f2a95776a90e27316f0902ca12cd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
108 files changed:
artifacts/pom.xml
bgp/benchmark-app/pom.xml
bgp/benchmark-app/src/main/java/org/opendaylight/protocol/bgp/benchmark/app/AppPeerBenchmark.java
bgp/benchmark-app/src/test/java/org/opendaylight/protocol/bgp/benchmark/app/AppPeerBenchmarkTest.java
bgp/bgp-artifacts/pom.xml
bgp/cli/pom.xml
bgp/concepts/pom.xml
bgp/config-example/pom.xml
bgp/extensions/evpn/pom.xml
bgp/extensions/mvpn/pom.xml
bgp/extensions/pom.xml
bgp/openconfig-api/pom.xml
bgp/openconfig-rp-impl/pom.xml
bgp/openconfig-rp-spi/pom.xml
bgp/openconfig-rp-statement/pom.xml
bgp/openconfig-state/pom.xml
bgp/parser-api/pom.xml
bgp/parser-impl/pom.xml
bgp/parser-mock/pom.xml
bgp/parser-spi/pom.xml
bgp/path-selection-mode/pom.xml
bgp/peer-acceptor/pom.xml
bgp/pom.xml
bgp/rib-api/pom.xml
bgp/rib-impl/pom.xml
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/AbstractAddPathTest.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/AbstractRIBTestSetup.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/config/AbstractConfig.java
bgp/rib-impl/src/test/java/org/opendaylight/protocol/bgp/rib/impl/config/BgpPeerTest.java
bgp/rib-mock/pom.xml
bgp/rib-spi/pom.xml
bgp/testtool/pom.xml
bgp/testtool/src/main/java/org/opendaylight/protocol/bgp/testtool/BGPPeerBuilder.java
bgp/topology-provider/pom.xml
bgp/topology-provider/src/main/java/org/opendaylight/bgpcep/bgp/topology/provider/AbstractReachabilityTopologyBuilder.java
bgp/topology-provider/src/test/java/org/opendaylight/bgpcep/bgp/topology/provider/Ipv4ReachabilityTopologyBuilderTest.java
bgp/topology-provider/src/test/java/org/opendaylight/bgpcep/bgp/topology/provider/Ipv6ReachabilityTopologyBuilderTest.java
bgp/topology-provider/src/test/java/org/opendaylight/bgpcep/bgp/topology/provider/LinkstateTopologyBuilderTest.java
bgp/util/pom.xml
binding-parent/pom.xml
bmp/bmp-api/pom.xml
bmp/bmp-artifacts/pom.xml
bmp/bmp-config-example/pom.xml
bmp/bmp-impl/pom.xml
bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/app/BmpMonitorImplTest.java
bmp/bmp-mock/pom.xml
bmp/bmp-parser-impl/pom.xml
bmp/bmp-spi/pom.xml
bmp/pom.xml
concepts/pom.xml
config-loader/config-loader-artifacts/pom.xml
config-loader/config-loader-impl/pom.xml
config-loader/pom.xml
distribution-karaf/pom.xml
feature-repo-parent/pom.xml
features/bgp/odl-bgpcep-bgp-dependencies/src/main/feature/feature.xml
features/bgp/pom.xml
features/bgpcep-extras/pom.xml
features/bmp/pom.xml
features/concepts/pom.xml
features/config-loader/pom.xml
features/pcep/pom.xml
features/pom.xml
features/rsvp/pom.xml
features/topology/pom.xml
parent/pom.xml
pcep/api/pom.xml
pcep/auto-bandwidth-extension/pom.xml
pcep/base-parser/pom.xml
pcep/config-example/pom.xml
pcep/ietf-stateful07/pom.xml
pcep/impl/pom.xml
pcep/pcc-mock/pom.xml
pcep/pcep-artifacts/pom.xml
pcep/pom.xml
pcep/segment-routing/pom.xml
pcep/topology/pom.xml
pcep/topology/topology-api/pom.xml
pcep/topology/topology-provider/pom.xml
pcep/topology/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/TopologyNodeState.java
pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/PCEPTriggeredReSynchronizationProcedureTest.java
pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/StateSynchronizationAvoidanceProcedureTest.java
pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/Stateful07TopologySessionListenerTest.java
pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/TopologyProgrammingTest.java
pcep/topology/topology-spi/src/main/java/org/opendaylight/bgpcep/pcep/topology/spi/AbstractInstructionExecutor.java
pcep/topology/topology-stats/src/main/java/org/opendaylight/bgpcep/pcep/topology/stats/provider/TopologyStatsProviderImpl.java
pcep/tunnel/pom.xml
pcep/tunnel/tunnel-provider/pom.xml
pcep/tunnel/tunnel-provider/src/main/java/org/opendaylight/bgpcep/pcep/tunnel/provider/NodeChangedListener.java
pcep/tunnel/tunnel-provider/src/main/java/org/opendaylight/bgpcep/pcep/tunnel/provider/PCEPTunnelTopologyProvider.java
pcep/tunnel/tunnel-provider/src/main/java/org/opendaylight/bgpcep/pcep/tunnel/provider/TunnelProviderDeployer.java
pcep/tunnel/tunnel-provider/src/test/java/org/opendaylight/bgpcep/pcep/tunnel/provider/NodeChangedListenerTest.java
pom.xml
programming/api/pom.xml
programming/pom.xml
programming/programming-artifacts/pom.xml
rsvp/api/pom.xml
rsvp/impl/pom.xml
rsvp/pom.xml
rsvp/rsvp-artifacts/pom.xml
single-feature-parent/pom.xml
testtool-parent/pom.xml
testtool-util/src/main/java/org/opendaylight/protocol/util/CheckUtil.java
testtool-util/src/test/java/org/opendaylight/protocol/util/CheckUtilTest.java
topology/pom.xml
topology/topology-artifacts/pom.xml
topology/tunnel-api/pom.xml
util/pom.xml

index 469b8e8f00a3786cd572b1772217d7c922343763..767756331e38fb8d484bbe65377dbd4fbfb5e74c 100644 (file)
@@ -14,7 +14,7 @@
     <parent>
       <groupId>org.opendaylight.odlparent</groupId>
       <artifactId>odlparent-lite</artifactId>
-      <version>3.1.3</version>
+      <version>4.0.2</version>
       <relativePath/>
     </parent>
 
index 93230ddda092eeea9d298075515b77ed822edcff..42983bedc515c163177a5fa984d14aa21b21fbd1 100644 (file)
@@ -48,8 +48,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>${project.groupId}</groupId>
index fc729fd6fe113432068bc9e3ae5fbfc1f551db17..ce4314d881b85a204dd78228f406190cdf06c1e7 100644 (file)
@@ -137,8 +137,8 @@ public class AppPeerBenchmark implements OdlBgpAppPeerBenchmarkService, Transact
     @Override
     public void onTransactionChainFailed(final TransactionChain<?, ?> chain, final AsyncTransaction<?, ?> transaction,
             final Throwable cause) {
-        LOG.error("Broken chain {} in DatastoreBaAbstractWrite, transaction {}, cause {}", chain,
-                transaction.getIdentifier(), cause);
+        LOG.error("Broken chain {} in DatastoreBaAbstractWrite, transaction {}", chain, transaction.getIdentifier(),
+            cause);
         close();
     }
 
@@ -278,4 +278,4 @@ public class AppPeerBenchmark implements OdlBgpAppPeerBenchmarkService, Transact
     private static String createKey(final String address) {
         return address + PREFIX;
     }
-}
\ No newline at end of file
+}
index ba64fe10f908c32180011fd3579733762968efa2..e2bd03627d6f466f9f56aa991f95c4facc23f827 100644 (file)
@@ -5,15 +5,14 @@
  * 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.benchmark.app;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
 import static org.mockito.Mockito.doNothing;
 import static org.mockito.Mockito.doReturn;
 import static org.opendaylight.protocol.util.CheckUtil.checkEquals;
+import static org.opendaylight.protocol.util.CheckUtil.checkNotPresentConfiguration;
 import static org.opendaylight.protocol.util.CheckUtil.readDataConfiguration;
 
 import javax.management.MalformedObjectNameException;
@@ -86,11 +85,8 @@ public class AppPeerBenchmarkTest extends AbstractConcurrentDataBrokerTest {
         final Result deleteResult = deleteRpcResult.getResult().getResult();
         checkEquals(() -> assertEquals(1, deleteResult.getCount().intValue()));
         checkEquals(() -> assertEquals(1, deleteResult.getRate().intValue()));
-        readDataConfiguration(getDataBroker(), routesIID, routes -> {
-            assertNotNull(routes.getIpv4Route());
-            assertTrue(routes.getIpv4Route().isEmpty());
-            return routes;
-        });
+
+        checkNotPresentConfiguration(getDataBroker(), appPeerBenchmark.getIpv4RoutesIID());
 
         appPeerBenchmark.close();
     }
index eb9f53f2cf969b1eac5d65d4030ce8206f7fc5da..ddf70f98d5fbea2f8f080f124ddab94b6f9acdbe 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index a879f4943fe22a87ea933bd8b946d029ae69a03c..9a8c00dcd780311568b95a35c1878717d4750402 100644 (file)
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.apache.karaf.shell</groupId>
index 688d0d360515e3b921119e9cc56c1347f6102c96..775e0bdf3eff100b09624c7da658d0f1b6ec0000 100644 (file)
@@ -68,8 +68,8 @@
                     discovered by the plugin.
         -->
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
     </dependencies>
 
index 7202d4f8e086505bf4e7e4536c8cd6880c6d79c1..83939a9fb463b30e48bcdc3ecc3645df1a28f371 100644 (file)
@@ -15,7 +15,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 97f8c900c1c7050e39820e6b6bfe62451119a34b..17cad67908a563944878f6d4250690a8ede31cbc 100644 (file)
@@ -41,8 +41,8 @@
             <artifactId>yang-ext</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.osgi</groupId>
index 01bdf17d1bd988068af317dc2c1a223baf5140c7..132be76a9f1590a112aeca39c8a75692a1459b7e 100644 (file)
@@ -32,8 +32,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
             <groupId>org.opendaylight.mdsal</groupId>
             <artifactId>mdsal-binding-spec-util</artifactId>
         </dependency>
+        <dependency>
+            <groupId>javax.xml.bind</groupId>
+            <artifactId>jaxb-api</artifactId>
+            <version>2.2.1</version>
+            <exclusions>
+                <exclusion>
+                    <!-- Provided by JDK -->
+                    <groupId>javax.xml.stream</groupId>
+                    <artifactId>stax-api</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
 
         <!-- test scope dependencies -->
         <dependency>
index 75fda50fcb395fbaa2ad513ca277a09b1e487524..70bb5d9a7681350f7615a01462057824047e138c 100644 (file)
@@ -89,8 +89,8 @@
             <artifactId>mdsal-binding-dom-codec</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.controller</groupId>
index fd490b1091e4531e1a10c56f2f4180538a2d7b61..ba26c328e807eb48ecae3c1ea569174f69e7d103 100644 (file)
 
     <dependencies>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-interfaces</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc8343</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index 0234a58fd22fcb59a6a96ec712f433e0853d098d..4d29a88041abc65dcf68cc9e3d2517d86d1d1c8d 100644 (file)
@@ -56,8 +56,8 @@
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
index c3abb60c578236ce924010dc5f32f36b88941746..e1972284845e3a27c60d76a3427d534ce3cc8a82 100644 (file)
@@ -56,8 +56,8 @@
             <artifactId>mdsal-binding-spec-util</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.yangtools</groupId>
index 31375178d563e7773be38988b6813693cc7894a6..68ca4c68f54574064f81e20a459aa72a66376423 100644 (file)
@@ -64,8 +64,8 @@
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
index a5bcd97d13bdea5932ae33ced2eb8ca266585557..66fa081532019c799f58b505a019f44c8c593356 100644 (file)
             <artifactId>mdsal-common-api</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
index 19572e909866ed5b3a585382875fc48b878f8d9a..000c2e7dee3697bcb3c5fde7e1f637266f81979c 100644 (file)
@@ -62,8 +62,8 @@
         </dependency>
 
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
index 929d4d0d1b2b6097662cdb700dd29bc9503526c7..742a6fb3a8000c48fdb14f54b92303bc7bff6432 100644 (file)
@@ -63,8 +63,8 @@
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.yangtools</groupId>
index 81190ba109385c32f835d3c0786e4f6932fab4b6..eb8290567e859369046c45d7038ce395fff5b373 100644 (file)
@@ -46,8 +46,8 @@
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
index c410e9d089e6a71893997a8d69e03ee934910c1e..50d31f766dbcb547b72424e0412b0dc3dcd82790 100644 (file)
@@ -67,8 +67,8 @@
             <artifactId>yang-data-impl</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
 
         <dependency>
index 765b1186d0ea4ce887dc823debbfca0b3157636d..c1843e1c54cec727eb2f2267033482d65775fe08 100644 (file)
@@ -46,8 +46,8 @@
             <artifactId>bgp-concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index 11a1fa9e1dd8131389e6e1ee31f03a49b1a45af4..adf1e1d1879c65a889e95944cb0cdd01bd567096 100644 (file)
@@ -41,8 +41,8 @@
             <artifactId>bgp-parser-api</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.yangtools</groupId>
index 61d2e937e99fef00c1476bfbe6bb8b2f2dd9e22b..8a4bb2cd5283e2824194c0ce2958e4289caff848 100644 (file)
@@ -19,7 +19,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 8fd4384729716e9c2cb548e9333604352337b819..3211b6fc81a7841c02b21188292ceb1138b6cbfc 100644 (file)
@@ -54,8 +54,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
index 3b4ffc8e118c004807c08f5ee2a815613a8d0385..04f1f44eaa87ffb255ed2bc1841ebb1431e19ea3 100644 (file)
             <artifactId>mdsal-binding-generator-impl</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index c5e9275b26f261fee3e526f8257632eaf6b1eb36..f091647779e6f9005a571f297b6da53f84abe38a 100644 (file)
@@ -7,7 +7,7 @@
  */
 package org.opendaylight.protocol.bgp.rib.impl;
 
-import static org.mockito.Matchers.any;
+import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doReturn;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
 import static org.opendaylight.protocol.util.CheckUtil.waitFutureSuccess;
@@ -50,6 +50,7 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev180329.bgp.rib.rib.loc.rib.tables.routes.Ipv4RoutesCase;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev180329.ipv4.prefixes.DestinationIpv4Builder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev180329.ipv4.prefixes.destination.ipv4.Ipv4PrefixesBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev180329.ipv4.routes.Ipv4Routes;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev180329.ipv4.routes.ipv4.routes.Ipv4Route;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.inet.rev180329.update.attributes.mp.reach.nlri.advertized.routes.destination.type.DestinationIpv4CaseBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.message.rev180329.NotifyBuilder;
@@ -138,6 +139,7 @@ public abstract class AbstractAddPathTest extends DefaultRibPoliciesMockTest {
     protected StrictBGPPeerRegistry serverRegistry;
     protected CodecsRegistryImpl codecsRegistry;
 
+    @Override
     @Before
     public void setUp() throws Exception {
         super.setUp();
@@ -165,6 +167,7 @@ public abstract class AbstractAddPathTest extends DefaultRibPoliciesMockTest {
                 this.ribExtension.getClassLoadingStrategy());
     }
 
+    @Override
     @After
     public void tearDown() throws ExecutionException, InterruptedException {
         this.serverDispatcher.close();
@@ -203,12 +206,25 @@ public abstract class AbstractAddPathTest extends DefaultRibPoliciesMockTest {
     }
 
     private void checkLocRib(final int expectedRoutesOnDS) throws Exception {
+        // FIXME: remove this sleep
         Thread.sleep(100);
         readDataOperational(getDataBroker(), BGP_IID, bgpRib -> {
             final Ipv4RoutesCase routes = (Ipv4RoutesCase) bgpRib.getRib().get(0).getLocRib().getTables().get(0)
                 .getRoutes();
-            final List<Ipv4Route> routeList = routes.getIpv4Routes().getIpv4Route();
-            Assert.assertEquals(expectedRoutesOnDS, routeList.size());
+            final int size;
+            if (routes != null) {
+                final Ipv4Routes routesCase = routes.getIpv4Routes();
+                if (routesCase != null) {
+                    final List<Ipv4Route> routeList = routesCase.getIpv4Route();
+                    size = routeList == null ? 0 : routeList.size();
+                } else {
+                    size = 0;
+                }
+            } else {
+                size = 0;
+            }
+
+            Assert.assertEquals(expectedRoutesOnDS, size);
             return bgpRib;
         });
     }
index 6f88071caac5a8ce436ad6d84648dd30074f1f41..5e28c92fdfa0e7682e1249b437793b1731d3220d 100644 (file)
@@ -7,8 +7,8 @@
  */
 package org.opendaylight.protocol.bgp.rib.impl;
 
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.eq;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.doCallRealMethod;
 import static org.mockito.Mockito.doNothing;
 import static org.mockito.Mockito.doReturn;
@@ -211,7 +211,7 @@ public class AbstractRIBTestSetup extends DefaultRibPoliciesMockTest {
         doNothing().when(getTransaction()).put(eq(LogicalDatastoreType.OPERATIONAL),
                 eq(YangInstanceIdentifier.of(BgpRib.QNAME)), any(NormalizedNode.class));
         doReturn(map).when(this.dom).getSupportedExtensions();
-        doReturn(this.domChain).when(this.dom).createTransactionChain(any(BGPPeer.class));
+        doReturn(this.domChain).when(this.dom).createTransactionChain(any(AbstractPeer.class));
         doReturn(this.transWrite).when(this.chain).newWriteOnlyTransaction();
         doReturn(false).when(this.o).isPresent();
         doReturn(this.o).when(this.future).get();
index 3c9f100f2c10aad455fa79cf7b866375a373ff14..39bd7989d0543a74154c20b0973af80972de4d72 100644 (file)
@@ -8,9 +8,9 @@
 
 package org.opendaylight.protocol.bgp.rib.impl.config;
 
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.anyInt;
-import static org.mockito.Matchers.eq;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyInt;
+import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.doNothing;
 import static org.mockito.Mockito.doReturn;
@@ -88,6 +88,7 @@ class AbstractConfig extends DefaultRibPoliciesMockTest {
     private DOMDataTreeChangeService dataTreeChangeService;
     private final BGPPeerTracker peerTracker = new BGPPeerTrackerImpl();
 
+    @Override
     @Before
     public void setUp() throws Exception {
         super.setUp();
@@ -128,7 +129,7 @@ class AbstractConfig extends DefaultRibPoliciesMockTest {
         doReturn(new BgpId("127.0.0.1")).when(this.rib).getBgpIdentifier();
         doReturn(true).when(this.future).cancel(true);
         doReturn(this.future).when(this.dispatcher).createReconnectingClient(any(InetSocketAddress.class),
-                any(InetSocketAddress.class), anyInt(), any(KeyMapping.class));
+                any(), anyInt(), any(KeyMapping.class));
         doReturn(this.dispatcher).when(this.rib).getDispatcher();
 
         doReturn(java.util.Optional.of(new BgpTableTypeImpl(Ipv4AddressFamily.class,
index 1973f16eaf266afdde9fdff522c32098f3b92c4d..1802a0079c93d203a91a8a9b1ebb7f9b500ee8b8 100644 (file)
@@ -12,8 +12,8 @@ import static junit.framework.TestCase.fail;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.anyInt;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyInt;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 
@@ -142,7 +142,7 @@ public class BgpPeerTest extends AbstractConfig {
         this.bgpPeer.instantiateServiceInstance();
         verify(this.bgpPeerRegistry).addPeer(any(), any(), any());
         verify(this.dispatcher).createReconnectingClient(any(InetSocketAddress.class),
-                any(InetSocketAddress.class), anyInt(), any(KeyMapping.class));
+                any(), anyInt(), any(KeyMapping.class));
 
         try {
             this.bgpPeer.start(this.rib, neighbor, null, this.peerGroupLoader, this.tableTypeRegistry);
index a1d3ec179865b0c6f877979b1db49f392203dc45..cb4596f13bfff5e1dc2a1503f3a89e003d5521be 100644 (file)
@@ -54,8 +54,8 @@
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
 
         <dependency>
index 24122e85b3a228d92fd6c25399d07f20b8d37feb..2c3574f17453d0470ebec7012b1a4df19a812faf 100644 (file)
@@ -50,8 +50,8 @@
             <artifactId>bgp-openconfig-api</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index e07f1e5e3fc8618bc9db43ff1b484554dc741733..38e86c715343e050bfe4af6a8ba5e7448d32bab2 100644 (file)
@@ -77,8 +77,8 @@
             <artifactId>yang-binding</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>${project.groupId}</groupId>
index 1e02c17e3ac55d06eb40220a32e5c7f80a16910a..5b0448e1d63f926f50ef2bb99afd1f4c8a3cca38 100644 (file)
@@ -55,7 +55,7 @@ final class BGPPeerBuilder {
             }
             addFutureListener(localAddress, dispatcher.createServer(localAddress));
         }
-        LOG.debug("{} {}", sessionListener, proposal);
+        LOG.debug("Listener {} received proposal {}", sessionListener, proposal);
     }
 
     private static <T> void addFutureListener(final InetSocketAddress localAddress, final Future<T> future) {
index 025e27a8ae496757b688d18e3724e62b8ba5713b..249579cc5a26ea6d07411dffd82671276e2c2d57 100644 (file)
             <artifactId>ietf-topology-l3-unicast-igp</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>io.netty</groupId>
index 4602a4b5d39689ca03938f3c84ef37b5604222d3..dcdab13d57954fb6b8dbc01a837f8bb65dd354bf 100644 (file)
@@ -13,6 +13,7 @@ import com.google.common.base.Optional;
 import com.google.common.base.Preconditions;
 import java.util.Collections;
 import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 import java.util.concurrent.ExecutionException;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
@@ -171,11 +172,14 @@ abstract class AbstractReachabilityTopologyBuilder<T extends Route> extends Abst
         if (present.useCount == 0) {
             final IgpNodeAttributes attrs = read(trans, present.attrId);
             if (attrs != null) {
-                present.useCount = attrs.getPrefix().size();
-                if (present.useCount == 0) {
-                    trans.delete(LogicalDatastoreType.OPERATIONAL, nodeInstanceId(ni));
-                    this.nodes.remove(ni);
-                }
+                final List<Prefix> prefix = attrs.getPrefix();
+                present.useCount = prefix == null ? 0 : prefix.size();
+            } else {
+                present.useCount = 0;
+            }
+            if (present.useCount == 0) {
+                trans.delete(LogicalDatastoreType.OPERATIONAL, nodeInstanceId(ni));
+                this.nodes.remove(ni);
             }
         }
     }
index aaaa6385b6b1dd5c060119eee0b487eff52c9063..d2b0bd2daa882f63df06781a5943dbb26cbec653 100755 (executable)
@@ -10,6 +10,7 @@ package org.opendaylight.bgpcep.bgp.topology.provider;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 import static org.opendaylight.protocol.util.CheckUtil.checkNotPresentOperational;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
 
@@ -98,7 +99,7 @@ public class Ipv4ReachabilityTopologyBuilderTest extends AbstractTopologyBuilder
         wTx.delete(LogicalDatastoreType.OPERATIONAL, this.ipv4RouteIID);
         wTx.commit();
         readDataOperational(getDataBroker(), this.ipv4TopoBuilder.getInstanceIdentifier(), topology -> {
-            assertEquals(0, topology.getNode().size());
+            assertNull(topology.getNode());
             return topology;
         });
 
index a67a01c452ba186f5460782c373b6127d55d56f7..9913df8619154e5a30e7dfd76d4d2218b3645ec2 100755 (executable)
@@ -10,6 +10,7 @@ package org.opendaylight.bgpcep.bgp.topology.provider;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 import static org.opendaylight.bgpcep.bgp.topology.provider.Ipv4ReachabilityTopologyBuilderTest.PATH_ID;
 import static org.opendaylight.protocol.util.CheckUtil.checkNotPresentOperational;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
@@ -98,7 +99,7 @@ public class Ipv6ReachabilityTopologyBuilderTest extends AbstractTopologyBuilder
         wTx.delete(LogicalDatastoreType.OPERATIONAL, this.ipv6RouteIID);
         wTx.commit();
         readDataOperational(getDataBroker(), this.ipv6TopoBuilder.getInstanceIdentifier(), topology -> {
-            assertEquals(0, topology.getNode().size());
+            assertNull(topology.getNode());
             return topology;
         });
 
index 2cbb3f5a0ce0bda8b21c85f761bf351f29c3e1f2..05b0fd152c2cedac1200800406d37ec21c304cc9 100755 (executable)
@@ -216,8 +216,8 @@ public class LinkstateTopologyBuilderTest extends AbstractTopologyBuilderTest {
         wTx.delete(LogicalDatastoreType.OPERATIONAL, this.linkstateRouteIID);
         wTx.commit();
         readDataOperational(getDataBroker(), this.linkstateTopoBuilder.getInstanceIdentifier(), topology -> {
-            assertEquals(0, topology.getNode().size());
-            assertEquals(0, topology.getLink().size());
+            assertNull(topology.getNode());
+            assertNull(topology.getLink());
             return topology;
         });
     }
index 042bc86d5ac8cf5074a337cae0332b0d23606a91..e4cca90a98ff0201bf2361e221cdbb43a5e9f897 100644 (file)
@@ -39,8 +39,8 @@
         </dependency>
         <!-- Test dependencies -->
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
index 4d52f00a9acab4ce19e1f23e023f7121ea05f228..1c497a238ce8ada5efb253a5416f0dfd5de10afc 100644 (file)
@@ -14,7 +14,7 @@
     <parent>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>binding-parent</artifactId>
-        <version>0.14.0-SNAPSHOT</version>
+        <version>3.0.1</version>
         <relativePath/>
     </parent>
 
@@ -48,7 +48,7 @@
                     <dependency>
                         <groupId>org.opendaylight.yangtools</groupId>
                         <artifactId>checkstyle-logging</artifactId>
-                        <version>2.0.10</version>
+                        <version>2.1.2</version>
                     </dependency>
                 </dependencies>
             </plugin>
index 68f85a5d9c004efff2fd21a411ad132f42a00850..0640424b70af24ecb13889cf0882480df0aad09d 100644 (file)
 
     <dependencies>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
index 97267915b0c686a40b08939b18c729ce70289325..feb5d9b8f082116d4b8e47ff416ce961752dc666 100644 (file)
@@ -13,7 +13,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index ee00cf4f1fd6179946da160a49b307aa295f2f0a..e6f0419af428cf215e3c16d09531847f16507926 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index a0381744c9a54503d1c2262bf650a88f507b9c4a..0d8f535aa8395b8dd26db68dc5c382c2c87c869c 100644 (file)
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.yangtools</groupId>
index 585f51f4b21d19669d7c0b8063b1f2c56699da5c..506d2fb91e54936bf8cf037fe3085e802934233f 100644 (file)
@@ -5,7 +5,6 @@
  * 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.bmp.impl.app;
 
 import static org.junit.Assert.assertEquals;
@@ -13,10 +12,11 @@ import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
-import static org.mockito.Matchers.any;
+import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doAnswer;
 import static org.opendaylight.protocol.bmp.parser.message.TestUtil.createRouteMonMsgWithEndOfRibMarker;
 import static org.opendaylight.protocol.bmp.parser.message.TestUtil.createRouteMonitMsg;
+import static org.opendaylight.protocol.util.CheckUtil.checkNotPresentOperational;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
 
 import com.google.common.net.InetAddresses;
@@ -213,10 +213,7 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest {
         this.bmpApp.close();
         this.mappingService.close();
 
-        readDataOperational(getDataBroker(), BMP_II, monitor -> {
-            assertTrue(monitor.getMonitor().isEmpty());
-            return monitor;
-        });
+        checkNotPresentOperational(getDataBroker(), BMP_II);
     }
 
     @Test(timeout = 20000)
@@ -266,7 +263,7 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest {
         channel4.close().sync();
 
         readDataOperational(getDataBroker(), MONITOR_IID, monitor -> {
-            assertEquals(0, monitor.getRouter().size());
+            assertNull(monitor.getRouter());
             return monitor;
         });
     }
@@ -281,7 +278,7 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest {
         final RouterId routerId = getRouterId(remoteRouterIpAddr);
 
         readDataOperational(getDataBroker(), MONITOR_IID, monitor -> {
-            assertFalse(monitor.getRouter().isEmpty());
+            assertNotNull(monitor.getRouter());
             // now find the current router instance
             Router router = null;
             for (final Router r : monitor.getRouter()) {
@@ -300,7 +297,7 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest {
                 .createInitMsg("description", "name", "some info")));
 
         readDataOperational(getDataBroker(), MONITOR_IID, monitor -> {
-            assertFalse(monitor.getRouter().isEmpty());
+            assertNotNull(monitor.getRouter());
             Router retRouter = null;
             for (final Router r : monitor.getRouter()) {
                 if (routerId.equals(r.getRouterId())) {
@@ -427,8 +424,7 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest {
         waitWriteAndFlushSuccess(channel.writeAndFlush(TestUtil.createPeerDownNotification(PEER1)));
 
         readDataOperational(getDataBroker(), routerIId, router -> {
-            final List<Peer> peersAfterDown = router.getPeer();
-            assertTrue(peersAfterDown.isEmpty());
+            assertNull(router.getPeer());
             return router;
         });
 
index b64606118952d1dd3d363dceac6d7497f839b1f2..533ffce476ffcf5d62fa185147fb6b35f9d2ae4f 100644 (file)
@@ -63,8 +63,8 @@
             <artifactId>util</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index f04a8ffa36e19842b621865a98b21b02d7643c70..473535688fb6b904f91b6ac3907cec5b567d88ad 100644 (file)
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index 6fb927dfbce5598d6762443b64423112e90314d4..50f6c7d5a11d3c39f610ce6800ab4b7ee588542e 100644 (file)
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.osgi</groupId>
index 3dce6d8606b217f92b6a083e90fa3ca18ed59f37..0fc91a4edb84aecccc7030a5a7f7bcf9f35074fa 100644 (file)
@@ -16,7 +16,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 5605ef56c20f918f3d85b1e74a6e7fc1bfdcbedb..fa2fe313702c802c505fc7d1840b6f83d7386202 100644 (file)
@@ -58,8 +58,8 @@
                     discovered by the plugin.
         -->
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
 
         <!-- Testing dependencies -->
index 7cd2db29f5fa3e362f7ff425ce34fb31cc4dedb1..2f370362847df66d7e4a438f0899a88e8cfb497e 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index a023d4afe2577f4c73554c372a1fd7e375fda884..1347695f312ef50b8a5b75035e4a14122fc7bb50 100644 (file)
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
         </dependency>
+        <dependency>
+            <groupId>javax.xml.stream</groupId>
+            <artifactId>stax-api</artifactId>
+        </dependency>
+
         <!-- test scope dependencies -->
         <dependency>
             <groupId>${project.groupId}</groupId>
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-yang-types-20130715</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-yang-types</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-interfaces</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc8343</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
index 3beac3d79039ccae7d2a7530bf2638d916cc0fd6..e315895767d24cdcbb953d9c1d0267cb190bd763 100644 (file)
@@ -18,7 +18,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index cf81f1164e169554f19cd46c228f3e90b823f002..be4cf80c44365817e549de6468d98feb3257c0d3 100644 (file)
@@ -9,7 +9,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>karaf4-parent</artifactId>
-    <version>3.1.3</version>
+    <version>4.0.2</version>
     <relativePath/>
   </parent>
 
index 60d3e9b20cb2720a157baacf33bd4cb004de60f3..d88ca2ad548ef77d15706fe21179738a2dc6c0cf 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>feature-repo-parent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 8a01f22c2000be2ba04e1807d46be32e32418a1d..53f222936f5ccea2aed0ffa5e12cbf57ee03a489 100644 (file)
@@ -8,6 +8,6 @@
   -->
 <features name="odl-bgpcep-bgp-dependencies-${project.version}" xmlns="http://karaf.apache.org/xmlns/features/v1.4.0">
     <feature name="odl-bgpcep-bgp-dependencies" version="${project.version}">
-        <feature version="[2.0.10,3)">odl-yangtools-data-api</feature>
+        <feature version="[2.1,3)">odl-yangtools-data-api</feature>
     </feature>
 </features>
index adfc4b1bee581cbae5194061b3ab629a81e7a65c..95f17e13394f820a54b1183a9823c0144b5fd3ff 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index a15bb2a1260221b4b016b96748208548eb3a0202..420e271ce9c10f6f86a7ad75450a5de7e857b1bc 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index db35cf1afe04daa29592db1a657c1c90a90305e0..b85a6488449fb3269b545ceef7a760464f175ea0 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index fbe1d78d99cfbd1d384cbb3166414a8bc4a904ec..1e988c3b59eebdaca30b599467b1de3adab7644b 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 6da3f0c79bc162c8b7358571b4a26d5234163b78..abf1b4fe0ebc3f24cb4ee7b663724382feee8ba3 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 413bbdf7b5679623ea32711352f6960951d2f70c..d7187c2c048fe66ef3689d527ba39caaa28d8409 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index e2555d7046ff1d9f88d43e66d9678f63b2071532..3e7e5268322280e01d53f7528a1fe7ed0bbb5b8c 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 694d7ed9cd3fcc2d598863b4349a08741dd0c917..a98304130c859a72c3d3d5c2a5157debc1565ac8 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 193e0f86c65c8cb7e55ef17cc397a653dad53bb1..a7c744470caacca22ddf30f467afa38137233aad 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index f2ba89d71ad4ce7103c346597a97bb4a23f965f6..7efaa6f69da69a62b39dc4cf0fe797a559c825ab 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>bundle-parent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
@@ -51,7 +51,7 @@
             <dependency>
                 <groupId>org.opendaylight.yangtools</groupId>
                 <artifactId>yangtools-artifacts</artifactId>
-                <version>2.0.10</version>
+                <version>2.1.2</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
@@ -59,7 +59,7 @@
             <dependency>
                 <groupId>org.opendaylight.mdsal</groupId>
                 <artifactId>mdsal-artifacts</artifactId>
-                <version>2.6.0-SNAPSHOT</version>
+                <version>3.0.1</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
+
+            <dependency>
+                <groupId>javax.xml.stream</groupId>
+                <artifactId>stax-api</artifactId>
+                <version>1.0-2</version>
+            </dependency>
         </dependencies>
     </dependencyManagement>
 
                     <dependency>
                         <groupId>org.opendaylight.yangtools</groupId>
                         <artifactId>checkstyle-logging</artifactId>
-                        <version>2.0.10</version>
+                        <version>2.1.2</version>
                     </dependency>
                 </dependencies>
                 <configuration>
index f0f48606bef5fd63a80ff179594e7ffd1b61fa76..b07c55c940b07085edfa065bef2f64b2a70d57a7 100644 (file)
@@ -69,8 +69,8 @@
                     discovered by the plugin.
         -->
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
 
         <!-- Testing dependencies -->
index 2bccf6d8fe82b00eddd61f5a016810abf8384189..1bfa192bbbade5c2ecba0e75c72b4f9b3ade5461 100644 (file)
@@ -74,8 +74,8 @@
             <artifactId>ietf-topology</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>io.netty</groupId>
index 8c11c963b19d91c25f85597b40d47bf621a82292..3d6c70b2e3c813487f4327d6afe70b4574b8d535 100644 (file)
@@ -46,8 +46,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>io.netty</groupId>
index f1b8dfd624be764cf5b0115c788cb0d88dd975ea..61402475f98a9357b6a87c435501c37bdfc3a810 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 2ffa164d257303161267c728b856ab6400b7f022..bc96739d189f9baee3640234928283c6a07a0db2 100644 (file)
@@ -87,8 +87,8 @@
                     discovered by the plugin.
         -->
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
index 2a3ee81dbd46b0d5eec4f9be01b343040797a1c3..8e44bf25eae74a5c2cee05e6499b995026c7898f 100644 (file)
@@ -89,8 +89,8 @@
             <artifactId>slf4j-api</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.yangtools</groupId>
index fbfcbf7e05662a1858860401faf5307957d506c6..3dd305e2053ff371526b0c3f45e0e6cee63952cf 100644 (file)
@@ -90,8 +90,8 @@
             <artifactId>logback-classic</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
 
         <!-- Testing dependencies -->
index 52d07e68320e0b7c7ce920acbd24f394a8ecb813..5f6850ce548a9f1d2e9c25b6b452bf600d12c6ae 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 24ba80e098d8f9dc434605c12d25335cd4de694b..61877824b4edbee30a34ebdcbe38a87db04c93fd 100644 (file)
@@ -19,7 +19,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 45106ae7ce780c888b74bfeb93a1d2dddb7594bc..1e1dd3c62be95bbbb548262d9a1eacd87395b46d 100644 (file)
@@ -65,8 +65,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
index 9a3790a5fc00fc8d4b9aef98c2f0a09f75248ee2..ade2e36758cdb56fa8425332a39b8c04428508dc 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 7828efa490f75065b382cd9656878d741f494725..f4b664fcbb1edec4cccaff41266d532ec82669a2 100644 (file)
@@ -45,8 +45,8 @@
             <artifactId>programming-topology-api</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
index 2aa53098ec1b836875a108e9fd3ba1c4383e2d6a..a466ba4ccf54a06868b6b4a6eb027f9cd58b7f96 100644 (file)
@@ -82,8 +82,8 @@
             <artifactId>util</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal</groupId>
index c6ceb5a2c0fde8445ffbadb21bf50c78eb27321f..e4e63f1cfeb640a3bb4e80b00993fcc1f0a20988 100644 (file)
@@ -179,7 +179,7 @@ final class TopologyNodeState implements AutoCloseable, TransactionChainListener
 
             @Override
             public void onFailure(final Throwable throwable) {
-                LOG.trace("Put topology Node failed {}, value {}, {}", TopologyNodeState.this.nodeId, node, throwable);
+                LOG.trace("Put topology Node failed {}, value {}", TopologyNodeState.this.nodeId, node, throwable);
             }
         }, MoreExecutors.directExecutor());
     }
index 889cd391452d3b0d8f1b966024ac3f4d0fa96a5c..b79f817b7857779ff456e26a8e3ed4993ff5add4 100644 (file)
@@ -9,6 +9,8 @@ package org.opendaylight.bgpcep.pcep.topology.provider;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 import static org.opendaylight.protocol.pcep.pcc.mock.spi.MsgBuilderUtil.createLsp;
 import static org.opendaylight.protocol.pcep.pcc.mock.spi.MsgBuilderUtil.createPath;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
@@ -95,7 +97,9 @@ public class PCEPTriggeredReSynchronizationProcedureTest
 
         this.listener.onMessage(this.session, pcRpt);
         readDataOperational(getDataBroker(), this.pathComputationClientIId, pcc -> {
-            assertEquals(1, pcc.getReportedLsp().size());
+            final List<?> lsp = pcc.getReportedLsp();
+            assertNotNull(lsp);
+            assertEquals(1, lsp.size());
             return pcc;
         });
 
@@ -162,7 +166,7 @@ public class PCEPTriggeredReSynchronizationProcedureTest
             //check node - synchronized
             assertEquals(PccSyncState.Synchronized, pcc.getStateSync());
             //check reported LSP
-            assertEquals(0, pcc.getReportedLsp().size());
+            assertNull(pcc.getReportedLsp());
             return pcc;
         });
     }
index decf7671ca39ce013db8442bdde975db7b3064a8..7f30c3ad0b65c7b3bff435169301a141c3e48f60 100644 (file)
@@ -10,7 +10,7 @@ package org.opendaylight.bgpcep.pcep.topology.provider;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertNull;
 import static org.opendaylight.protocol.pcep.pcc.mock.spi.MsgBuilderUtil.createLsp;
 import static org.opendaylight.protocol.pcep.pcc.mock.spi.MsgBuilderUtil.createPath;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
@@ -158,7 +158,7 @@ public class StateSynchronizationAvoidanceProcedureTest extends
             //check node - synchronized
             assertEquals(PccSyncState.Synchronized, pcc.getStateSync());
             //check reported LSP is empty, LSP state from previous session was purged
-            assertTrue(pcc.getReportedLsp().isEmpty());
+            assertNull(pcc.getReportedLsp());
             return pcc;
         });
     }
index ebc8ae2911fcdda44de84315a5d81d1a6b99504f..33c7c7e8b77c74627d1785b547e53047c023074e 100755 (executable)
@@ -270,7 +270,7 @@ public class Stateful07TopologySessionListenerTest
 
         // check if lsp was removed
         readDataOperational(getDataBroker(), this.pathComputationClientIId, pcc -> {
-            assertEquals(0, pcc.getReportedLsp().size());
+            assertNull(pcc.getReportedLsp());
             return pcc;
         });
         // check stats
index 960578d367c880bb60641d4ba03e656a1e3425e8..c6dff0c6965c6763e91691040ab6ecd53b355478 100644 (file)
@@ -7,7 +7,7 @@
  */
 package org.opendaylight.bgpcep.pcep.topology.provider;
 
-import static org.mockito.Matchers.any;
+import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.doNothing;
 import static org.mockito.Mockito.doReturn;
@@ -38,18 +38,14 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.programming.rev171025.SubmitTriggerSyncInputBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.programming.rev171025.SubmitUpdateLspInputBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.AddLspArgs;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.AddLspInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.AddLspOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.EnsureLspOperationalInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.EnsureLspOperationalOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.RemoveLspArgs;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.RemoveLspInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.RemoveLspOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.TriggerSyncArgs;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.TriggerSyncInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.TriggerSyncOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.UpdateLspArgs;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.UpdateLspInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.UpdateLspOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev171025.ensure.lsp.operational.args.ArgumentsBuilder;
 import org.opendaylight.yangtools.yang.common.RpcResult;
@@ -124,19 +120,19 @@ public class TopologyProgrammingTest extends AbstractPCEPSessionTest<MockedTopol
         doAnswer(invocation -> {
             TopologyProgrammingTest.this.addLspArgs = (AddLspArgs) invocation.getArguments()[0];
             return TopologyProgrammingTest.this.futureAddLspOutput;
-        }).when(listener).addLsp(any(AddLspInput.class));
+        }).when(listener).addLsp(any());
         doAnswer(invocation -> {
             TopologyProgrammingTest.this.updateLspArgs = (UpdateLspArgs) invocation.getArguments()[0];
             return TopologyProgrammingTest.this.futureUpdateLspOutput;
-        }).when(listener).updateLsp(any(UpdateLspInput.class));
+        }).when(listener).updateLsp(any());
         doAnswer(invocation -> {
             TopologyProgrammingTest.this.removeLspArgs = (RemoveLspArgs) invocation.getArguments()[0];
             return TopologyProgrammingTest.this.futureRemoveLspOutput;
-        }).when(listener).removeLsp(any(RemoveLspInput.class));
+        }).when(listener).removeLsp(any());
         doAnswer(invocation -> {
             TopologyProgrammingTest.this.triggerSyncArgs = (TriggerSyncArgs) invocation.getArguments()[0];
             return TopologyProgrammingTest.this.futureTriggerSyncOutput;
-        }).when(listener).triggerSync(any(TriggerSyncInput.class));
+        }).when(listener).triggerSync(any());
         doAnswer(invocation -> {
             TopologyProgrammingTest.this.ensureLspInput = (EnsureLspOperationalInput) invocation.getArguments()[0];
             return TopologyProgrammingTest.this.futureEnsureLspOutput;
index b9b09a5e31005ea4cc493e2f71800974fb15ac61..fb805e6b9922c0f089d587b647e845bb2831a1b3 100644 (file)
@@ -36,11 +36,12 @@ public abstract class AbstractInstructionExecutor implements FutureCallback<Inst
     }
 
     public static FailureCase schedule(final InstructionScheduler scheduler, final AbstractInstructionExecutor fwd) {
+        final SubmitInstructionInput input = fwd.getInput();
         final ListenableFuture<Instruction> listenableFuture;
         try {
-            listenableFuture = scheduler.scheduleInstruction(fwd.getInput());
+            listenableFuture = scheduler.scheduleInstruction(input);
         } catch (final SchedulerException e) {
-            LOG.info("Instuction {} failed to schedule", e.getMessage(), e);
+            LOG.info("Instuction {} failed to schedule", input, e);
             return new FailureCaseBuilder().setFailure(e.getFailure()).build();
         }
         Futures.addCallback(listenableFuture, fwd, MoreExecutors.directExecutor());
index d667aaa4772eedcbd598a9fdb8b71ea06f92ec21..966e99d6023f36eb061fa1b4d5ac5e88d77babe0 100644 (file)
@@ -62,7 +62,7 @@ public final class TopologyStatsProviderImpl implements TransactionChainListener
     }
 
     public synchronized void init() {
-        LOG.info("Initializing TopologyStatsProvider service.", this);
+        LOG.info("Initializing TopologyStatsProvider service.");
         this.transactionChain = this.dataBroker.createTransactionChain(this);
         final TimerTask task = new TimerTask() {
             @Override
@@ -89,12 +89,12 @@ public final class TopologyStatsProviderImpl implements TransactionChainListener
             }
             tx.commit().addCallback(new FutureCallback<CommitInfo>() {
                 @Override
-                public void onSuccess(CommitInfo result) {
+                public void onSuccess(final CommitInfo result) {
                     LOG.debug("Successfully committed Topology stats update");
                 }
 
                 @Override
-                public void onFailure(Throwable ex) {
+                public void onFailure(final Throwable ex) {
                     LOG.error("Failed to commit Topology stats update", ex);
                 }
             }, MoreExecutors.directExecutor());
@@ -107,7 +107,7 @@ public final class TopologyStatsProviderImpl implements TransactionChainListener
     @Override
     public synchronized void close() throws Exception {
         if (closed.compareAndSet(false, true)) {
-            LOG.info("Closing TopologyStatsProvider service.", this);
+            LOG.info("Closing TopologyStatsProvider service.");
             this.scheduleTask.cancel(true);
             final WriteTransaction wTx = this.transactionChain.newWriteOnlyTransaction();
             for (final KeyedInstanceIdentifier<Node, NodeKey> statId : this.statsMap.keySet()) {
index 890220fba56672d0ad7c093fee64a46a377e8039..3ef5a3e778591bc0033af1c4eef54263179bc174 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index e648b17eee20c12036d9a76f52062ddf3d1fb464..b6162182e05c26114094a7ca2dfd87e1d5630c74 100644 (file)
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
             <groupId>org.opendaylight.mdsal.model</groupId>
             <artifactId>ietf-topology-l3-unicast-igp</artifactId>
         </dependency>
-        <dependency>
-            <groupId>${project.groupId}</groupId>
-            <artifactId>pcep-topology-provider</artifactId>
-        </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
index 178b11712417cf7fdee4e110929df691cfeb4cd1..92d00283b94a762808c8420e48d673cd524ea957 100644 (file)
@@ -11,6 +11,7 @@ import static java.util.Objects.requireNonNull;
 
 import com.google.common.base.Optional;
 import com.google.common.base.Preconditions;
+import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Lists;
 import com.google.common.util.concurrent.FutureCallback;
 import com.google.common.util.concurrent.MoreExecutors;
@@ -109,7 +110,7 @@ public final class NodeChangedListener implements ClusteredDataTreeChangeListene
     private static void enumerateLsps(final InstanceIdentifier<Node> id, final Node node,
             final Set<InstanceIdentifier<ReportedLsp>> lsps) {
         if (node == null) {
-            LOG.trace("Skipping null node", id);
+            LOG.trace("Skipping null node {}", id);
             return;
         }
         final Node1 pccnode = node.augmentation(Node1.class);
@@ -326,7 +327,7 @@ public final class NodeChangedListener implements ClusteredDataTreeChangeListene
         boolean orphDstNode = true;
         boolean orphDstTp = true;
         boolean orphSrcTp = true;
-        for (final Link lw : topology.getLink()) {
+        for (final Link lw : nonNullList(topology.getLink())) {
             LOG.trace("Checking link {}", lw);
 
             final NodeId sn = lw.getSource().getSourceNode();
@@ -513,4 +514,8 @@ public final class NodeChangedListener implements ClusteredDataTreeChangeListene
     DataBroker getDataProvider() {
         return dataProvider;
     }
+
+    private static <T> List<T> nonNullList(final List<T> nullable) {
+        return nullable != null ? nullable : ImmutableList.of();
+    }
 }
index 633e369ac6cd4a22c6a558db0d1b0be4597f1f41..c5bf3314999781e66af066e2c1d8255e75b2cb4e 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.bgpcep.pcep.tunnel.provider;
 import java.util.ArrayList;
 import java.util.concurrent.ExecutionException;
 import javax.annotation.concurrent.GuardedBy;
-import org.opendaylight.bgpcep.pcep.topology.provider.config.PCEPTopologyDeployerImpl;
 import org.opendaylight.bgpcep.topology.DefaultTopologyReference;
 import org.opendaylight.bgpcep.topology.TopologyReference;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
@@ -32,7 +31,7 @@ import org.slf4j.LoggerFactory;
 
 public final class PCEPTunnelTopologyProvider extends DefaultTopologyReference implements AutoCloseable {
 
-    private static final Logger LOG = LoggerFactory.getLogger(PCEPTopologyDeployerImpl.class);
+    private static final Logger LOG = LoggerFactory.getLogger(PCEPTunnelTopologyProvider.class);
 
     private final NodeChangedListener ncl;
     private final InstanceIdentifier<Node> src;
index 3ed8af34c05d25d44fea08de61313c2923bb9d1c..cae2414f841505d1bfad64cb162536bf8cbbaebf 100644 (file)
@@ -18,7 +18,6 @@ import java.util.stream.Collectors;
 import javax.annotation.Nonnull;
 import javax.annotation.concurrent.GuardedBy;
 import org.apache.commons.lang3.StringUtils;
-import org.opendaylight.bgpcep.pcep.topology.provider.config.PCEPTopologyDeployerImpl;
 import org.opendaylight.controller.md.sal.binding.api.ClusteredDataTreeChangeListener;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.DataObjectModification;
@@ -41,7 +40,7 @@ import org.slf4j.LoggerFactory;
 
 public final class TunnelProviderDeployer implements ClusteredDataTreeChangeListener<Topology>, AutoCloseable {
 
-    private static final Logger LOG = LoggerFactory.getLogger(PCEPTopologyDeployerImpl.class);
+    private static final Logger LOG = LoggerFactory.getLogger(TunnelProviderDeployer.class);
 
     private static final long TIMEOUT_NS = TimeUnit.SECONDS.toNanos(5);
     private final TunnelProviderDependencies dependencies;
index 4f9a6bc27ef487a90a27f4742dfda8d4e2d70937..1cf4fab69480b2fab44bfea600817713913556eb 100644 (file)
@@ -10,6 +10,7 @@ package org.opendaylight.bgpcep.pcep.tunnel.provider;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 import static org.opendaylight.protocol.util.CheckUtil.readDataOperational;
 
 import com.google.common.collect.Lists;
@@ -187,8 +188,8 @@ public class NodeChangedListenerTest extends AbstractConcurrentDataBrokerTest {
         removeNode(NODE1_ID);
         removeNode(NODE2_ID);
         readDataOperational(getDataBroker(), TUNNEL_TOPO_IID, removedNodeTopo -> {
-            assertEquals(0, removedNodeTopo.getNode().size());
-            assertEquals(0, removedNodeTopo.getLink().size());
+            assertNull(removedNodeTopo.getNode());
+            assertNull(removedNodeTopo.getLink());
             return removedNodeTopo;
         });
     }
diff --git a/pom.xml b/pom.xml
index b81589b947492b14a3b790caf689a1ab786c2d8e..204ba6e37237f11d6b0c65edbacfa3a10039e94d 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -19,7 +19,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 1e74a5b415048b5a6b3b265726692c649413bc6d..f8fc121e8eaef9747d50d60624294ccf19a55529 100644 (file)
@@ -38,8 +38,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
index a254435e224668c04afa0e00bfee7392df3f2c1e..1c8d6acaab74d1976a9b41c6ae5ec54e214198f2 100644 (file)
@@ -20,7 +20,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index f989462ae40206a59386bdcec4c21e39c6a14718..c3a5c2ac4424b9cbf8175360e63a390fad092ead 100644 (file)
@@ -14,7 +14,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 8b465daa025ed62be0fe98c88d47df3fdad132fc..422ceb2427807afbafd23a86c575bc0b095c9c43 100644 (file)
@@ -42,8 +42,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
index 467a3583db1f6eda61b3bc885fd9776b77d3119e..7f1ca122020c07f59984e8482b3d0b5f21243bee 100644 (file)
@@ -50,8 +50,8 @@
             <artifactId>slf4j-api</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>io.netty</groupId>
index b066a708ea1fc7a29da37f0a02310a6ac080e9c9..046db1f55c894ba9f87b2ccb1785f43999ec61f8 100644 (file)
@@ -20,7 +20,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index ac2c7182dc2777bdc5b092e811764a42894590b6..59ff4c7b47df18b6ff337dfa515b93bcc5856fc3 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 1dbe55f850fdacfdc774cae90f5c80488c004d7f..a5f78c111b88be2a5e513f790df824a764e5d4b6 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 7ad25d813c4af4de850cc5a744e63c41f9bb0894..2f1fb6dc18ae8db01544196ca28ed343aa42b39b 100644 (file)
@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
@@ -52,7 +52,7 @@
                     <dependency>
                         <groupId>org.opendaylight.yangtools</groupId>
                         <artifactId>checkstyle-logging</artifactId>
-                        <version>2.0.10</version>
+                        <version>2.1.2</version>
                     </dependency>
                 </dependencies>
             </plugin>
index df37034c3aedae7c20f09efa535ef40f824b76a1..8edd0a147c01ecc21e778aa175de8cef30155310 100644 (file)
@@ -71,7 +71,7 @@ public final class CheckUtil {
             throws ReadFailedException {
         AssertionError lastError = null;
         final Stopwatch sw = Stopwatch.createStarted();
-        while (sw.elapsed(TimeUnit.SECONDS) <= timeout) {
+        do {
             try (ReadOnlyTransaction tx = dataBroker.newReadOnlyTransaction()) {
                 final Optional<T> data = tx.read(ldt, iid).checkedGet();
                 if (data.isPresent()) {
@@ -83,7 +83,7 @@ public final class CheckUtil {
                     }
                 }
             }
-        }
+        } while (sw.elapsed(TimeUnit.SECONDS) <= timeout);
         throw lastError;
     }
 
@@ -172,4 +172,4 @@ public final class CheckUtil {
     public interface CheckEquals {
         void check() throws ExecutionException, InterruptedException;
     }
-}
\ No newline at end of file
+}
index 617407efe86208855a0c21130e8222616d5bc26f..d6e1dd2f326a36b959c058375079e3ab6cc26ec5 100644 (file)
@@ -9,7 +9,7 @@ package org.opendaylight.protocol.util;
 
 import static junit.framework.TestCase.assertNull;
 import static org.junit.Assert.assertNotNull;
-import static org.mockito.Matchers.any;
+import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.when;
@@ -71,7 +71,7 @@ public class CheckUtilTest extends AbstractConcurrentDataBrokerTest {
     public void testWaitFutureSuccess() {
         when(this.future.isSuccess()).thenReturn(true);
         doAnswer(invocation -> {
-            invocation.getArgumentAt(0, GenericFutureListener.class).operationComplete(CheckUtilTest.this.future);
+            invocation.<GenericFutureListener>getArgument(0).operationComplete(CheckUtilTest.this.future);
             return CheckUtilTest.this.future;
         }).when(this.future).addListener(any());
         waitFutureSuccess(this.future);
index 87b4986ec2ae7c22e146c7ff1d94fe821651126e..f754b3df38819f3202fafd60a4d3e47d0e354f52 100644 (file)
@@ -19,7 +19,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index abca32838225a7ff13d26295b8176549fe02d331..2ddfdb34bb476aef587c09cb54761e4d5163c590 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>3.1.3</version>
+        <version>4.0.2</version>
         <relativePath/>
     </parent>
 
index 17beb554f41d664927984481301c9606c66a2d7c..7de8cf372fe3f05289f48462493a7f79a4e10e27 100644 (file)
@@ -38,8 +38,8 @@
             <artifactId>concepts</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <dependency>
             <groupId>org.opendaylight.mdsal.model</groupId>
index a4f64ef9e2a778a6c893a4e175dbfeb92f57ed17..8c86d5b5068049e7e52a766379cadc7152382131 100644 (file)
@@ -48,8 +48,8 @@
             <artifactId>netty-buffer</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.opendaylight.mdsal.model</groupId>
-            <artifactId>ietf-inet-types-2013-07-15</artifactId>
+            <groupId>org.opendaylight.mdsal.binding.model.ietf</groupId>
+            <artifactId>rfc6991-ietf-inet-types</artifactId>
         </dependency>
         <!-- Testing dependencies -->
         <dependency>