Magnesium platform upgrade 01/84901/7
authorLori Jakab <lojakab@cisco.com>
Wed, 9 Oct 2019 13:12:26 +0000 (15:12 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 11 Oct 2019 14:21:06 +0000 (16:21 +0200)
This patch prepares the lispflowmapping project for Magnesium: update
odlparent to 6.0.0, mdsal to 5.0.3, and deal with the fallout,
including, but not limited to, checkstyle and spotbugs being on by
default, the uint changes in yangtools, etc.

Change-Id: Id1b3bf0201755ef29678e17e8c60fa9adbf253a6
Signed-off-by: Lori Jakab <lojakab@cisco.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
50 files changed:
artifacts/pom.xml
commons/parent/pom.xml
commons/unittest_tools/pom.xml
features/features-lispflowmapping/pom.xml
features/odl-lispflowmapping-inmemorydb/pom.xml
features/odl-lispflowmapping-mappingservice-shell/pom.xml
features/odl-lispflowmapping-mappingservice/pom.xml
features/odl-lispflowmapping-models/pom.xml
features/odl-lispflowmapping-msmr/pom.xml
features/odl-lispflowmapping-neutron/pom.xml
features/odl-lispflowmapping-southbound/pom.xml
features/odl-lispflowmapping-southbound/src/main/feature/feature.xml
features/pom.xml
integrationtest/src/test/java/org/opendaylight/lispflowmapping/integrationtest/MultiSiteScenario.java
lispflowmapping-karaf/pom.xml
mappingservice/api/pom.xml
mappingservice/config/src/main/java/org/opendaylight/lispflowmapping/config/ConfigIni.java
mappingservice/dsbackend/pom.xml
mappingservice/dsbackend/src/main/java/org/opendaylight/lispflowmapping/dsbackend/DataStoreBackEnd.java
mappingservice/dsbackend/src/main/java/org/opendaylight/lispflowmapping/dsbackend/InstanceIdentifierUtil.java
mappingservice/implementation/pom.xml
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/MappingSystem.java
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/lisp/MapResolver.java
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/util/MappingMergeUtil.java
mappingservice/implementation/src/test/java/org/opendaylight/lispflowmapping/implementation/util/MappingMergeUtilTest.java
mappingservice/inmemorydb/pom.xml
mappingservice/inmemorydb/src/main/java/org/opendaylight/lispflowmapping/inmemorydb/HashMapDb.java
mappingservice/lisp-proto/pom.xml
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/serializer/LocatorRecordSerializer.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/serializer/address/ApplicationDataSerializer.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/serializer/address/ServicePathSerializer.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressStringifier.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressUtil.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/util/MaskUtil.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/util/NumberUtil.java
mappingservice/lisp-proto/src/test/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressUtilTest.java
mappingservice/lisp-proto/src/test/java/org/opendaylight/lispflowmapping/lisp/util/MaskUtilTest.java
mappingservice/mapcache/src/main/java/org/opendaylight/lispflowmapping/mapcache/AuthKeyDb.java
mappingservice/mapcache/src/main/java/org/opendaylight/lispflowmapping/mapcache/MultiTableMapCache.java
mappingservice/mapcache/src/main/java/org/opendaylight/lispflowmapping/mapcache/SimpleMapCache.java
mappingservice/neutron/pom.xml
mappingservice/neutron/src/main/java/org/opendaylight/lispflowmapping/neutron/intenthandler/listener/VppEndpointListener.java
mappingservice/neutron/src/test/java/org/opendaylight/lispflowmapping/neutron/PortDataProcessorTest.java
mappingservice/pom.xml
mappingservice/shell/src/main/java/org/opendaylight/lispflowmapping/shell/LispAddKey.java
mappingservice/shell/src/main/java/org/opendaylight/lispflowmapping/shell/LispKeys.java
mappingservice/shell/src/main/java/org/opendaylight/lispflowmapping/shell/LispMappings.java
mappingservice/southbound/pom.xml
mappingservice/southbound/src/main/java/org/opendaylight/lispflowmapping/southbound/LispSouthboundPlugin.java
mappingservice/southbound/src/main/java/org/opendaylight/lispflowmapping/southbound/lisp/LispSouthboundHandler.java

index 642c16215eb414042535541ab318e8bcc7e297f5..1d05fa69d1b7ed8b8e52550c77b31daf796a921b 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index 3ad968dcda35127a2a04486bfad333c6805b5e3f..2e219554ac7e9a6acdff4e11e99cce99aa87a40e 100644 (file)
@@ -15,7 +15,7 @@
          for plugin configuration etc. -->
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
@@ -50,7 +50,7 @@
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>4.0.6</version>
+        <version>5.0.3</version>
         <type>pom</type>
        <scope>import</scope>
       </dependency>
index 2fa3b560efb5ee5a3e3b42e753dd539ef5482f44..032f1d4591f034170b96ac853a11845560e98ac8 100644 (file)
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <properties>
+    <odlparent.checkstyle.enforce>false</odlparent.checkstyle.enforce>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
+  </properties>
+
   <dependencies>
     <dependency>
       <groupId>junit</groupId>
index 0240744cfdae65053b4e837b18884469f41d0488..961637b4b5714fbb66beec3c19875ac7bb23bdcc 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>feature-repo-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index 2a99fccdb4e8a0da10e187984fa3236bb93218d5..38a54c4997cd2a4d933dd382f087221b5494baeb 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index 496717f0199672c7fed7ec70e7870dc2e147c821..7190de2eb2480b47def99a0b3d9bcf12dc208860 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index 8461706bb2c01fecc92f6e305c47f90807baf19c..6a48d2a24c4ed2d74d13a1d3f7f8b56e73522225 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index 539a7f1762c5efbf11fd2c95e43c61f16f149e74..6ca9b4e063c42465b2c06e51663633ed9c349271 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
@@ -29,7 +29,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>4.0.6</version>
+        <version>5.0.3</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
index f7b64a0b85de194668fb796e492b04995f962471..70bef791eac8e6073780ccc79a9c1affda8b36df 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index cfe207ecd6596fd441eb05ecdcefd64b8db1f968..0607af7bd2b07761760d73c483287ac70d273d85 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
@@ -29,7 +29,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>4.0.6</version>
+        <version>5.0.3</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
index cfbcacd236ee11bdf8ed584cfcc3143e0c7240b6..42fd315a05e9634f95220e92d362249c4277d4ee 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index bbe014fca1629737e3e560d898cd44e34a7dfde3..26fdada85feb5330d19aaea6cc652ed1069c2664 100644 (file)
@@ -8,7 +8,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
 -->
 <features xmlns="http://karaf.apache.org/xmlns/features/v1.4.0" name="odl-lispflowmapping-southbound">
     <feature name="odl-lispflowmapping-southbound">
-        <feature version="[5,6)">odl-netty-4</feature>
-        <feature version="[5,6)">odl-guava</feature>
+        <feature version="[6,7)">odl-netty-4</feature>
+        <feature version="[6,7)">odl-guava</feature>
     </feature>
 </features>
index d992085e0e59db06187c5c6a3f9a701ae3fc8276..4fce222ce02671884150d0fc33e09bc69b0dff12 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index b0afc02717eaac76ec04c3070aeeef962456576d..e7c4a79c98df7e47e0de5e7c1fc1e67db83e9c4c 100644 (file)
@@ -14,6 +14,7 @@ import static org.junit.Assert.fail;
 import static org.opendaylight.lispflowmapping.integrationtest.MappingServiceIntegrationTest.ourAddress;
 import static org.opendaylight.lispflowmapping.integrationtest.MultiSiteScenarioUtil.SITE_A;
 import static org.opendaylight.lispflowmapping.integrationtest.MultiSiteScenarioUtil.SITE_D5;
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
 
 import com.google.common.net.InetAddresses;
 import java.nio.ByteBuffer;
@@ -78,7 +79,7 @@ class MultiSiteScenario {
     private final Boolean DEFAULT_ROUTED = true;
     private final byte[] DEFAULT_SITE_ID = new byte[]{0, 0, 0, 0, 0, 0, 0, 0};
 
-    private final MappingAuthkey NULL_AUTH_KEY = new MappingAuthkeyBuilder().setKeyType(0).build();
+    private final MappingAuthkey NULL_AUTH_KEY = new MappingAuthkeyBuilder().setKeyType(fromJava(0)).build();
     private final IMappingService mapService;
     private final IFlowMapping lms;
 
@@ -261,10 +262,10 @@ class MultiSiteScenario {
         final LocatorRecordBuilder locatorRecordBuilder = new LocatorRecordBuilder();
         locatorRecordBuilder.setRloc(rloc);
         locatorRecordBuilder.setLocatorId(rlocStr);
-        locatorRecordBuilder.setPriority(priority);
-        locatorRecordBuilder.setWeight(weight);
-        locatorRecordBuilder.setMulticastPriority(DEFAULT_MULTICAST_PRIORITY);
-        locatorRecordBuilder.setMulticastWeight(DEFAULT_MULTICAST_WEIGHT);
+        locatorRecordBuilder.setPriority(fromJava(priority));
+        locatorRecordBuilder.setWeight(fromJava(weight));
+        locatorRecordBuilder.setMulticastPriority(fromJava(DEFAULT_MULTICAST_PRIORITY));
+        locatorRecordBuilder.setMulticastWeight(fromJava(DEFAULT_MULTICAST_WEIGHT));
         locatorRecordBuilder.setLocalLocator(DEFAULT_LOCAL_LOCATOR);
         locatorRecordBuilder.setRlocProbed(DEFAULT_RLOC_PROBED);
         locatorRecordBuilder.setRouted(DEFAULT_ROUTED);
@@ -345,14 +346,15 @@ class MultiSiteScenario {
                         return true;
                     }
 
-                    final boolean isWeightEquals = expectedTargetSite.getWeight() == locatorRecord.getWeight();
+                    final boolean isWeightEquals = expectedTargetSite.getWeight() == locatorRecord.getWeight().toJava();
                     if (isPossibleAssertPingResultImmediately(expectedPingWorks, isWeightEquals, "Weight isn't equal." +
                             "Expected value " + expectedTargetSite.getWeight() + ". Value from mapping" +
                             locatorRecord.getWeight() + ".")) {
                         return true;
                     }
 
-                    final boolean isPriorityEquals = expectedTargetSite.getPriority() == locatorRecord.getPriority();
+                    final boolean isPriorityEquals =
+                            expectedTargetSite.getPriority() == locatorRecord.getPriority().toJava();
                     if (isPossibleAssertPingResultImmediately(expectedPingWorks, isPriorityEquals, "Priority isn't " +
                             "equal. Expected value " + expectedTargetSite.getPriority() + ". Value from mapping" +
                             locatorRecord.getPriority() + ".")) {
@@ -449,7 +451,7 @@ class MultiSiteScenario {
         final Set<Eid> eids = new HashSet<>();
         for (String host : hosts) {
             eids.add(LispAddressUtil.asIpv4PrefixBinaryEid(host + "/" + IP_MASK,
-                    new InstanceIdType(MultiSiteScenarioUtil.VNI2)));
+                    new InstanceIdType(fromJava(MultiSiteScenarioUtil.VNI2))));
         }
         return eids;
     }
index c7dc4bd04c73bc9a3017b3d2f62b1d06f81b93a1..5a931132aeeef95c6cf8f9729d5466323dfad0de 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>karaf4-parent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
index 8a8f24e5f59af2aef5717a6bb45589dc7a9123e2..b8361f600a6d875a4dbdc8103a7a045ce3c39753 100644 (file)
@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.opendaylight.mdsal</groupId>
     <artifactId>binding-parent</artifactId>
-    <version>4.0.6</version>
+    <version>5.0.3</version>
     <relativePath/>
   </parent>
 
@@ -19,6 +19,7 @@
 
   <properties>
     <pmd.cpd.fail>true</pmd.cpd.fail>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
   </properties>
 
   <dependencies>
       <artifactId>rfc6991-ietf-inet-types</artifactId>
     </dependency>
   </dependencies>
-
-  <build>
-    <pluginManagement>
-      <plugins>
-        <plugin>
-          <!-- Enforce odlparent checkstyle -->
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-checkstyle-plugin</artifactId>
-          <configuration>
-            <includeTestSourceDirectory>true</includeTestSourceDirectory>
-            <propertyExpansion>checkstyle.violationSeverity=error</propertyExpansion>
-          </configuration>
-        </plugin>
-      </plugins>
-    </pluginManagement>
-  </build>
 </project>
index bb9488da74ca1e43901fcd3ee1c25a8ab833a10e..1e87312af741ab12a2735050d3e89819529495eb 100644 (file)
@@ -17,7 +17,7 @@ import org.slf4j.LoggerFactory;
 
 public final class ConfigIni {
 
-    protected static final Logger LOG = LoggerFactory.getLogger(ConfigIni.class);
+    private static final Logger LOG = LoggerFactory.getLogger(ConfigIni.class);
     private boolean mappingMerge;
     private boolean smr;
     private String elpPolicy;
@@ -269,7 +269,7 @@ public final class ConfigIni {
         }
 
         try {
-            this.smrRetryCount = Integer.valueOf(str);
+            this.smrRetryCount = Integer.parseInt(str);
             LOG.debug("Setting configuration variable '{}' to '{}'", LISP_SMR_RETRY_COUNT, smrRetryCount);
         } catch (NumberFormatException e) {
             LOG.debug("Configuration variable '{}' was not set correctly. SMR retry count "
@@ -297,7 +297,7 @@ public final class ConfigIni {
         }
 
         try {
-            this.smrTimeout = Long.valueOf(str);
+            this.smrTimeout = Long.parseLong(str);
             LOG.debug("Setting configuration variable '{}' to '{}'", LISP_SMR_TIMEOUT, smrTimeout);
         } catch (NumberFormatException e) {
             LOG.debug("Configuration variable '{}' was not set correctly. SMR timeout "
@@ -351,7 +351,7 @@ public final class ConfigIni {
         }
 
         try {
-            this.negativeMappingTTL = Integer.valueOf(str);
+            this.negativeMappingTTL = Integer.parseInt(str);
             LOG.debug("Setting configuration variable '{}' to '{}'", LISP_NEGATIVE_MAPPING_TTL, negativeMappingTTL);
         } catch (NumberFormatException e) {
             LOG.debug("Configuration variable '{}' was not set correctly. Negative TTL "
index 47d0474cb6d860e87d9ccdfcd7056b7a56fb8d87..e5e30658e5b9574d79d75831b995812c32e95cf2 100644 (file)
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <properties>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
+  </properties>
+
   <dependencies>
     <dependency>
       <groupId>${project.groupId}</groupId>
index 52aa77b816efc3696496d3cd15cb4ff13eddff6c..2ce6aafdf3c1ce86d4697f3185244ff8c48fbd0c 100644 (file)
@@ -45,7 +45,7 @@ import org.slf4j.LoggerFactory;
  *
  */
 public class DataStoreBackEnd implements TransactionChainListener {
-    protected static final Logger LOG = LoggerFactory.getLogger(DataStoreBackEnd.class);
+    private static final Logger LOG = LoggerFactory.getLogger(DataStoreBackEnd.class);
     private static final InstanceIdentifier<MappingDatabase> DATABASE_ROOT =
             InstanceIdentifier.create(MappingDatabase.class);
     private static final InstanceIdentifier<LastUpdated> LAST_UPDATED =
index 30a781efee80ad0682bcd697540c77de335c9558..cc84cb358db9aedfa143c49d6ea0a3eeb08ef55c 100644 (file)
@@ -77,9 +77,9 @@ public final class InstanceIdentifierUtil {
     private static long getLispInstanceId(Eid eid) {
         Address address = eid.getAddress();
         if (address instanceof InstanceId) {
-            return ((InstanceId) address).getInstanceId().getIid().getValue();
+            return ((InstanceId) address).getInstanceId().getIid().getValue().toJava();
         } else if (eid.getVirtualNetworkId() != null) {
-            return eid.getVirtualNetworkId().getValue();
+            return eid.getVirtualNetworkId().getValue().toJava();
         }
         return 0L;
     }
index 06efeb04f0d83b51a434ac44cb3f8987efc6cb77..ca60e60725dc43c3bc8b91c2c3c88a0db08cbcae 100644 (file)
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <properties>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
+  </properties>
+
   <dependencies>
     <dependency>
       <groupId>org.osgi</groupId>
index 8ce1e77c2018c6b1886fd575548e93ee85bb1562..1c9799d7cc2bdfa7ab979bd44cb97d599e80b2fb 100644 (file)
@@ -367,7 +367,7 @@ public class MappingSystem implements IMappingSystem {
         }
 
         LocatorRecord locatorRecord = mappingData.getRecord().getLocatorRecord().get(0);
-        long serviceIndex = ((ServicePath) eid.getAddress()).getServicePath().getServiceIndex();
+        long serviceIndex = ((ServicePath) eid.getAddress()).getServicePath().getServiceIndex().toJava();
         int index = LispAddressUtil.STARTING_SERVICE_INDEX - (int) serviceIndex;
         Rloc rloc = locatorRecord.getRloc();
         if (rloc.getAddress() instanceof Ipv4 || rloc.getAddress() instanceof Ipv6) {
@@ -742,13 +742,13 @@ public class MappingSystem implements IMappingSystem {
     private static Eid getVirtualParent(Eid eid) {
         if (eid.getAddress() instanceof Ipv4PrefixBinary) {
             Ipv4PrefixBinary prefix = (Ipv4PrefixBinary) eid.getAddress();
-            short parentPrefixLength = (short) (prefix.getIpv4MaskLength() - 1);
+            short parentPrefixLength = (short) (prefix.getIpv4MaskLength().toJava() - 1);
             byte[] parentPrefix = MaskUtil.normalizeByteArray(prefix.getIpv4AddressBinary().getValue(),
                     parentPrefixLength);
             return LispAddressUtil.asIpv4PrefixBinaryEid(eid, parentPrefix, parentPrefixLength);
         } else if (eid.getAddress() instanceof Ipv6PrefixBinary) {
             Ipv6PrefixBinary prefix = (Ipv6PrefixBinary) eid.getAddress();
-            short parentPrefixLength = (short) (prefix.getIpv6MaskLength() - 1);
+            short parentPrefixLength = (short) (prefix.getIpv6MaskLength().toJava() - 1);
             byte[] parentPrefix = MaskUtil.normalizeByteArray(prefix.getIpv6AddressBinary().getValue(),
                     parentPrefixLength);
             return LispAddressUtil.asIpv6PrefixBinaryEid(eid, parentPrefix, parentPrefixLength);
index 98cea8f77e28e5bb3824690fa5a90cac61d17ba4..9d410247f17b9406e4d672ee27a7ba1e69f47fa3 100644 (file)
@@ -8,6 +8,8 @@
 
 package org.opendaylight.lispflowmapping.implementation.lisp;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import com.google.common.base.Preconditions;
 import com.google.common.collect.Lists;
 import java.util.ArrayList;
@@ -273,7 +275,7 @@ public class MapResolver implements IMapResolverAsync {
                 ExplicitLocatorPath teAddress = ((ExplicitLocatorPath) container.getAddress());
                 SimpleAddress nextHop = getNextELPHop(teAddress, itrRlocs);
                 if (nextHop != null) {
-                    java.lang.Short priority = record.getPriority();
+                    java.lang.Short priority = record.getPriority().toJava();
                     if (elpPolicy.equalsIgnoreCase("both")) {
                         recordBuilder.getLocatorRecord().add(
                                 new LocatorRecordBuilder().setLocalLocator(record.isLocalLocator())
@@ -294,7 +296,7 @@ public class MapResolver implements IMapResolverAsync {
                     recordBuilder.getLocatorRecord().add(
                             new LocatorRecordBuilder().setLocalLocator(record.isLocalLocator())
                                     .setRlocProbed(record.isRlocProbed()).setWeight(record.getWeight())
-                                    .setPriority(priority).setMulticastWeight(record.getMulticastWeight())
+                                    .setPriority(fromJava(priority)).setMulticastWeight(record.getMulticastWeight())
                                     .setMulticastPriority(record.getMulticastPriority()).setRouted(record.isRouted())
                                     .setRloc(LispAddressUtil.toRloc(nextHop))
                                     .setLocatorId(record.getLocatorId()).build());
index 75a59edfa98bf7a72a8c8f64822b50ed74872762..540d08a5500c47acbaf407f3a08ce3898500b4bf 100644 (file)
@@ -7,6 +7,8 @@
  */
 package org.opendaylight.lispflowmapping.implementation.util;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import com.google.common.base.Preconditions;
 import java.util.ArrayList;
 import java.util.Date;
@@ -302,11 +304,11 @@ public final class MappingMergeUtil {
             if (sbLocatorMap.containsKey(nbRloc)) {
                 // common locator found. use the NB record as the common locator.
 
-                if (sbLocatorMap.get(nbRloc).getPriority() == (short) 255) {
+                if (sbLocatorMap.get(nbRloc).getPriority().toJava() == (short) 255) {
                     // if SB locator has p == 255 then common locator takes all NB fields except for p
                     // which must be set to 255
                     LocatorRecordBuilder lrb = new LocatorRecordBuilder(nbLocator);
-                    lrb.setPriority((short) 255);
+                    lrb.setPriority(fromJava((short) 255));
                     commonLocators.add(lrb.build());
                 } else {
                     commonLocators.add(nbLocator);
index 38620100014659505ec244738f59c85264eaccc8..71a5197cc75dd6f17f3f286b484ca1b82ba45007 100644 (file)
@@ -11,6 +11,7 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
 
 import java.util.ArrayList;
 import java.util.Date;
@@ -208,20 +209,20 @@ public class MappingMergeUtilTest {
     public void getCommonLocatorRecords_priorityCheck() {
         LocatorRecordBuilder nbLocatorRecordBuilder1 = new LocatorRecordBuilder()
                 .setRloc(IPV4_RLOC_1)
-                .setPriority((short) 1)
+                .setPriority(fromJava((short) 1))
                 .setLocatorId("NB-locator-id");
         LocatorRecordBuilder sbLocatorRecordBuilder1 = new LocatorRecordBuilder()
                 .setRloc(IPV4_RLOC_1)
-                .setPriority((short) 255)
+                .setPriority(fromJava((short) 255))
                 .setLocatorId("SB-locator-id");
 
         LocatorRecordBuilder nbLocatorRecordBuilder2 = new LocatorRecordBuilder()
                 .setRloc(IPV4_RLOC_2)
-                .setPriority((short) 1)
+                .setPriority(fromJava((short) 1))
                 .setLocatorId("NB-locator-id");
         LocatorRecordBuilder sbLocatorRecordBuilder2 = new LocatorRecordBuilder()
                 .setRloc(IPV4_RLOC_2)
-                .setPriority((short) 254)
+                .setPriority(fromJava((short) 254))
                 .setLocatorId("SB-locator-id");
 
         final MappingRecordBuilder nbMappingRecordBuilder = getDefaultMappingRecordBuilder();
@@ -245,11 +246,11 @@ public class MappingMergeUtilTest {
 
         assertEquals("NB-locator-id", resultLocator_1.getLocatorId());
         // priority changed to 255
-        assertEquals(255, (short) resultLocator_1.getPriority());
+        assertEquals(255, (short) resultLocator_1.getPriority().toJava());
 
         assertEquals("NB-locator-id", resultLocator_2.getLocatorId());
         // priority remains original
-        assertEquals(1, (short) resultLocator_2.getPriority());
+        assertEquals(1, (short) resultLocator_2.getPriority().toJava());
     }
 
     private static MappingData getDefaultMappingData() {
index dc8a9a296ca3e49ed44709a8e511e3c1d3c43b20..c38ca571f6e31c54e204c95b78242e169a19ea41 100644 (file)
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <properties>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
+  </properties>
+
   <dependencies>
     <dependency>
       <groupId>${project.groupId}</groupId>
       <version>${project.version}</version>
     </dependency>
   </dependencies>
-
-  <build>
-    <pluginManagement>
-      <plugins>
-        <plugin>
-          <!-- Enforce odlparent checkstyle -->
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-checkstyle-plugin</artifactId>
-          <configuration>
-            <includeTestSourceDirectory>true</includeTestSourceDirectory>
-            <propertyExpansion>checkstyle.violationSeverity=error</propertyExpansion>
-          </configuration>
-        </plugin>
-      </plugins>
-    </pluginManagement>
-  </build>
 </project>
index 93f2b377bc9cf1c8458af07420ce64a07e67429b..8048cf807a556019f4d492360122ce098d9fb9a7 100644 (file)
@@ -49,10 +49,10 @@ public class HashMapDb implements ILispDAO, AutoCloseable {
             Eid eid = (Eid) key;
             if (eid.getAddress() instanceof Ipv4PrefixBinary) {
                 Ipv4PrefixBinary prefix = (Ipv4PrefixBinary) eid.getAddress();
-                ip4Trie.insert(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength(), key);
+                ip4Trie.insert(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength().toJava(), key);
             } else if (eid.getAddress() instanceof Ipv6PrefixBinary) {
                 Ipv6PrefixBinary prefix = (Ipv6PrefixBinary) eid.getAddress();
-                ip6Trie.insert(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength(), key);
+                ip6Trie.insert(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength().toJava(), key);
             }
         }
     }
@@ -85,10 +85,10 @@ public class HashMapDb implements ILispDAO, AutoCloseable {
     private RadixTrie<Object>.TrieNode lookupBestNode(Eid eid) {
         if (eid.getAddress() instanceof Ipv4PrefixBinary) {
             Ipv4PrefixBinary prefix = (Ipv4PrefixBinary) eid.getAddress();
-            return ip4Trie.lookupBest(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength());
+            return ip4Trie.lookupBest(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength().toJava());
         } else if (eid.getAddress() instanceof Ipv6PrefixBinary) {
             Ipv6PrefixBinary prefix = (Ipv6PrefixBinary) eid.getAddress();
-            return ip6Trie.lookupBest(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength());
+            return ip6Trie.lookupBest(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength().toJava());
         }
         return null;
     }
@@ -140,21 +140,23 @@ public class HashMapDb implements ILispDAO, AutoCloseable {
             switch (method) {
                 case COVERING:
                     node = ip4Trie.lookupCoveringLessSpecific(prefix.getIpv4AddressBinary().getValue(),
-                            prefix.getIpv4MaskLength());
+                            prefix.getIpv4MaskLength().toJava());
                     break;
                 case PARENT:
-                    node = ip4Trie.lookupParent(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength());
+                    node = ip4Trie.lookupParent(prefix.getIpv4AddressBinary().getValue(),
+                            prefix.getIpv4MaskLength().toJava());
                     break;
                 case SIBLING:
-                    node = ip4Trie.lookupSibling(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength());
+                    node = ip4Trie.lookupSibling(prefix.getIpv4AddressBinary().getValue(),
+                            prefix.getIpv4MaskLength().toJava());
                     break;
                 case VIRTUAL_PARENT_SIBLING:
                     node = ip4Trie.lookupVirtualParentSibling(prefix.getIpv4AddressBinary().getValue(),
-                            prefix.getIpv4MaskLength());
+                            prefix.getIpv4MaskLength().toJava());
                     break;
                 case WIDEST_NEGATIVE:
                     node = ip4Trie.lookupWidestNegative(prefix.getIpv4AddressBinary().getValue(),
-                            prefix.getIpv4MaskLength());
+                            prefix.getIpv4MaskLength().toJava());
                     break;
                 default:
                     node = null;
@@ -169,21 +171,23 @@ public class HashMapDb implements ILispDAO, AutoCloseable {
             switch (method) {
                 case COVERING:
                     node = ip6Trie.lookupCoveringLessSpecific(prefix.getIpv6AddressBinary().getValue(),
-                            prefix.getIpv6MaskLength());
+                            prefix.getIpv6MaskLength().toJava());
                     break;
                 case PARENT:
-                    node = ip6Trie.lookupParent(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength());
+                    node = ip6Trie.lookupParent(prefix.getIpv6AddressBinary().getValue(),
+                            prefix.getIpv6MaskLength().toJava());
                     break;
                 case SIBLING:
-                    node = ip6Trie.lookupSibling(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength());
+                    node = ip6Trie.lookupSibling(prefix.getIpv6AddressBinary().getValue(),
+                            prefix.getIpv6MaskLength().toJava());
                     break;
                 case VIRTUAL_PARENT_SIBLING:
                     node = ip6Trie.lookupVirtualParentSibling(prefix.getIpv6AddressBinary().getValue(),
-                            prefix.getIpv6MaskLength());
+                            prefix.getIpv6MaskLength().toJava());
                     break;
                 case WIDEST_NEGATIVE:
                     node = ip6Trie.lookupWidestNegative(prefix.getIpv6AddressBinary().getValue(),
-                            prefix.getIpv6MaskLength());
+                            prefix.getIpv6MaskLength().toJava());
                     break;
                 default:
                     node = null;
@@ -227,10 +231,12 @@ public class HashMapDb implements ILispDAO, AutoCloseable {
         Set<RadixTrie<Object>.TrieNode> nodes = null;
         if (key.getAddress() instanceof Ipv4PrefixBinary) {
             Ipv4PrefixBinary prefix = (Ipv4PrefixBinary) key.getAddress();
-            nodes = ip4Trie.lookupSubtree(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength());
+            nodes = ip4Trie.lookupSubtree(prefix.getIpv4AddressBinary().getValue(),
+                    prefix.getIpv4MaskLength().toJava());
         } else if (key.getAddress() instanceof Ipv6PrefixBinary) {
             Ipv6PrefixBinary prefix = (Ipv6PrefixBinary) key.getAddress();
-            nodes = ip6Trie.lookupSubtree(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength());
+            nodes = ip6Trie.lookupSubtree(prefix.getIpv6AddressBinary().getValue(),
+                    prefix.getIpv6MaskLength().toJava());
         }
         return nodesToEids(key, nodes);
     }
@@ -258,10 +264,10 @@ public class HashMapDb implements ILispDAO, AutoCloseable {
             Eid eid = (Eid) key;
             if (eid.getAddress() instanceof Ipv4PrefixBinary) {
                 Ipv4PrefixBinary prefix = (Ipv4PrefixBinary) eid.getAddress();
-                ip4Trie.remove(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength());
+                ip4Trie.remove(prefix.getIpv4AddressBinary().getValue(), prefix.getIpv4MaskLength().toJava());
             } else if (eid.getAddress() instanceof Ipv6PrefixBinary) {
                 Ipv6PrefixBinary prefix = (Ipv6PrefixBinary) eid.getAddress();
-                ip6Trie.remove(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength());
+                ip6Trie.remove(prefix.getIpv6AddressBinary().getValue(), prefix.getIpv6MaskLength().toJava());
             }
         }
     }
index 83cdffe1496fd21523f01bcfff122a19e218f2b3..ed210b18c63649c3f597259949b0d7bdf19955da 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.mdsal</groupId>
     <artifactId>binding-parent</artifactId>
-    <version>4.0.6</version>
+    <version>5.0.3</version>
     <relativePath/>
   </parent>
 
@@ -26,6 +26,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
 
   <properties>
     <pmd.cpd.fail>true</pmd.cpd.fail>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
   </properties>
 
   <dependencies>
@@ -62,20 +63,4 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <scope>test</scope>
     </dependency>
   </dependencies>
-
-  <build>
-    <pluginManagement>
-      <plugins>
-        <plugin>
-          <!-- Enforce odlparent checkstyle -->
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-checkstyle-plugin</artifactId>
-          <configuration>
-            <includeTestSourceDirectory>true</includeTestSourceDirectory>
-            <propertyExpansion>checkstyle.violationSeverity=error</propertyExpansion>
-          </configuration>
-        </plugin>
-      </plugins>
-    </pluginManagement>
-  </build>
 </project>
index 391a0ac7defe79b763ac70fc3ee52e3acf868a02..8d736d8e4dd6e867c754966fb3a2d6b1c613828f 100644 (file)
@@ -7,6 +7,8 @@
  */
 package org.opendaylight.lispflowmapping.lisp.serializer;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import java.nio.ByteBuffer;
 
 import org.apache.commons.lang3.BooleanUtils;
@@ -32,10 +34,10 @@ public final class LocatorRecordSerializer {
 
     protected LocatorRecord deserialize(ByteBuffer buffer) {
         LocatorRecordBuilder builder = new LocatorRecordBuilder();
-        builder.setPriority((short) ByteUtil.getUnsignedByte(buffer));
-        builder.setWeight((short) ByteUtil.getUnsignedByte(buffer));
-        builder.setMulticastPriority((short) ByteUtil.getUnsignedByte(buffer));
-        builder.setMulticastWeight((short) ByteUtil.getUnsignedByte(buffer));
+        builder.setPriority(fromJava((short) ByteUtil.getUnsignedByte(buffer)));
+        builder.setWeight(fromJava((short) ByteUtil.getUnsignedByte(buffer)));
+        builder.setMulticastPriority(fromJava((short) ByteUtil.getUnsignedByte(buffer)));
+        builder.setMulticastWeight(fromJava((short) ByteUtil.getUnsignedByte(buffer)));
         byte flags = (byte) buffer.getShort();
         builder.setLocalLocator(ByteUtil.extractBit(flags, Flags.LOCAL_LOCATOR));
         builder.setRlocProbed(ByteUtil.extractBit(flags, Flags.RLOC_PROBED));
index e5c306a05b51f9bd4e004ac07611586773bef314..da276104c270f5a688d235ce46f9841ff47ef830 100644 (file)
@@ -7,6 +7,8 @@
  */
 package org.opendaylight.lispflowmapping.lisp.serializer.address;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import java.nio.ByteBuffer;
 
 import org.opendaylight.lispflowmapping.lisp.type.LispCanonicalAddressFormatEnum;
@@ -59,7 +61,7 @@ public final class ApplicationDataSerializer extends LcafSerializer {
         ApplicationData appData = ((org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types
                 .rev151105.lisp.address.address.ApplicationData) lispAddress.getAddress()).getApplicationData();
         buffer.put(ByteUtil.partialIntToByteArray(NumberUtil.asInt(appData.getIpTos()), Length.TOC));
-        buffer.put((byte) NumberUtil.asShort(appData.getProtocol()));
+        buffer.put((byte) NumberUtil.asShort(appData.getProtocol().toJava()));
         if (appData.getLocalPortLow() != null) {
             buffer.putShort(NumberUtil.asShort(appData.getLocalPortLow().getValue().shortValue()));
         } else {
@@ -107,11 +109,11 @@ public final class ApplicationDataSerializer extends LcafSerializer {
         byte[] rawIPTos = new byte[3];
         buffer.get(rawIPTos);
         builder.setIpTos(ByteUtil.getPartialInt(rawIPTos));
-        builder.setProtocol((short) ByteUtil.getUnsignedByte(buffer));
-        builder.setLocalPortLow(new PortNumber(ByteUtil.asUnsignedShort(buffer.getShort())));
-        builder.setLocalPortHigh(new PortNumber(ByteUtil.asUnsignedShort(buffer.getShort())));
-        builder.setRemotePortLow(new PortNumber(ByteUtil.asUnsignedShort(buffer.getShort())));
-        builder.setRemotePortHigh(new PortNumber(ByteUtil.asUnsignedShort(buffer.getShort())));
+        builder.setProtocol(fromJava((short) ByteUtil.getUnsignedByte(buffer)));
+        builder.setLocalPortLow(new PortNumber(fromJava(ByteUtil.asUnsignedShort(buffer.getShort()))));
+        builder.setLocalPortHigh(new PortNumber(fromJava(ByteUtil.asUnsignedShort(buffer.getShort()))));
+        builder.setRemotePortLow(new PortNumber(fromJava(ByteUtil.asUnsignedShort(buffer.getShort()))));
+        builder.setRemotePortHigh(new PortNumber(fromJava(ByteUtil.asUnsignedShort(buffer.getShort()))));
         SimpleAddress address = SimpleAddressSerializer.getInstance().deserialize(buffer, ctx);
         builder.setAddress(address);
         return new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
index 175b23c410b2f8740eff942d7ba35dda43c94dd9..7d933759f73b0869780720fbf3c7d0dff978c471 100644 (file)
@@ -7,6 +7,8 @@
  */
 package org.opendaylight.lispflowmapping.lisp.serializer.address;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import java.nio.ByteBuffer;
 import org.opendaylight.lispflowmapping.lisp.type.LispCanonicalAddressFormatEnum;
 import org.opendaylight.lispflowmapping.lisp.util.ByteUtil;
@@ -59,7 +61,7 @@ public final class ServicePathSerializer extends LcafSerializer {
         ServicePath sp = (ServicePath) lispAddress.getAddress();
         buffer.put(ByteUtil.partialIntToByteArray(NumberUtil.asInt(
                 sp.getServicePath().getServicePathId().getValue().intValue()), Length.SPI));
-        buffer.put((byte) NumberUtil.asShort(sp.getServicePath().getServiceIndex()));
+        buffer.put((byte) NumberUtil.asShort(sp.getServicePath().getServiceIndex().toJava()));
     }
 
     @Override
@@ -75,8 +77,8 @@ public final class ServicePathSerializer extends LcafSerializer {
         ServicePathBuilder spb = new ServicePathBuilder();
         byte[] spi = new byte[3];
         buffer.get(spi);
-        spb.setServicePathId(new ServicePathIdType((long) ByteUtil.getPartialInt(spi)));
-        spb.setServiceIndex((short) ByteUtil.getUnsignedByte(buffer));
+        spb.setServicePathId(new ServicePathIdType(fromJava((long) ByteUtil.getPartialInt(spi))));
+        spb.setServiceIndex(fromJava((short) ByteUtil.getUnsignedByte(buffer)));
         return new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                 .lisp.address.address.ServicePathBuilder().setServicePath(spb.build()).build();
     }
index 4362edc2cfa0b9b28efa6601178a3fc2e8bcff70..29e2409077818fff41ac68a0df89c26daf0e2916 100644 (file)
@@ -138,7 +138,7 @@ public final class LispAddressStringifier {
         Long vni = null;
 
         if (lispAddress.getVirtualNetworkId() != null) {
-            vni = lispAddress.getVirtualNetworkId().getValue();
+            vni = lispAddress.getVirtualNetworkId().getValue().toJava();
         }
 
         return getAddrString(dst, addr, vni);
index 7f7b6c2505e0a4680e5faa7a46abad21d1f5cbd1..f7fac320f0b0815dd9eea349e4fccae2fd2d6f31 100644 (file)
@@ -7,6 +7,8 @@
  */
 package org.opendaylight.lispflowmapping.lisp.util;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import com.google.common.base.Preconditions;
 import com.google.common.net.InetAddresses;
 import java.net.Inet4Address;
@@ -473,7 +475,7 @@ public final class LispAddressUtil {
     }
 
     public static Eid asIpv4PrefixBinaryEid(long vni, final String prefix) {
-        InstanceIdType iid = new InstanceIdType(vni);
+        InstanceIdType iid = new InstanceIdType(fromJava(vni));
         return asIpv4PrefixBinaryEid(prefix, iid);
     }
 
@@ -482,7 +484,7 @@ public final class LispAddressUtil {
         short mask = Short.valueOf(MaskUtil.getPrefixMask(prefix));
         byte[] ipv4 = InetAddresses.forString(address).getAddress();
         return toEid(new Ipv4PrefixBinaryBuilder().setIpv4AddressBinary(new Ipv4AddressBinary(ipv4))
-                .setIpv4MaskLength(mask).build(), iiType);
+                .setIpv4MaskLength(fromJava(mask)).build(), iiType);
     }
 
     public static Eid asIpv4PrefixBinaryEid(Eid eid, byte[] address, short mask) {
@@ -490,7 +492,7 @@ public final class LispAddressUtil {
     }
 
     public static Eid asIpv4PrefixBinaryEid(long vni, byte[] address, short mask) {
-        return asIpv4PrefixBinaryEid(new InstanceIdType(vni), address, mask);
+        return asIpv4PrefixBinaryEid(new InstanceIdType(fromJava(vni)), address, mask);
     }
 
     public static Eid asIpv4PrefixBinaryEid(InstanceIdType vni, byte[] address, short mask) {
@@ -500,7 +502,7 @@ public final class LispAddressUtil {
         builder.setAddressType(Ipv4PrefixBinaryAfi.class);
         builder.setVirtualNetworkId(vni);
         builder.setAddress(new Ipv4PrefixBinaryBuilder().setIpv4AddressBinary(new Ipv4AddressBinary(address))
-                .setIpv4MaskLength(mask).build());
+                .setIpv4MaskLength(fromJava(mask)).build());
         return builder.build();
     }
 
@@ -509,7 +511,8 @@ public final class LispAddressUtil {
     }
 
     public static Eid asIpv4Eid(String address, long vni) {
-        return toEid(new Ipv4AddressBinary(InetAddresses.forString(address).getAddress()), new InstanceIdType(vni));
+        return toEid(new Ipv4AddressBinary(InetAddresses.forString(address).getAddress()),
+                new InstanceIdType(fromJava(vni)));
     }
 
     public static Eid asIpv6Eid(String address) {
@@ -517,7 +520,8 @@ public final class LispAddressUtil {
     }
 
     public static Eid asIpv6Eid(String address, long vni) {
-        return toEid(new Ipv6AddressBinary(InetAddresses.forString(address).getAddress()), new InstanceIdType(vni));
+        return toEid(new Ipv6AddressBinary(InetAddresses.forString(address).getAddress()),
+                new InstanceIdType(fromJava(vni)));
     }
 
     public static Eid asIpv6PrefixEid(String prefix) {
@@ -542,7 +546,7 @@ public final class LispAddressUtil {
     }
 
     public static Eid asIpv6PrefixBinaryEid(long vni, final String prefix) {
-        InstanceIdType iid = new InstanceIdType(vni);
+        InstanceIdType iid = new InstanceIdType(fromJava(vni));
         return asIpv6PrefixBinaryEid(prefix, iid);
     }
 
@@ -551,7 +555,7 @@ public final class LispAddressUtil {
         short mask = Short.valueOf(MaskUtil.getPrefixMask(prefix));
         byte[] ipv6 = InetAddresses.forString(address).getAddress();
         return toEid(new Ipv6PrefixBinaryBuilder().setIpv6AddressBinary(new Ipv6AddressBinary(ipv6))
-                .setIpv6MaskLength(mask).build(), iiType);
+                .setIpv6MaskLength(fromJava(mask)).build(), iiType);
     }
 
     public static Eid asIpv6PrefixBinaryEid(Eid eid, byte[] address, short mask) {
@@ -559,7 +563,7 @@ public final class LispAddressUtil {
     }
 
     public static Eid asIpv6PrefixBinaryEid(long vni, byte[] address, short mask) {
-        return asIpv6PrefixBinaryEid(new InstanceIdType(vni), address, mask);
+        return asIpv6PrefixBinaryEid(new InstanceIdType(fromJava(vni)), address, mask);
     }
 
     public static Eid asIpv6PrefixBinaryEid(InstanceIdType vni, byte[] address, short mask) {
@@ -569,7 +573,7 @@ public final class LispAddressUtil {
         builder.setAddressType(Ipv6PrefixBinaryAfi.class);
         builder.setVirtualNetworkId(vni);
         builder.setAddress(new Ipv6PrefixBinaryBuilder().setIpv6AddressBinary(new Ipv6AddressBinary(address))
-                .setIpv6MaskLength(mask).build());
+                .setIpv6MaskLength(fromJava(mask)).build());
         return builder.build();
     }
 
@@ -639,19 +643,19 @@ public final class LispAddressUtil {
         EidBuilder eb = new EidBuilder();
         eb.setAddressType(ServicePathLcaf.class);
         if (vni >= 0) {
-            eb.setVirtualNetworkId(new InstanceIdType(vni));
+            eb.setVirtualNetworkId(new InstanceIdType(fromJava(vni)));
         }
         org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                 .lisp.address.address.service.path.ServicePathBuilder spb =
                 new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                 .lisp.address.address.service.path.ServicePathBuilder();
-        spb.setServicePathId(new ServicePathIdType(servicePathId));
-        spb.setServiceIndex(serviceIndex);
+        spb.setServicePathId(new ServicePathIdType(fromJava(servicePathId)));
+        spb.setServiceIndex(fromJava(serviceIndex));
         return eb.setAddress(addressFromServicePath(spb.build())).build();
     }
 
     public static Eid asMacEid(String address, long vni) {
-        return toEid(new MacAddress(address), new InstanceIdType(vni));
+        return toEid(new MacAddress(address), new InstanceIdType(fromJava(vni)));
     }
 
     public static Eid asMacEid(String address) {
@@ -659,7 +663,7 @@ public final class LispAddressUtil {
     }
 
     public static Eid asDistinguishedNameEid(String address, long vni) {
-        return toEid(new MacAddress(address), new InstanceIdType(vni));
+        return toEid(new MacAddress(address), new InstanceIdType(fromJava(vni)));
     }
 
     public static Eid asDistinguishedNameEid(String address) {
@@ -708,7 +712,7 @@ public final class LispAddressUtil {
     public static Eid asSrcDstEid(String src, String dst, int smask, int dmask, long vni) {
         EidBuilder builder = new EidBuilder();
         builder.setAddressType(SourceDestKeyLcaf.class);
-        builder.setVirtualNetworkId(new InstanceIdType(Long.valueOf(vni)));
+        builder.setVirtualNetworkId(new InstanceIdType(fromJava(Long.valueOf(vni))));
         builder.setAddress(
                 new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                         .lisp.address.address.SourceDestKeyBuilder()
@@ -753,8 +757,9 @@ public final class LispAddressUtil {
         List<LocatorRecord> locatorRecords = new ArrayList<LocatorRecord>();
         for (Rloc locator : locators) {
             LocatorRecordBuilder locatorBuilder = new LocatorRecordBuilder();
-            locatorBuilder.setLocalLocator(false).setRlocProbed(false).setWeight((short) 1).setPriority((short) 1)
-                    .setMulticastWeight((short) 1).setMulticastPriority((short) 1).setRouted(true)
+            locatorBuilder.setLocalLocator(false).setRlocProbed(false)
+                    .setWeight(fromJava((short) 1)).setPriority(fromJava((short) 1))
+                    .setMulticastWeight(fromJava((short) 1)).setMulticastPriority(fromJava((short) 1)).setRouted(true)
                     .setRloc(locator).setLocatorId("SFC_LISP").build();
             locatorRecords.add(locatorBuilder.build());
         }
@@ -788,7 +793,7 @@ public final class LispAddressUtil {
         Ipv4PrefixBinaryBuilder pb = new Ipv4PrefixBinaryBuilder();
         byte[] address = InetAddresses.forString(MaskUtil.getAddressStringForIpv4Prefix(prefix)).getAddress();
         pb.setIpv4AddressBinary(new Ipv4AddressBinary(address));
-        pb.setIpv4MaskLength(MaskUtil.getMaskForAddress(prefix));
+        pb.setIpv4MaskLength(fromJava(MaskUtil.getMaskForAddress(prefix)));
         return pb.build();
     }
 
@@ -797,7 +802,7 @@ public final class LispAddressUtil {
         Ipv6PrefixBinaryBuilder pb = new Ipv6PrefixBinaryBuilder();
         byte[] address = InetAddresses.forString(MaskUtil.getAddressStringForIpv6Prefix(prefix)).getAddress();
         pb.setIpv6AddressBinary(new Ipv6AddressBinary(address));
-        pb.setIpv6MaskLength(MaskUtil.getMaskForAddress(prefix));
+        pb.setIpv6MaskLength(fromJava(MaskUtil.getMaskForAddress(prefix)));
         return pb.build();
     }
 
@@ -889,14 +894,14 @@ public final class LispAddressUtil {
             .lisp.address.address.Ipv4Prefix convertFromBinary(Ipv4PrefixBinary prefix) {
         return new Ipv4PrefixBuilder().setIpv4Prefix(IetfInetUtil.INSTANCE.ipv4PrefixFor(
                 prefix.getIpv4AddressBinary().getValue(),
-                prefix.getIpv4MaskLength())).build();
+                prefix.getIpv4MaskLength().toJava())).build();
     }
 
     private static org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
             .lisp.address.address.Ipv6Prefix convertFromBinary(Ipv6PrefixBinary prefix) {
         return new Ipv6PrefixBuilder().setIpv6Prefix(IetfInetUtil.INSTANCE.ipv6PrefixFor(
                 prefix.getIpv6AddressBinary().getValue(),
-                prefix.getIpv6MaskLength())).build();
+                prefix.getIpv6MaskLength().toJava())).build();
     }
 
     private static Ipv4 convertFromBinary(Ipv4Binary address) {
@@ -999,9 +1004,9 @@ public final class LispAddressUtil {
     public static short getIpPrefixMask(Eid eid) {
         Address addr = eid.getAddress();
         if (addr instanceof Ipv4PrefixBinary) {
-            return ((Ipv4PrefixBinary) addr).getIpv4MaskLength();
+            return ((Ipv4PrefixBinary) addr).getIpv4MaskLength().toJava();
         } else if (addr instanceof Ipv6PrefixBinary) {
-            return ((Ipv6PrefixBinary) addr).getIpv6MaskLength();
+            return ((Ipv6PrefixBinary) addr).getIpv6MaskLength().toJava();
         }
         return 0;
     }
index 9be7e2bbe96940e0da02a3d9e1675349cc2d9987..b6cf1a9bf97c7058f5ea77d5079478f55bec64c4 100644 (file)
@@ -114,11 +114,11 @@ public final class MaskUtil {
         try {
             if (address instanceof Ipv4PrefixBinary) {
                 byte[] addr = ((Ipv4PrefixBinary) address).getIpv4AddressBinary().getValue();
-                short mask = ((Ipv4PrefixBinary) address).getIpv4MaskLength();
+                short mask = ((Ipv4PrefixBinary) address).getIpv4MaskLength().toJava();
                 return LispAddressUtil.asIpv4PrefixBinaryEid(eid, normalizeByteArray(addr, mask), mask);
             } else if (address instanceof Ipv6PrefixBinary) {
                 byte[] addr = ((Ipv6PrefixBinary) address).getIpv6AddressBinary().getValue();
-                short mask = ((Ipv6PrefixBinary) address).getIpv6MaskLength();
+                short mask = ((Ipv6PrefixBinary) address).getIpv6MaskLength().toJava();
                 return LispAddressUtil.asIpv6PrefixBinaryEid(eid, normalizeByteArray(addr, mask), mask);
             } else if (address instanceof Ipv4Prefix) {
                 String[] v4prefix = splitPrefix(((Ipv4Prefix)address).getIpv4Prefix().getValue());
@@ -141,8 +141,8 @@ public final class MaskUtil {
                 return normalizeSrcDst(eid);
             } else if (address instanceof ServicePath) {
                 // Build new Service Path eid with service index set to 0
-                long spi = ((ServicePath) address).getServicePath().getServicePathId().getValue();
-                long vni = eid.getVirtualNetworkId() != null ? eid.getVirtualNetworkId().getValue() : -1;
+                long spi = ((ServicePath) address).getServicePath().getServicePathId().getValue().toJava();
+                long vni = eid.getVirtualNetworkId() != null ? eid.getVirtualNetworkId().getValue().toJava() : -1;
                 return LispAddressUtil.asServicePathEid(vni, spi, (short)0);
             }
         } catch (UnknownHostException e) {
@@ -258,9 +258,9 @@ public final class MaskUtil {
         } else if (address instanceof InstanceId) {
             return getMaskForAddress(((InstanceId)address).getInstanceId().getAddress());
         } else if (address instanceof Ipv4PrefixBinary) {
-            return ((Ipv4PrefixBinary) address).getIpv4MaskLength();
+            return ((Ipv4PrefixBinary) address).getIpv4MaskLength().toJava();
         } else if (address instanceof Ipv6PrefixBinary) {
-            return ((Ipv6PrefixBinary) address).getIpv6MaskLength();
+            return ((Ipv6PrefixBinary) address).getIpv6MaskLength().toJava();
         }
         return -1;
     }
index 13b28559db6512228189b3240157c6a9425f0bbd..537ca6055851721c6f575d60a6273dacfabbf46c 100644 (file)
@@ -7,6 +7,8 @@
  */
 package org.opendaylight.lispflowmapping.lisp.util;
 
+import org.opendaylight.yangtools.yang.common.Uint8;
+
 public final class NumberUtil {
     // Utility class, should not be instantiated
     private NumberUtil() {
@@ -33,6 +35,13 @@ public final class NumberUtil {
         return 0;
     }
 
+    public static short asShort(Uint8 number) {
+        if (number != null) {
+            return number.toJava();
+        }
+        return 0;
+    }
+
     public static long asLong(Long number) {
         if (number != null) {
             return number;
index c36e716ceb5193a81c40cb21e44928106147ead5..513c1d60bd99e2bd183f3f9b2364574fcb8aad0d 100644 (file)
@@ -13,6 +13,7 @@ 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.opendaylight.yangtools.yang.common.UintConversions.fromJava;
 
 import com.google.common.net.InetAddresses;
 import java.net.Inet4Address;
@@ -102,7 +103,8 @@ public class LispAddressUtilTest {
     private static final Long AS_NUMBER_TEST = 100L;
     private static final Long INSTANCE_ID_TYPE_VALUE_TEST = 121L;
     private static final Short INSTANCE_ID_TYPE_VALUE_SHORT_TEST = 122;
-    private static final InstanceIdType INSTANCE_ID_TYPE_TEST = new InstanceIdType(INSTANCE_ID_TYPE_VALUE_TEST);
+    private static final InstanceIdType INSTANCE_ID_TYPE_TEST = new InstanceIdType(
+            fromJava(INSTANCE_ID_TYPE_VALUE_TEST));
     private static final String INCORRECT_IP_ADDRESS_TEST = "incorrect ip address";
 
     private static final String MAC_ADDRESS_VALUE_TEST = "aa:bb:cc:dd:ee:ff";
@@ -124,10 +126,10 @@ public class LispAddressUtilTest {
     private static final Ipv4Prefix IPV4_ADDRESS_PREFIX_TEST = new Ipv4Prefix(IPV4_ADDRESS_PREFIX_VALUE_TEST);
     private static final Ipv4PrefixBinary IPV4_ADDRESS_PREFIX_BINARY_1 = new Ipv4PrefixBinaryBuilder()
             .setIpv4AddressBinary(new Ipv4AddressBinary(IPV4_ADDRESS_BYTES_A_TEST))
-            .setIpv4MaskLength(MASK_OK_TEST).build();
+            .setIpv4MaskLength(fromJava(MASK_OK_TEST)).build();
     private static final Ipv4PrefixBinary IPV4_ADDRESS_PREFIX_BINARY_2 = new Ipv4PrefixBinaryBuilder()
             .setIpv4AddressBinary(new Ipv4AddressBinary(IPV4_ADDRESS_BYTES_A_TEST))
-            .setIpv4MaskLength(MASK_OK_DEFAULT_IPV4_TEST).build();
+            .setIpv4MaskLength(fromJava(MASK_OK_DEFAULT_IPV4_TEST)).build();
     private static final IpPrefix IP_ADDRESS_PREFIX_WITH_IPV4_TEST = new IpPrefix(IPV4_ADDRESS_PREFIX_TEST);
     private static final org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105.lisp
             .address.address.Ipv4Prefix IPV4_PREFIX = new Ipv4PrefixBuilder()
@@ -178,10 +180,10 @@ public class LispAddressUtilTest {
     private static final IpAddress IP_ADDRESS_IPV6_TEST = new IpAddress(IPV6_ADDRESS_TEST);
     private static final Ipv6PrefixBinary IPV6_ADDRESS_PREFIX_BINARY_1 = new Ipv6PrefixBinaryBuilder()
             .setIpv6AddressBinary(new Ipv6AddressBinary(IPV6_ADDRESS_BYTES_A_TEST))
-            .setIpv6MaskLength(MASK_OK_TEST).build();
+            .setIpv6MaskLength(fromJava(MASK_OK_TEST)).build();
     private static final Ipv6PrefixBinary IPV6_ADDRESS_PREFIX_BINARY_2 = new Ipv6PrefixBinaryBuilder()
             .setIpv6AddressBinary(new Ipv6AddressBinary(IPV6_ADDRESS_BYTES_A_TEST))
-            .setIpv6MaskLength(MASK_OK_DEFAULT_IPV6_TEST).build();
+            .setIpv6MaskLength(fromJava(MASK_OK_DEFAULT_IPV6_TEST)).build();
     private static final Eid IPV6_ADDRESS_PREFIX_BINARY_EID_1 = new EidBuilder()
             .setAddress(IPV6_ADDRESS_PREFIX_BINARY_1)
             .setAddressType(Ipv6PrefixBinaryAfi.class)
@@ -289,7 +291,7 @@ public class LispAddressUtilTest {
      */
     @Test
     public void addressFromSimpleAddressTest_asNumber() {
-        final SimpleAddress simpleAddress = new SimpleAddress(new AsNumber(NUMBER_TEST));
+        final SimpleAddress simpleAddress = new SimpleAddress(new AsNumber(fromJava(NUMBER_TEST)));
         final Class<? extends LispAddressFamily> addressClass = LispAddressUtil.addressTypeFromSimpleAddress(
                 simpleAddress);
         assertEquals(AsNumberAfi.class, addressClass);
@@ -533,8 +535,8 @@ public class LispAddressUtilTest {
                 .address.service.path.ServicePathBuilder servicePathBuilder = new org.opendaylight.yang.gen.v1.urn.ietf
                 .params.xml.ns.yang.ietf.lisp.address.types.rev151105.lisp.address
                 .address.service.path.ServicePathBuilder();
-        servicePathBuilder.setServiceIndex(SERVICE_INDEX_TEST);
-        servicePathBuilder.setServicePathId(new ServicePathIdType(DUMMY_SERVICE_PATH_ID_TYPE));
+        servicePathBuilder.setServiceIndex(fromJava(SERVICE_INDEX_TEST));
+        servicePathBuilder.setServicePathId(new ServicePathIdType(fromJava(DUMMY_SERVICE_PATH_ID_TYPE)));
 
         ServicePath expectedAddress = servicePathBuilder.build();
         final Address testedAddress = LispAddressUtil.addressFromServicePath(expectedAddress);
@@ -580,7 +582,7 @@ public class LispAddressUtilTest {
      */
     @Test
     public void addressFromAsNumberTest_withAdNumber() {
-        final AsNumber expectedAddress = new AsNumber(AS_NUMBER_TEST);
+        final AsNumber expectedAddress = new AsNumber(fromJava(AS_NUMBER_TEST));
         final Address testedAddress = LispAddressUtil.addressFromAsNumber(expectedAddress);
         assertTrue(testedAddress instanceof org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp
                 .address.types.rev151105.lisp.address.address.AsNumber);
@@ -862,7 +864,7 @@ public class LispAddressUtilTest {
         assertArrayEquals(IPV6_ADDRESS_BYTES_A_TEST, ((Ipv6PrefixBinary) eidFromIpv6PrefixBinary
                 .getAddress()).getIpv6AddressBinary().getValue());
         assertEquals(expectedMask,
-                (short) ((Ipv6PrefixBinary) eidFromIpv6PrefixBinary.getAddress()).getIpv6MaskLength());
+                (short) ((Ipv6PrefixBinary) eidFromIpv6PrefixBinary.getAddress()).getIpv6MaskLength().toJava());
     }
 
     private void verifyToEidWithIpv4(final Eid eidFromIpv4, final boolean isVniChecked) {
@@ -930,7 +932,7 @@ public class LispAddressUtilTest {
         assertArrayEquals(IPV4_ADDRESS_BYTES_A_TEST,
                 ((Ipv4PrefixBinary) eidFromIpv4PrefixBinary.getAddress()).getIpv4AddressBinary().getValue());
         assertEquals(expectedMask,
-                (short) ((Ipv4PrefixBinary) eidFromIpv4PrefixBinary.getAddress()).getIpv4MaskLength());
+                (short) ((Ipv4PrefixBinary) eidFromIpv4PrefixBinary.getAddress()).getIpv4MaskLength().toJava());
     }
 
     private void verifyToEidWithIpv6Binary(final Eid eidFromIpv6, final boolean isVniChecked) {
@@ -1248,10 +1250,10 @@ public class LispAddressUtilTest {
             assertFalse(locatorRecord.isLocalLocator());
             assertFalse(locatorRecord.isRlocProbed());
             assertTrue(locatorRecord.isRouted());
-            assertTrue(1 == locatorRecord.getWeight());
-            assertTrue(1 == locatorRecord.getPriority());
-            assertTrue(1 == locatorRecord.getMulticastWeight());
-            assertTrue(1 == locatorRecord.getMulticastPriority());
+            assertTrue(1 == locatorRecord.getWeight().toJava());
+            assertTrue(1 == locatorRecord.getPriority().toJava());
+            assertTrue(1 == locatorRecord.getMulticastWeight().toJava());
+            assertTrue(1 == locatorRecord.getMulticastPriority().toJava());
             assertEquals("SFC_LISP", locatorRecord.getLocatorId());
         }
     }
@@ -1500,8 +1502,8 @@ public class LispAddressUtilTest {
         ServicePath servicePath = ((org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp
                 .address.types.rev151105.lisp.address.address.ServicePath) eid.getAddress()).getServicePath();
         assertNotNull(servicePath);
-        assertEquals(SERVICE_INDEX_TEST, servicePath.getServiceIndex());
-        assertEquals(SERVICE_PATH_ID_TEST, servicePath.getServicePathId().getValue());
+        assertEquals((short) SERVICE_INDEX_TEST, servicePath.getServiceIndex().toJava());
+        assertEquals((long) SERVICE_PATH_ID_TEST, servicePath.getServicePathId().getValue().toJava());
     }
 
     /**
index d87912c6e62833de0cda75a8c748436cbaadab2c..322f6b4d9a3abe73ee1c63b64ef0c8692d5fe743 100644 (file)
@@ -11,6 +11,7 @@ import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
 
 import java.net.Inet4Address;
 import java.net.Inet6Address;
@@ -87,9 +88,9 @@ public class MaskUtilTest {
     private static final Ipv4AddressBinary IPV4_ADDRESS_BINARY = new Ipv4AddressBinary(ipv4ByteAddress);
     private static final Ipv6AddressBinary IPV6_ADDRESS_BINARY = new Ipv6AddressBinary(ipv6ByteAddress);
     private static final Ipv4PrefixBinary IPV4_PREFIX_BINARY = new Ipv4PrefixBinaryBuilder()
-            .setIpv4AddressBinary(IPV4_ADDRESS_BINARY).setIpv4MaskLength((short) 32).build();
+            .setIpv4AddressBinary(IPV4_ADDRESS_BINARY).setIpv4MaskLength(fromJava((short) 32)).build();
     private static final Ipv6PrefixBinary IPV6_PREFIX_BINARY = new Ipv6PrefixBinaryBuilder()
-            .setIpv6AddressBinary(IPV6_ADDRESS_BINARY).setIpv6MaskLength((short) 128).build();
+            .setIpv6AddressBinary(IPV6_ADDRESS_BINARY).setIpv6MaskLength(fromJava((short) 128)).build();
 
     private static final Address ADDRESS_IPV4 = new Ipv4Builder().setIpv4(IPV4_ADDRESS).build();
     private static final Address ADDRESS_IPV6 = new Ipv6Builder().setIpv6(new Ipv6Address(IPV6_STRING)).build();
@@ -146,7 +147,7 @@ public class MaskUtilTest {
 
         // expected result
         assertArrayEquals(ipv4ExpectedByteAddress, resultPrefix.getIpv4AddressBinary().getValue());
-        assertEquals((short) 20, (short) resultPrefix.getIpv4MaskLength());
+        assertEquals((short) 20, (short) resultPrefix.getIpv4MaskLength().toJava());
     }
 
     /**
@@ -160,7 +161,7 @@ public class MaskUtilTest {
 
         // expected result
         assertArrayEquals(ipv6ExpectedByteAddress, resultPrefix.getIpv6AddressBinary().getValue());
-        assertEquals((short) 66, (short) resultPrefix.getIpv6MaskLength());
+        assertEquals((short) 66, (short) resultPrefix.getIpv6MaskLength().toJava());
     }
 
     /**
@@ -224,7 +225,7 @@ public class MaskUtilTest {
         // expected result
         final byte[] expectedResult = Inet4Address.getByName("255.255.240.0").getAddress();
         assertArrayEquals(expectedResult, resultPrefix.getIpv4AddressBinary().getValue());
-        assertEquals((short) 20, (short) resultPrefix.getIpv4MaskLength());
+        assertEquals((short) 20, (short) resultPrefix.getIpv4MaskLength().toJava());
     }
 
     /**
@@ -239,7 +240,7 @@ public class MaskUtilTest {
         // expected result
         final byte[] expectedResult = Inet6Address.getByName("1111:2222:3333:4444:4000::").getAddress();
         assertArrayEquals(expectedResult, resultPrefix.getIpv6AddressBinary().getValue());
-        assertEquals((short) 66, (short) resultPrefix.getIpv6MaskLength());
+        assertEquals((short) 66, (short) resultPrefix.getIpv6MaskLength().toJava());
     }
 
     /**
@@ -427,9 +428,9 @@ public class MaskUtilTest {
         return new InstanceIdBuilder()
                 .setInstanceId(new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types
                         .rev151105.lisp.address.address.instance.id.InstanceIdBuilder()
-                        .setIid(new InstanceIdType(1L))
+                        .setIid(new InstanceIdType(fromJava(1L)))
                         .setAddress(new SimpleAddress(new IpPrefix(IPV4_PREFIX)))
-                        .setMaskLength((short) 32).build())
+                        .setMaskLength(fromJava((short) 32)).build())
                 .build();
     }
 
@@ -437,7 +438,7 @@ public class MaskUtilTest {
         return new InstanceIdBuilder()
                 .setInstanceId(new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types
                         .rev151105.lisp.address.address.instance.id.InstanceIdBuilder()
-                        .setIid(new InstanceIdType(1L))
+                        .setIid(new InstanceIdType(fromJava(1L)))
                         .setAddress(new SimpleAddress(new IpAddress(new Ipv4Address(IPV4_STRING_1)))).build())
                 .build();
     }
@@ -448,8 +449,8 @@ public class MaskUtilTest {
                 .xml.ns.yang.ietf.lisp.address.types.rev151105.lisp.address.address.service.path.ServicePathBuilder();
 
         return new ServicePathBuilder().setServicePath(servicePathBuilder
-                        .setServiceIndex((short) 1)
-                        .setServicePathId(new ServicePathIdType(1L)).build())
+                        .setServiceIndex(fromJava((short) 1))
+                        .setServicePathId(new ServicePathIdType(fromJava(1L))).build())
                 .build();
     }
 }
index 131da871fa047b8e80e8d295e8a5c2c51a858ef3..2d29a9edeef69ab5a614fdc744da882eeb12e4ff 100644 (file)
@@ -39,7 +39,7 @@ public class AuthKeyDb implements IAuthKeyDb {
         if (eid.getVirtualNetworkId() == null) {
             return 0;
         } else {
-            return eid.getVirtualNetworkId().getValue();
+            return eid.getVirtualNetworkId().getValue().toJava();
         }
     }
 
index 6f62f72163906e9a6bde1ce4bad38a882283b3ee..90d0d8a6f84ad09ba498c13b7b225bf539057937 100644 (file)
@@ -39,7 +39,7 @@ public class MultiTableMapCache implements IMapCache {
         if (eid.getVirtualNetworkId() == null) {
             return 0;
         } else {
-            return eid.getVirtualNetworkId().getValue();
+            return eid.getVirtualNetworkId().getValue().toJava();
         }
     }
 
index 4310e35cbd438c47b3a1ab9fad12bc1c38020965..7823d7fa3aecb62befb2ec1b0f91603544b6ea08 100644 (file)
@@ -45,7 +45,7 @@ public class SimpleMapCache implements ILispMapCache {
         if (eid.getVirtualNetworkId() == null) {
             return 0;
         } else {
-            return eid.getVirtualNetworkId().getValue();
+            return eid.getVirtualNetworkId().getValue().toJava();
         }
     }
 
@@ -116,8 +116,7 @@ public class SimpleMapCache implements ILispMapCache {
 
     @Override
     public Object getMapping(Eid srcEid, Eid dstEid) {
-        final XtrId xtrId = null;
-        return getMapping(dstEid, xtrId);
+        return getMapping(dstEid, (XtrId) null);
     }
 
     @Override
index bb35a5cc0ffeba77574ab0368d309e33c5b7dda7..a0c124f9ba33924482ed17caf3b80c1605c507bd 100644 (file)
@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.opendaylight.mdsal</groupId>
     <artifactId>binding-parent</artifactId>
-    <version>4.0.6</version>
+    <version>5.0.3</version>
     <relativePath/>
   </parent>
 
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <properties>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
+  </properties>
+
   <dependencyManagement>
     <dependencies>
       <dependency>
@@ -77,7 +81,7 @@
     <dependency>
       <groupId>org.opendaylight.mdsal</groupId>
       <artifactId>mdsal-singleton-common-api</artifactId>
-      <version>4.0.6</version>
+      <version>5.0.3</version>
     </dependency>
 
     <!--Needed for testing-->
index 0c99a36d4d268f8d4be58d00938639e6dd9bf6f5..2dcb8d9c677ceed5be5ed6decf5f488d3fd7759b 100644 (file)
@@ -146,8 +146,8 @@ public class VppEndpointListener implements AutoCloseable, ClusteredDataTreeChan
                 });
 
         return Futures.transform(probeVppNodeForConnection,
-                vd -> nodeIdToKeyedInstanceIdentifierMap.get(newOrModifiedNode.getNodeId()).iterator().next(),
-                MoreExecutors.directExecutor());
+            vd -> nodeIdToKeyedInstanceIdentifierMap.get(newOrModifiedNode.getNodeId()).iterator().next(),
+            MoreExecutors.directExecutor());
     }
 
     private void processNodeOnConnection(final Node newOrModifiedNode) {
index 9140c66389a11693eda5614b2576aa422ab44751..e44e216265c08fc635597f09038eb343f74640fc 100644 (file)
@@ -73,7 +73,7 @@ public class PortDataProcessorTest {
             Field field = portDataProcessor.getClass().getDeclaredField("hostInformationManager");
             field.setAccessible(true);
             field.set(portDataProcessor, hostInformationManager);
-        } catch (Exception e) {
+        } catch (NoSuchFieldException | IllegalAccessException e) {
             throw new RuntimeException("Unable to set hostInformationManager", e);
         }
 
index be6a6ffd25f997f1204c78c3912984a5ca701fd0..4a8fdf390de08c5561d9d029abe779245fd6123c 100644 (file)
@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent</artifactId>
-    <version>5.0.2</version>
+    <version>6.0.0</version>
     <relativePath/>
   </parent>
 
@@ -39,7 +39,7 @@
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>4.0.6</version>
+        <version>5.0.3</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
   </dependencyManagement>
 
   <build>
-    <pluginManagement>
-      <plugins>
-        <plugin>
-          <!-- Enforce odlparent checkstyle -->
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-checkstyle-plugin</artifactId>
-          <configuration>
-            <includeTestSourceDirectory>true</includeTestSourceDirectory>
-            <propertyExpansion>checkstyle.violationSeverity=error</propertyExpansion>
-          </configuration>
-        </plugin>
-      </plugins>
-    </pluginManagement>
     <plugins>
       <plugin>
         <groupId>org.apache.felix</groupId>
index f47739a55ded9134d166bc70b8da18d12d4b4bf0..31aefc41bda9236bbb142708f15805e89d9f8bd0 100644 (file)
@@ -24,8 +24,10 @@ public class LispAddKey  extends OsgiCommandSupport {
 
     @Override
     protected Object doExecute() throws Exception {
-        mappingServiceShell.addDefaultKeyIPv4();
-        mappingServiceShell.addDefaultKeyIPv6();
+        if (mappingServiceShell != null) {
+            mappingServiceShell.addDefaultKeyIPv4();
+            mappingServiceShell.addDefaultKeyIPv6();
+        }
         return null;
     }
 
index 2a684e981a5ac55539146563dcdd02c7851c459d..f4952ba8957aadb5508555e14c189a9594765e6c 100644 (file)
@@ -33,10 +33,12 @@ public class LispKeys  extends OsgiCommandSupport {
     @Override
     @SuppressWarnings("checkstyle:RegexpSinglelineJava")
     protected Object doExecute() throws Exception {
-        if (debug) {
-            System.out.print(mappingServiceShell.printKeys());
-        } else {
-            System.out.print(mappingServiceShell.prettyPrintKeys());
+        if (mappingServiceShell != null) {
+            if (debug) {
+                System.out.print(mappingServiceShell.printKeys());
+            } else {
+                System.out.print(mappingServiceShell.prettyPrintKeys());
+            }
         }
         return null;
     }
index 01a8de054aca2e240258d03af8a1c908d217a690..5fee82e831ed6129cb142d863c12339f9bdc4d68 100644 (file)
@@ -29,10 +29,12 @@ public class LispMappings  extends OsgiCommandSupport {
     @Override
     @SuppressWarnings("checkstyle:RegexpSinglelineJava")
     protected Object doExecute() throws Exception {
-        if (debug) {
-            System.out.print(mappingServiceShell.printMappings());
-        } else {
-            System.out.print(mappingServiceShell.prettyPrintMappings());
+        if (mappingServiceShell != null) {
+            if (debug) {
+                System.out.print(mappingServiceShell.printMappings());
+            } else {
+                System.out.print(mappingServiceShell.prettyPrintMappings());
+            }
         }
         return null;
     }
index ca780ebe4e77ac2d3b066b122eddf9e500de1f9d..593d08820095e86ad4beda2f086d85794dc26d8e 100644 (file)
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <properties>
+    <odlparent.spotbugs.enforce>false</odlparent.spotbugs.enforce>
+  </properties>
+
   <dependencies>
     <dependency>
       <groupId>org.osgi</groupId>
       <groupId>org.opendaylight.lispflowmapping</groupId>
       <artifactId>common.unittest.tools</artifactId>
     </dependency>
+    <dependency>
+      <groupId>org.mockito</groupId>
+      <artifactId>mockito-core</artifactId>
+      <version>2.28.2</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-inline</artifactId>
+      <version>2.28.2</version>
       <scope>test</scope>
     </dependency>
     <dependency>
index 289cd9f7c694dfaf5135b8589254f3988e90f342..6bc70e716799155b0cab62e0cdde2e3880974fc7 100644 (file)
@@ -239,7 +239,7 @@ public class LispSouthboundPlugin implements IConfigLispSouthboundPlugin, AutoCl
     public void handleSerializedLispBuffer(TransportAddress address, ByteBuffer outBuffer,
                                            final MessageType packetType) {
         InetAddress ip = getInetAddress(address);
-        handleSerializedLispBuffer(ip, outBuffer, packetType, address.getPort().getValue(), null);
+        handleSerializedLispBuffer(ip, outBuffer, packetType, address.getPort().getValue().toJava(), null);
     }
 
     public void handleSerializedLispBuffer(InetAddress address, ByteBuffer outBuffer,
index 6a9ae4250bbd296ccc50df47a8f7366a5050897b..b6972db6d43fa39d2145221925224c434f1ecf53 100644 (file)
@@ -8,6 +8,8 @@
 
 package org.opendaylight.lispflowmapping.southbound.lisp;
 
+import static org.opendaylight.yangtools.yang.common.UintConversions.fromJava;
+
 import io.netty.buffer.ByteBufUtil;
 import io.netty.channel.Channel;
 import io.netty.channel.ChannelHandler;
@@ -130,7 +132,7 @@ public class LispSouthboundHandler extends SimpleChannelInboundHandler<DatagramP
             TransportAddressBuilder transportAddressBuilder = new TransportAddressBuilder();
             transportAddressBuilder.setIpAddress(
                     LispNotificationHelper.getIpAddressBinaryFromInetAddress(finalSourceAddress));
-            transportAddressBuilder.setPort(new PortNumber(port));
+            transportAddressBuilder.setPort(new PortNumber(fromJava(port)));
             requestMappingBuilder.setTransportAddress(transportAddressBuilder.build());
             lispSbPlugin.sendNotificationIfPossible(requestMappingBuilder.build());
         } catch (RuntimeException re) {
@@ -204,7 +206,7 @@ public class LispSouthboundHandler extends SimpleChannelInboundHandler<DatagramP
                 TransportAddressBuilder transportAddressBuilder = new TransportAddressBuilder();
                 transportAddressBuilder.setIpAddress(LispNotificationHelper.getIpAddressBinaryFromInetAddress(
                         sourceAddress));
-                transportAddressBuilder.setPort(new PortNumber(port));
+                transportAddressBuilder.setPort(new PortNumber(fromJava(port)));
                 addMappingBuilder.setTransportAddress(transportAddressBuilder.build());
                 lispSbPlugin.sendNotificationIfPossible(addMappingBuilder.build());
                 if (artificialEntry != null) {
@@ -310,7 +312,7 @@ public class LispSouthboundHandler extends SimpleChannelInboundHandler<DatagramP
                                   MapRegisterCacheValue mapRegisterValue) {
         if (mapRegisterValue.getMappingAuthkey().getKeyType() != null) {
             ByteBuffer outBuffer = transformMapRegisterToMapNotify(inBuffer);
-            if (mapRegisterValue.getMappingAuthkey().getKeyType() != 0) {
+            if (mapRegisterValue.getMappingAuthkey().getKeyType().toJava() != 0) {
                 outBuffer = calculateAndSetNewMAC(outBuffer, mapRegisterValue.getMappingAuthkey().getKeyString());
             }
             outBuffer.position(0);
@@ -406,7 +408,7 @@ public class LispSouthboundHandler extends SimpleChannelInboundHandler<DatagramP
             TransportAddressBuilder transportAddressBuilder = new TransportAddressBuilder();
             transportAddressBuilder.setIpAddress(LispNotificationHelper
                     .getIpAddressBinaryFromInetAddress(sourceAddress));
-            transportAddressBuilder.setPort(new PortNumber(port));
+            transportAddressBuilder.setPort(new PortNumber(fromJava(port)));
             gotMapNotifyBuilder.setTransportAddress(transportAddressBuilder.build());
             lispSbPlugin.sendNotificationIfPossible(gotMapNotifyBuilder.build());
         } catch (RuntimeException re) {
@@ -426,7 +428,7 @@ public class LispSouthboundHandler extends SimpleChannelInboundHandler<DatagramP
             TransportAddressBuilder transportAddressBuilder = new TransportAddressBuilder();
             transportAddressBuilder.setIpAddress(LispNotificationHelper
                     .getIpAddressBinaryFromInetAddress(sourceAddress));
-            transportAddressBuilder.setPort(new PortNumber(port));
+            transportAddressBuilder.setPort(new PortNumber(fromJava(port)));
             gotMapReplyBuilder.setTransportAddress(transportAddressBuilder.build());
             lispSbPlugin.sendNotificationIfPossible(gotMapReplyBuilder.build());
         } catch (RuntimeException re) {