Bump upstreams 71/106571/8
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 20 Jun 2023 23:57:05 +0000 (01:57 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 5 Jul 2023 23:23:55 +0000 (01:23 +0200)
Adopt:
- odparent-13.0.3
- infrautils-6.0.1
- yangtools-11.0.0
- mdsal-12.0.0
- controller-8.0.0
- aaa-0.18.0
- netconf-6.0.0

Change-Id: Ib9356fbcefafbdc2a72d849c33bc433efcd4ab4d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
130 files changed:
applications/arbitratorreconciliation/pom.xml
applications/device-ownership-service/pom.xml
applications/forwardingrules-manager/src/test/java/test/mock/util/AbstractFRMTest.java
applications/pom.xml
applications/topology-manager/pom.xml
artifacts/pom.xml
distribution/karaf/pom.xml
extension/features-extension-aggregator/features-openflowplugin-extension/pom.xml
extension/features-extension-aggregator/odl-openflowplugin-eric-extensions/pom.xml
extension/features-extension-aggregator/odl-openflowplugin-nxm-extensions/pom.xml
extension/features-extension-aggregator/odl-openflowplugin-onf-extensions/pom.xml
extension/features-extension-aggregator/pom.xml
extension/openflowjava-extension-nicira/src/main/java/org/opendaylight/openflowjava/nx/codec/action/ConntrackCodec.java
extension/openflowjava-extension-nicira/src/main/java/org/opendaylight/openflowjava/nx/codec/match/ArpShaCodec.java
extension/openflowjava-extension-nicira/src/main/java/org/opendaylight/openflowjava/nx/codec/match/ArpThaCodec.java
extension/openflowjava-extension-nicira/src/main/java/org/opendaylight/openflowjava/nx/codec/match/EthDstCodec.java
extension/openflowjava-extension-nicira/src/main/java/org/opendaylight/openflowjava/nx/codec/match/EthSrcCodec.java
extension/openflowjava-extension-nicira/src/test/java/org/opendaylight/openflowjava/nx/codec/action/ConntrackCodecTest.java
extension/openflowjava-extension-nicira/src/test/java/org/opendaylight/openflowjava/nx/codec/match/ArpShaCodecTest.java
extension/openflowjava-extension-nicira/src/test/java/org/opendaylight/openflowjava/nx/codec/match/ArpThaCodecTest.java
extension/openflowjava-extension-nicira/src/test/java/org/opendaylight/openflowjava/nx/codec/match/EthDstCodecTest.java
extension/openflowjava-extension-nicira/src/test/java/org/opendaylight/openflowjava/nx/codec/match/EthSrcCodecTest.java
extension/openflowplugin-extension-nicira/src/main/java/org/opendaylight/openflowplugin/extension/vendor/nicira/convertor/match/MatchUtil.java
extension/openflowplugin-extension-onf/pom.xml
extension/pom.xml
features-aggregator/features-openflowplugin/pom.xml
features-aggregator/odl-openflowplugin-app-arbitratorreconciliation/pom.xml
features-aggregator/odl-openflowplugin-app-bulk-o-matic/pom.xml
features-aggregator/odl-openflowplugin-app-config-pusher/pom.xml
features-aggregator/odl-openflowplugin-app-forwardingrules-manager/pom.xml
features-aggregator/odl-openflowplugin-app-forwardingrules-sync/pom.xml
features-aggregator/odl-openflowplugin-app-lldp-speaker/pom.xml
features-aggregator/odl-openflowplugin-app-reconciliation-framework/pom.xml
features-aggregator/odl-openflowplugin-app-southbound-cli/pom.xml
features-aggregator/odl-openflowplugin-app-table-miss-enforcer/pom.xml
features-aggregator/odl-openflowplugin-app-topology-lldp-discovery/pom.xml
features-aggregator/odl-openflowplugin-app-topology-manager/pom.xml
features-aggregator/odl-openflowplugin-app-topology/pom.xml
features-aggregator/odl-openflowplugin-drop-test/pom.xml
features-aggregator/odl-openflowplugin-flow-services-rest/pom.xml
features-aggregator/odl-openflowplugin-flow-services/pom.xml
features-aggregator/odl-openflowplugin-libraries/pom.xml
features-aggregator/odl-openflowplugin-libraries/src/main/feature/feature.xml
features-aggregator/odl-openflowplugin-nsf-model/pom.xml
features-aggregator/odl-openflowplugin-nsf-model/src/main/feature/feature.xml
features-aggregator/odl-openflowplugin-southbound/pom.xml
features-aggregator/odl-openflowplugin-southbound/src/main/feature/feature.xml
features-aggregator/pom.xml
libraries/liblldp/pom.xml
libraries/pom.xml
model/pom.xml
openflowjava/features-openflowjava-aggregator/features-openflowjava/pom.xml
openflowjava/features-openflowjava-aggregator/odl-openflowjava-protocol/pom.xml
openflowjava/features-openflowjava-aggregator/odl-openflowjava-protocol/src/main/feature/feature.xml
openflowjava/features-openflowjava-aggregator/pom.xml
openflowjava/openflow-protocol-impl/pom.xml
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/SwitchConnectionProviderImpl.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/deserialization/match/OxmDeserializerHelper.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/action/OF10SetDlDstActionSerializer.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/action/OF10SetDlSrcActionSerializer.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/action/OF10SetNwDstActionSerializer.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/action/OF10SetNwSrcActionSerializer.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/MultipartReplyMessageFactory.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/OF10FeaturesReplyMessageFactory.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/OF10PortModInputMessageFactory.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/OF10PortStatusMessageFactory.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/PortModInputMessageFactory.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/PortStatusMessageFactory.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/match/AbstractOxmIpv4AddressSerializer.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/match/AbstractOxmMacAddressSerializer.java
openflowjava/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/OF10MatchSerializer.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/core/connection/SwitchConnectionProviderImplTest.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/MultipartReplyMessageFactoryTest.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/OF10FeaturesReplyMessageFactoryTest.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/OF10PortModInputMessageFactoryTest.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/OF10PortStatusMessageFactoryTest.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/PortModInputMessageFactoryTest.java
openflowjava/openflow-protocol-impl/src/test/java/org/opendaylight/openflowjava/protocol/impl/serialization/factories/PortStatusMessageFactoryTest.java
openflowjava/openflow-protocol-it/src/test/java/org/opendaylight/openflowjava/protocol/it/integration/IntegrationTest.java
openflowjava/openflowjava-util/src/main/java/org/opendaylight/openflowjava/util/ByteBufUtils.java
openflowplugin-impl/pom.xml
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/DefaultDiagStatusProvider.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/AbstractMacAddressFilterEntrySerializer.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/AbstractMatchEntrySerializer.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/protocol/serialization/messages/PortMessageSerializer.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/util/AddressNormalizationUtil.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/util/DeviceInitializationUtil.java
openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/util/FlowUtil.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/ArpSourceHardwareAddressEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/ArpSourceTransportAddressEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/ArpTargetHardwareAddressEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/ArpTargetTransportAddressEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/EthernetDestinationEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/EthernetSourceEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv4DestinationEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv4SourceEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv6DestinationEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv6NdSllEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv6NdTargetEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv6NdTllEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/match/Ipv6SourceEntryDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/deserialization/messages/PortMessageDeserializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/actions/SetDlDstActionSerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/actions/SetDlSrcActionSerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/ArpSourceHardwareAddressEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/ArpTargetHardwareAddressEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/EthernetDestinationEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/EthernetSourceEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6ArbitraryBitMaskDestinationEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6ArbitraryBitMaskSourceEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6DestinationEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6NdSllEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6NdTargetEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6NdTllEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/match/Ipv6SourceEntrySerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/protocol/serialization/messages/PortMessageSerializerTest.java
openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/util/DeviceInitializationUtilTest.java
openflowplugin/pom.xml
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/common/IpConversionUtil.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/MatchConvertor.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/MatchV10ResponseConvertor.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/cases/OfToSalArpShaCase.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/cases/OfToSalArpThaCase.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/cases/OfToSalEthDstCase.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/cases/OfToSalEthSrcCase.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/cases/SalToOfArpMatchCase.java
parent/pom.xml
samples/pom.xml
test-common/src/main/java/org/opendaylight/openflowplugin/testcommon/AbstractDropTest.java
test-provider/pom.xml

index 7835b9db13774402d45c2762ed4c5b229e259193..3108a1205d3e0bb365df8426984523f7b28648f3 100644 (file)
@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 1038225409a42e1ac26d808e45479f2c32c3f52e..fc7d02ce03357d9d6abda215eef6599292168302 100644 (file)
             <groupId>org.opendaylight.mdsal</groupId>
             <artifactId>mdsal-eos-binding-api</artifactId>
         </dependency>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <optional>true</optional>
+        </dependency>
         <dependency>
             <groupId>com.guicedee.services</groupId>
             <artifactId>javax.inject</artifactId>
index 5796c24cde1adbc935391177af82092113316e00..092c234a10a6068c06ef69f5f42209da5c075659 100644 (file)
@@ -104,7 +104,7 @@ public abstract class AbstractFRMTest extends AbstractDataBrokerTest {
         Nodes nodes = new NodesBuilder()
             .setNode(BindingMap.of(new NodeBuilder()
                 .withKey(nodeKey)
-                .addAugmentation(new FlowCapableNodeBuilder().build())
+                .addAugmentation(new FlowCapableNodeBuilder().setDescription("test node").build())
                 .build()))
             .build();
 
index 957bf6aab231bafb3ea8ff4c2dcf5e79f4fa6c61..accafb0fd5319dbef0168633521cbb1a8337e994 100644 (file)
@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index cf8efbbc831570a8705191195a8caeb0665ef501..d346fe0408d3cca597a0e5ebd0419237cccf0f80 100644 (file)
       <artifactId>model-topology</artifactId>
     </dependency>
     <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.mockito</groupId>
-      <artifactId>mockito-core</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>slf4j-simple</artifactId>
-      <scope>test</scope>
+      <groupId>com.github.spotbugs</groupId>
+      <artifactId>spotbugs-annotations</artifactId>
+      <optional>true</optional>
     </dependency>
     <dependency>
       <groupId>com.guicedee.services</groupId>
index 069db69c4b768760be81c250f76b03354716f8c6..d5ed22ff218bcfcdfd86d453f7641b1a58458fe6 100644 (file)
@@ -13,7 +13,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>12.0.3</version>
+    <version>13.0.3</version>
     <relativePath/>
   </parent>
 
index 5515aa78e18febdac8e4e45d9fe64d6d0d5b2d67..7dfc1f6971d10d867d5a8025d3a678a9f8fbfad2 100644 (file)
@@ -4,7 +4,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>karaf4-parent</artifactId>
-    <version>12.0.3</version>
+    <version>13.0.3</version>
   </parent>
   <groupId>org.opendaylight.openflowplugin</groupId>
   <artifactId>openflowplugin-karaf</artifactId>
index ea1638da74f112d872a02753321d66e83756efab..a6e9d14044694d0eb62a7c76b0aa109ef165d4f8 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>feature-repo-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 138837ca47a3559dc7a942b4f143eb7511ac97f4..7d644eb79051eed1369decc0dea855a79cd198db 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 76a5f32cb943c6354a1dc635dd45fb0b6df1d112..dd92874514dd33d1406fd44314ded6c4a0110bb1 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 2da3cc455296c9f4f5763b75d6bcc0ea5bd647b1..7ab5dfc0a16413cb650745b368b09ea3ecec4db1 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index f9b4113f594261487166b968e2ddd0cf4bbaf464..543df66ed13351c6339e1221399faab48e140041 100644 (file)
@@ -3,7 +3,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
     <modelVersion>4.0.0</modelVersion>
index 503db82d3126696057b1f715c3813f4541bda6b7..a44a4c865493fb8d56d075dc4c02e7a4099098c9 100644 (file)
@@ -261,12 +261,12 @@ public class ConntrackCodec extends AbstractActionCodec {
     }
 
     private static IpAddress readIpv4Address(final ByteBuf message) {
-        return new IpAddress(IetfInetUtil.INSTANCE.ipv4AddressFor(message.readInt()));
+        return new IpAddress(IetfInetUtil.ipv4AddressFor(message.readInt()));
     }
 
     private static void writeIpv4Address(final ByteBuf outBuffer, final IpAddress ipAddress) {
         if (ipAddress != null) {
-            outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(ipAddress.getIpv4Address()));
+            outBuffer.writeBytes(IetfInetUtil.ipv4AddressBytes(ipAddress.getIpv4Address()));
         }
     }
 
index 9321608145927296f14e093c46c5b98f6f826945..84865fc46df851c080b80c9b3d450a97685dab9e 100644 (file)
@@ -35,7 +35,7 @@ public class ArpShaCodec extends AbstractMatchCodec {
     public void serialize(final MatchEntry input, final ByteBuf outBuffer) {
         serializeHeader(input, outBuffer);
         ArpShaCaseValue value = (ArpShaCaseValue) input.getMatchEntryValue();
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(value.getArpShaValues().getMacAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(value.getArpShaValues().getMacAddress()));
     }
 
     @Override
@@ -45,7 +45,7 @@ public class ArpShaCodec extends AbstractMatchCodec {
         message.readBytes(address);
         ArpShaCaseValueBuilder caseBuilder = new ArpShaCaseValueBuilder();
         caseBuilder.setArpShaValues(new ArpShaValuesBuilder()
-            .setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(address))
+            .setMacAddress(IetfYangUtil.macAddressFor(address))
             .build());
         matchEntriesBuilder.setMatchEntryValue(caseBuilder.build());
         return matchEntriesBuilder.build();
index 11c6e1562d3f0ce859414678fcfd8298cd8fea53..8b61b34d6cebee9cae661f0b08ef6d76684cac7e 100644 (file)
@@ -35,7 +35,7 @@ public class ArpThaCodec extends AbstractMatchCodec {
     public void serialize(final MatchEntry input, final ByteBuf outBuffer) {
         serializeHeader(input, outBuffer);
         ArpThaCaseValue arpThaCase = (ArpThaCaseValue) input.getMatchEntryValue();
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpThaCase.getArpThaValues().getMacAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(arpThaCase.getArpThaValues().getMacAddress()));
     }
 
     @Override
@@ -45,7 +45,7 @@ public class ArpThaCodec extends AbstractMatchCodec {
         message.readBytes(address);
         ArpThaCaseValueBuilder caseBuilder = new ArpThaCaseValueBuilder();
         caseBuilder.setArpThaValues(new ArpThaValuesBuilder()
-            .setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(address))
+            .setMacAddress(IetfYangUtil.macAddressFor(address))
             .build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
         return matchEntryBuilder.build();
index 2e677eba6466d0c81b89624b25685cd367c36c89..de16173467b62d534c9451ccb4b123481213271a 100644 (file)
@@ -35,7 +35,7 @@ public class EthDstCodec extends AbstractMatchCodec {
     public void serialize(final MatchEntry input, final ByteBuf outBuffer) {
         serializeHeader(input, outBuffer);
         EthDstCaseValue ethDstCase = (EthDstCaseValue) input.getMatchEntryValue();
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethDstCase.getEthDstValues().getMacAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(ethDstCase.getEthDstValues().getMacAddress()));
     }
 
     @Override
@@ -45,7 +45,7 @@ public class EthDstCodec extends AbstractMatchCodec {
         message.readBytes(address);
         EthDstCaseValueBuilder caseBuilder = new EthDstCaseValueBuilder();
         caseBuilder.setEthDstValues(new EthDstValuesBuilder()
-            .setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(address))
+            .setMacAddress(IetfYangUtil.macAddressFor(address))
             .build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
         return matchEntryBuilder.build();
index 4489c69ab099f6661e8e6bfbace1d14149cb4db2..f90aa6848792db9e335c9f5a9f6e30701df9143a 100644 (file)
@@ -35,7 +35,7 @@ public class EthSrcCodec extends AbstractMatchCodec {
     public void serialize(final MatchEntry input, final ByteBuf outBuffer) {
         serializeHeader(input, outBuffer);
         EthSrcCaseValue ethSrcCase = (EthSrcCaseValue) input.getMatchEntryValue();
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethSrcCase.getEthSrcValues().getMacAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(ethSrcCase.getEthSrcValues().getMacAddress()));
     }
 
     @Override
@@ -45,7 +45,7 @@ public class EthSrcCodec extends AbstractMatchCodec {
         message.readBytes(address);
         EthSrcCaseValueBuilder caseBuilder = new EthSrcCaseValueBuilder();
         caseBuilder.setEthSrcValues(new EthSrcValuesBuilder()
-            .setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(address))
+            .setMacAddress(IetfYangUtil.macAddressFor(address))
             .build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
         matchEntryBuilder.setHasMask(false);
index 6a5fc1b7af614d4eaea6ae57f3aab26054de5114..e048d645d452ff693e0f80b594962b63752adb2c 100644 (file)
@@ -264,9 +264,9 @@ public class ConntrackCodecTest {
         //RANGE PRESENT
         message.writeShort(0x3F);
         //IP ADDRESS MIN
-        message.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address("192.168.0.0")));
+        message.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address("192.168.0.0")));
         //IP ADDRESS MAX
-        message.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address("192.168.10.0")));
+        message.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address("192.168.10.0")));
         //PORT MIN
         message.writeShort(3000);
         //PORT MAX
@@ -283,7 +283,7 @@ public class ConntrackCodecTest {
         //RANGE PRESENT
         message.writeShort(0x21);
         //IP ADDRESS MIN
-        message.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address("192.168.0.0")));
+        message.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address("192.168.0.0")));
         //PORT MAX
         message.writeShort(4000);
         message.writeZero(2);
index da50e47e861d59e95182ddbaba7a24ae30fe8753..2d908fa1c24254e1abd9b48bb181ca9970b1cb4e 100644 (file)
@@ -34,7 +34,7 @@ public class ArpShaCodecTest {
     private static final int NXM_FIELD_CODE = 17;
 
     private static byte[] resAddress = new byte[VALUE_LENGTH];
-    private static final MacAddress RESULT_ADDRESS = IetfYangUtil.INSTANCE.macAddressFor(resAddress);
+    private static final MacAddress RESULT_ADDRESS = IetfYangUtil.macAddressFor(resAddress);
 
     @Before
     public void setUp() {
@@ -81,7 +81,7 @@ public class ArpShaCodecTest {
         matchEntryBuilder.setHasMask(false);
 
 
-        valuesBuilder.setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(new byte[VALUE_LENGTH]));
+        valuesBuilder.setMacAddress(IetfYangUtil.macAddressFor(new byte[VALUE_LENGTH]));
 
         caseBuilder.setArpShaValues(valuesBuilder.build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
index 73734928345df5dd2d4dc75f620993405353d8d7..009bd9eb8df2fe6f1da51b86fa873d16ec4672d8 100644 (file)
@@ -35,7 +35,7 @@ public class ArpThaCodecTest {
     private static final int NXM_FIELD_CODE = 18;
 
     private static final byte[] TEST_ADDR = new byte[VALUE_LENGTH];
-    private static final MacAddress TEST_ADDRESS = IetfYangUtil.INSTANCE.macAddressFor(TEST_ADDR);
+    private static final MacAddress TEST_ADDRESS = IetfYangUtil.macAddressFor(TEST_ADDR);
 
     @Before
     public void setUp() {
@@ -78,7 +78,7 @@ public class ArpThaCodecTest {
         matchEntryBuilder.setOxmMatchField(NxmNxArpTha.VALUE);
         matchEntryBuilder.setHasMask(false);
 
-        valuesBuilder.setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(new byte[VALUE_LENGTH]));
+        valuesBuilder.setMacAddress(IetfYangUtil.macAddressFor(new byte[VALUE_LENGTH]));
 
         caseBuilder.setArpThaValues(valuesBuilder.build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
index 049c9c62fee1a31d97690892e805f3b46629866d..b4a479800ba55913170e6cfe79bd13ab76dc38e5 100644 (file)
@@ -34,7 +34,7 @@ public class EthDstCodecTest {
     private static final int NXM_FIELD_CODE = 1;
 
     private static final byte[] TEST_ADDR = new byte[VALUE_LENGTH];
-    private static final MacAddress TEST_ADDRESS = IetfYangUtil.INSTANCE.macAddressFor(TEST_ADDR);
+    private static final MacAddress TEST_ADDRESS = IetfYangUtil.macAddressFor(TEST_ADDR);
 
     @Test
     public void serializeTest() {
@@ -72,7 +72,7 @@ public class EthDstCodecTest {
         matchEntryBuilder.setOxmMatchField(NxmOfEthDst.VALUE);
         matchEntryBuilder.setHasMask(false);
 
-        valuesBuilder.setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(new byte[VALUE_LENGTH]));
+        valuesBuilder.setMacAddress(IetfYangUtil.macAddressFor(new byte[VALUE_LENGTH]));
 
         caseBuilder.setEthDstValues(valuesBuilder.build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
index 550659fef28768e50138d3a071ce11c9ed7db6df..435c45b1bca44bd39f87922b11e6619f294b873b 100644 (file)
@@ -35,7 +35,7 @@ public class EthSrcCodecTest {
     private static final int NXM_FIELD_CODE = 2;
 
     private static final byte[] TEST_ADDR = new byte[VALUE_LENGTH];
-    private static final MacAddress TEST_ADDRESS = IetfYangUtil.INSTANCE.macAddressFor(TEST_ADDR);
+    private static final MacAddress TEST_ADDRESS = IetfYangUtil.macAddressFor(TEST_ADDR);
 
     @Test
     public void serializeTest() {
@@ -74,7 +74,7 @@ public class EthSrcCodecTest {
         matchEntryBuilder.setHasMask(false);
 
 
-        valuesBuilder.setMacAddress(IetfYangUtil.INSTANCE.macAddressFor(new byte[VALUE_LENGTH]));
+        valuesBuilder.setMacAddress(IetfYangUtil.macAddressFor(new byte[VALUE_LENGTH]));
 
         caseBuilder.setEthSrcValues(valuesBuilder.build());
         matchEntryBuilder.setMatchEntryValue(caseBuilder.build());
index 68f0aa4c1513353310adf618be420116d75b4acb..a43523e74a465f9f02223d43dfdfdd8d9ed0d189 100644 (file)
@@ -181,10 +181,10 @@ public final class MatchUtil {
     }
 
     public static Uint32 ipv4ToUint32(final Ipv4Address ipv4) {
-        return Uint32.fromIntBits(IetfInetUtil.INSTANCE.ipv4AddressBits(ipv4));
+        return Uint32.fromIntBits(IetfInetUtil.ipv4AddressBits(ipv4));
     }
 
     public static Ipv4Address uint32ToIpv4Address(final Uint32 value) {
-        return IetfInetUtil.INSTANCE.ipv4AddressFor(value.intValue());
+        return IetfInetUtil.ipv4AddressFor(value.intValue());
     }
 }
index 8580f722630f412de41eb22582a75a2e61f3e29a..fac0d7479b8bd1c97121e9676182e9fab21e9fc4 100644 (file)
             <groupId>org.opendaylight.yangtools</groupId>
             <artifactId>yang-common-netty</artifactId>
         </dependency>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <optional>true</optional>
+        </dependency>
     </dependencies>
-
 </project>
index b38f712237a8ca9e43f24a5d7b6d1a246ec8d347..811934652d053f2f62c5448ebd47d42536daa7cd 100644 (file)
@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 8f583d0dc80a36aebde63d01331cff17ac261c22..0fb39c4d8c4f0b9c947db313657b7d3dafa51898 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>feature-repo-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 433e3e44a7392689d70191e1244d4eaef2795b74..00496e79e024d87665bdd64ee91bceac752985e3 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 16620989ff59e869c9dbda8b0b3d0c766ec2a19e..ed7b933cd38644b681a7e28326cd368eee1d0d4c 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 46870cd5f1b08a748cad53fa18847fa0ee07bac9..a9edfec03e6d4626754d18850d30ca6a2b73530a 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index a0956f8fbd37eaae9ee73b07f8cdb65e4bb55997..99770550442b525d5b749b38dde193d634dbf71b 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index d4cd6005661e9fbd201576e4ceb7b09afd11e83b..2c219741adf27a262fd3092337d2fbfd70755122 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index ee5146c60f6705e085e2f86c7613141986d175f5..738bbefe305489dc605a56f284f3593abcfdd1b6 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 196f409bb7787b1c67d1ba364854bbe0b4dd1265..74c82e3e36d94ec044d1a223443080eae635c6fd 100644 (file)
@@ -4,7 +4,7 @@
 <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>12.0.3</version>
+    <version>13.0.3</version>
     <relativePath/>
 </parent>
 
index 5065c66a3989cd2b15b6cac400da1d138a523413..589e3a48fe1ed8469179b643d1f49e2caacdc6f2 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 9785b822c25c9353ca570674b2e6344a2beb4842..06638883f0090f40281f81de2731b68113de5178 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 592721c9016552ddf192b3781d2ce305e5bc40c8..6f390a1ec17ce070e5e668113fc22ab6a0cb6acf 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 61aff09b032867a932a96cda3d27ae6c1da0adfd..6d72cf79d77031bd3798e8cd1c9ec086a774f566 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 84c65c4210cc8237fdf828e07778d126dc5f40ef..1d9c4317fb7b78499c7d3444f43e99c58114de48 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 037ed66ca25b39d20fc1c90851044c60d2af22ce..56080bd786a1325f60aac39bfc09fe86ca7b4693 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index c220f38bb4d885c518e77d035acc82d111f06a27..b037acb8409978675c67f7046507fbc57eb9eb10 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
@@ -21,7 +21,7 @@
             <dependency>
                 <groupId>org.opendaylight.netconf</groupId>
                 <artifactId>netconf-artifacts</artifactId>
-                <version>5.0.4</version>
+                <version>6.0.0</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
index 9c9ed83ce6d837c169b8654acd5ae054fb4f01fb..3c6b27d56daf22f77f437eb581bce107099c0cab 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index bc43c5f0dc4be0fd463cce802ef2f0c2524ee12a..40741cf4029805964a5dc555c1d2141f3d60f392 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index a53fc7f3f7bed33ff8fb6312352344a27b5557bb..dbe0cfe08269137ca944e2936577b5d8e9fad9ad 100644 (file)
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
 <features xmlns="http://karaf.apache.org/xmlns/features/v1.4.0" name="odl-openflowplugin-libraries">
     <feature name="odl-openflowplugin-libraries">
-        <feature version="[12,13)">odl-apache-commons-lang3</feature>
-        <feature version="[12,13)">odl-guava</feature>
+        <feature version="[13,14)">odl-apache-commons-lang3</feature>
+        <feature version="[13,14)">odl-guava</feature>
     </feature>
 </features>
index 9ee79ab6af4a6efa45a6a1e2ee6bcfce967bc323..b3161f3058aab7499af7a301ad05e942ea4ca41f 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
@@ -21,7 +21,7 @@
             <dependency>
                 <groupId>org.opendaylight.mdsal</groupId>
                 <artifactId>mdsal-artifacts</artifactId>
-                <version>11.0.7</version>
+                <version>12.0.0</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
@@ -30,7 +30,7 @@
             <dependency>
                 <groupId>org.opendaylight.controller</groupId>
                 <artifactId>controller-artifacts</artifactId>
-                <version>7.0.4</version>
+                <version>8.0.0</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
index 55968b81c8b21195c619d0e3cd1615ac9ba80e46..08fba54e14a487782d9d8afbc801dcb8df7685b5 100644 (file)
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <features name="odl-openflowplugin-nsf-model-${project.version}" xmlns="http://karaf.apache.org/xmlns/features/v1.2.0">
     <feature name="odl-openflowplugin-nsf-model" version="${project.version}">
-        <feature version="[11,12)">odl-mdsal-model-draft-clemm-netmod-yang-network-topo-01-minimal</feature>
-        <feature version="[11,12)">odl-mdsal-model-odl-l2-types</feature>
+        <feature version="[12,13)">odl-mdsal-model-draft-clemm-netmod-yang-network-topo-01-minimal</feature>
+        <feature version="[12,13)">odl-mdsal-model-odl-l2-types</feature>
     </feature>
 </features>
index 3db02a9331dc57f310040e26c5e68877fb813fb9..9bf811c8cd3f32b2187522570a293048e9ff06bb 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
             <dependency>
                 <groupId>org.opendaylight.controller</groupId>
                 <artifactId>controller-artifacts</artifactId>
-                <version>7.0.4</version>
+                <version>8.0.0</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
             <dependency>
                 <groupId>org.opendaylight.infrautils</groupId>
                 <artifactId>infrautils-artifacts</artifactId>
-                <version>5.0.3</version>
+                <version>6.0.1</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
index 376e47685acd6986c231e783c3d1b8bc152015e8..d09aad69054ef62aa1a449ca92f1d0d63c4c367e 100644 (file)
@@ -2,7 +2,7 @@
 <features name="openflowplugin-${project.version}" xmlns="http://karaf.apache.org/xmlns/features/v1.2.0">
     <feature name="odl-openflowplugin-southbound" version="${project.version}">
         <configfile finalname="etc/org.opendaylight.openflowplugin.cfg">mvn:org.opendaylight.openflowplugin/openflowplugin-blueprint-config/${project.version}/cfg/config</configfile>
-        <feature version="[7,8)">odl-mdsal-broker</feature>
-        <feature version="[5,6)">odl-infrautils-diagstatus</feature>
+        <feature version="[8,9)">odl-mdsal-broker</feature>
+        <feature version="[6,7)">odl-infrautils-diagstatus</feature>
     </feature>
 </features>
index c97bbcfac70bac05d2ccebaebcdc2b0e7252b01a..90f9f56f0ad595b28eb2222f70b0429664d9e639 100644 (file)
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 7e7dffadb0e5e877356da3fd132cbf0c628134d0..cfc3dc72f8501aa95140ee2e86041e9dc04c3d3d 100644 (file)
   <packaging>bundle</packaging>
 
   <dependencies>
+    <dependency>
+      <groupId>com.github.spotbugs</groupId>
+      <artifactId>spotbugs-annotations</artifactId>
+      <optional>true</optional>
+    </dependency>
     <dependency>
       <groupId>com.google.guava</groupId>
       <artifactId>guava</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.apache.commons</groupId>
-      <artifactId>commons-lang3</artifactId>
+      <groupId>jakarta.xml.bind</groupId>
+      <artifactId>jakarta.xml.bind-api</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>slf4j-simple</artifactId>
-      <scope>test</scope>
+      <groupId>org.apache.commons</groupId>
+      <artifactId>commons-lang3</artifactId>
     </dependency>
     <dependency>
-      <groupId>jakarta.xml.bind</groupId>
-      <artifactId>jakarta.xml.bind-api</artifactId>
+      <groupId>org.eclipse.jdt</groupId>
+      <artifactId>org.eclipse.jdt.annotation</artifactId>
     </dependency>
   </dependencies>
 
index 1d3b46eeb0176798b777b76807d664635e246e24..94303f46b9a0d54c40b4694f3cd13260b1e3b6b8 100644 (file)
@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 5ced1db7a423828aa65f6370440e8305f9aa232c..a799b072b6433f5dca347d32067002844cd52c87 100644 (file)
@@ -6,7 +6,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>12.0.3</version>
+    <version>13.0.3</version>
     <relativePath/>
   </parent>
 
index a1bb289f5ed521f8770264522b0d01b35769f892..6afeea835f7fcde6ddb3fbbc9fb270ec1c96b558 100644 (file)
@@ -4,7 +4,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>feature-repo-parent</artifactId>
-    <version>12.0.3</version>
+    <version>13.0.3</version>
     <relativePath/>
   </parent>
 
index 6cac39c0b46e3b8d575420b13b2344e0adfa1ddb..c06e8ba8bd849820237b86b12cb8e66f09283abe 100644 (file)
@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>single-feature-parent</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
@@ -33,7 +33,7 @@
             <dependency>
                 <groupId>org.opendaylight.infrautils</groupId>
                 <artifactId>infrautils-artifacts</artifactId>
-                <version>5.0.3</version>
+                <version>6.0.1</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
@@ -42,7 +42,7 @@
             <dependency>
                 <groupId>org.opendaylight.yangtools</groupId>
                 <artifactId>yangtools-artifacts</artifactId>
-                <version>10.0.4</version>
+                <version>11.0.0</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
@@ -51,7 +51,7 @@
             <dependency>
                 <groupId>org.opendaylight.mdsal</groupId>
                 <artifactId>mdsal-artifacts</artifactId>
-                <version>11.0.7</version>
+                <version>12.0.0</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
index fdfd14f9d41ecb907d7f0fa842073908e7b8d3d9..687ffb891c173a8678823ccc5a0f3c49d0923e32 100644 (file)
@@ -4,8 +4,8 @@
     <feature name='odl-openflowjava-protocol' version='${project.version}'>
         <configfile finalname="etc/opendaylight/datastore/initial/config/default-openflow-connection-config.xml">mvn:org.opendaylight.openflowplugin.openflowjava/openflowjava-blueprint-config/${project.version}/xml/config</configfile>
         <configfile finalname="etc/opendaylight/datastore/initial/config/legacy-openflow-connection-config.xml">mvn:org.opendaylight.openflowplugin.openflowjava/openflowjava-blueprint-config/${project.version}/xml/legacyConfig</configfile>
-        <feature version="[10,11)">odl-yangtools-netty</feature>
-        <feature version="[11,12)">odl-mdsal-model-rfc6991</feature>
-        <feature version="[5,6)">odl-infrautils-diagstatus</feature>
+        <feature version="[11,12)">odl-yangtools-netty</feature>
+        <feature version="[12,13)">odl-mdsal-model-rfc6991</feature>
+        <feature version="[6,7)">odl-infrautils-diagstatus</feature>
     </feature>
 </features>
index 8f051a26d1830f911042bf40329c87a2fd1b17b2..88d5ed4d2937fd24084b1339905cc11581f7e185 100644 (file)
@@ -4,7 +4,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>12.0.3</version>
+    <version>13.0.3</version>
     <relativePath/>
   </parent>
 
index 741e4ea915aac6f56810f2994af71a1eef37f46f..d6a2b2ab7b4a83e216a216d1abad1e9f8c46faee 100644 (file)
             <groupId>org.kohsuke.metainf-services</groupId>
             <artifactId>metainf-services</artifactId>
         </dependency>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <optional>true</optional>
+        </dependency>
         <dependency>
             <groupId>com.guicedee.services</groupId>
             <artifactId>javax.inject</artifactId>
index d5c72a848783d47fdc2a2d415d4338eaa401aea7..923a9e8842ad5d9285ae70b5b55cf791de380195 100755 (executable)
@@ -5,12 +5,9 @@
  * 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.openflowjava.protocol.impl.core;
 
 import static com.google.common.base.Preconditions.checkState;
-import static java.util.Objects.requireNonNull;
 
 import com.google.common.util.concurrent.FutureCallback;
 import com.google.common.util.concurrent.Futures;
@@ -83,7 +80,6 @@ public class SwitchConnectionProviderImpl implements SwitchConnectionProvider, C
     private final DeserializerRegistry deserializerRegistry;
     private final DeserializationFactory deserializationFactory;
     private final ListeningExecutorService listeningExecutorService;
-    private final DiagStatusService diagStatus;
     private final String diagStatusIdentifier;
     private final String threadName;
 
@@ -95,7 +91,6 @@ public class SwitchConnectionProviderImpl implements SwitchConnectionProvider, C
             final @Nullable ConnectionConfiguration connConfig) {
         this.connConfig = connConfig;
         String connectionSuffix = createConnectionSuffix(connConfig);
-        this.diagStatus = requireNonNull(diagStatus);
         diagStatusIdentifier = OPENFLOW_JAVA_SERVICE_NAME_PREFIX + connectionSuffix;
         diagReg = diagStatus.register(diagStatusIdentifier);
 
@@ -147,12 +142,12 @@ public class SwitchConnectionProviderImpl implements SwitchConnectionProvider, C
             Futures.addCallback(listeningExecutorService.submit(serverFacade), new FutureCallback<Object>() {
                 @Override
                 public void onFailure(final Throwable throwable) {
-                    diagStatus.report(new ServiceDescriptor(diagStatusIdentifier, throwable));
+                    diagReg.report(new ServiceDescriptor(diagStatusIdentifier, throwable));
                 }
 
                 @Override
                 public void onSuccess(final Object result) {
-                    diagStatus.report(new ServiceDescriptor(diagStatusIdentifier, ServiceState.ERROR,
+                    diagReg.report(new ServiceDescriptor(diagStatusIdentifier, ServiceState.ERROR,
                         threadName + " terminated"));
                 }
             }, MoreExecutors.directExecutor());
@@ -183,7 +178,7 @@ public class SwitchConnectionProviderImpl implements SwitchConnectionProvider, C
 
         if (TransportProtocol.TCP.equals(transportProtocol) || TransportProtocol.TLS.equals(transportProtocol)) {
             server = new TcpHandler(connConfig.getAddress(), connConfig.getPort(),
-                () -> diagStatus.report(new ServiceDescriptor(diagStatusIdentifier, ServiceState.OPERATIONAL)));
+                () -> diagReg.report(new ServiceDescriptor(diagStatusIdentifier, ServiceState.OPERATIONAL)));
             final TcpChannelInitializer channelInitializer = factory.createPublishingChannelInitializer();
             ((TcpHandler) server).setChannelInitializer(channelInitializer);
             ((TcpHandler) server).initiateEventLoopGroups(connConfig.getThreadConfiguration(), isEpollEnabled);
@@ -193,7 +188,7 @@ public class SwitchConnectionProviderImpl implements SwitchConnectionProvider, C
             connectionInitializer.run();
         } else if (TransportProtocol.UDP.equals(transportProtocol)) {
             server = new UdpHandler(connConfig.getAddress(), connConfig.getPort(),
-                () -> diagStatus.report(new ServiceDescriptor(diagStatusIdentifier, ServiceState.OPERATIONAL)));
+                () -> diagReg.report(new ServiceDescriptor(diagStatusIdentifier, ServiceState.OPERATIONAL)));
             ((UdpHandler) server).initiateEventLoopGroups(connConfig.getThreadConfiguration(), isEpollEnabled);
             ((UdpHandler) server).setChannelInitializer(factory.createUdpChannelInitializer());
         } else {
index ec0805e7a807c97628df97ba64b982411e022c14..c4fde9c666bbcfddab38cb1fdaf865583f0b3f2b 100644 (file)
@@ -44,6 +44,6 @@ public final class OxmDeserializerHelper {
     public static MacAddress convertMacAddress(final ByteBuf input) {
         byte[] address = new byte[EncodeConstants.MAC_ADDRESS_LENGTH];
         input.readBytes(address);
-        return IetfYangUtil.INSTANCE.macAddressFor(address);
+        return IetfYangUtil.macAddressFor(address);
     }
 }
index 58a90bc4cb6f6b193c343914aea4b3e0a1cd9a8e..5e34a0bbf3ea8d9dcae9c861441790505977cd58 100644 (file)
@@ -25,7 +25,7 @@ public class OF10SetDlDstActionSerializer extends AbstractActionSerializer {
 
     @Override
     protected void serializeBody(final Action action, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(((SetDlDstCase) action.getActionChoice())
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(((SetDlDstCase) action.getActionChoice())
                 .getSetDlDstAction().getDlDstAddress()));
         outBuffer.writeZero(ActionConstants.PADDING_IN_DL_ADDRESS_ACTION);
     }
index f612eedeab6a7132836b30a1edc8d4cd04204e56..7eacf4bc846dcd1aa362585718cbdfabe00f2902 100644 (file)
@@ -25,7 +25,7 @@ public class OF10SetDlSrcActionSerializer extends AbstractActionSerializer {
 
     @Override
     protected void serializeBody(final Action action, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(((SetDlSrcCase) action.getActionChoice())
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(((SetDlSrcCase) action.getActionChoice())
                 .getSetDlSrcAction().getDlSrcAddress()));
         outBuffer.writeZero(ActionConstants.PADDING_IN_DL_ADDRESS_ACTION);
     }
index ae1e016f442121b463524bf3f72f0776b2dfdf6b..e29a1acd46cd78173de03fe0126b6f7901f4a756 100644 (file)
@@ -25,7 +25,7 @@ public class OF10SetNwDstActionSerializer extends AbstractActionSerializer {
 
     @Override
     protected void serializeBody(final Action action, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(
+        outBuffer.writeBytes(IetfInetUtil.ipv4AddressBytes(
             ((SetNwDstCase) action.getActionChoice()).getSetNwDstAction().getIpAddress()));
     }
 }
index 95250e65bb55c14283556cf611266660622e83bb..858b147115ac9682a3555ad21854aa88725b99f0 100644 (file)
@@ -25,7 +25,7 @@ public class OF10SetNwSrcActionSerializer extends AbstractActionSerializer {
 
     @Override
     protected void serializeBody(final Action action, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(
+        outBuffer.writeBytes(IetfInetUtil.ipv4AddressBytes(
             ((SetNwSrcCase) action.getActionChoice()).getSetNwSrcAction().getIpAddress()));
     }
 }
index e56d4894c5641b980b84897ee85b26a442e8a65b..0175d5c7e2e95420d8aa3868b5c1bc09e941e007 100644 (file)
@@ -727,7 +727,7 @@ public class MultipartReplyMessageFactory implements OFSerializer<MultipartReply
             outBuffer.writeInt(port.getPortNo().intValue()); // Assuming PortNo
                                                              // = PortId
             outBuffer.writeZero(PORT_DESC_PADDING_1);
-            outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(port.getHwAddr()));
+            outBuffer.writeBytes(IetfYangUtil.macAddressBytes(port.getHwAddr()));
             outBuffer.writeZero(PORT_DESC_PADDING_2);
             writeName(port.getName(), outBuffer);
             writePortConfig(port.getConfig(), outBuffer);
index cae2aef89f5afde2e8222f51c2181107998e244e..c062e93b88e3147c02169f83c88798620fe8e897 100644 (file)
@@ -44,7 +44,7 @@ public class OF10FeaturesReplyMessageFactory implements OFSerializer<GetFeatures
         outBuffer.writeInt(createActionsV10(message.getActionsV10()));
         for (PhyPort port : message.getPhyPort()) {
             outBuffer.writeShort(port.getPortNo().intValue());
-            outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(port.getHwAddr()));
+            outBuffer.writeBytes(IetfYangUtil.macAddressBytes(port.getHwAddr()));
             writeName(port.getName(), outBuffer);
             writePortConfig(port.getConfigV10(), outBuffer);
             writePortState(port.getStateV10(), outBuffer);
index 457e4a38fda7628e9a0a34452e0a41b055402db3..c77cdf69c31c69fdfdb38cc62cce6f3b3cf8df77 100644 (file)
@@ -30,7 +30,7 @@ public class OF10PortModInputMessageFactory implements OFSerializer<PortModInput
     public void serialize(final PortModInput message, final ByteBuf outBuffer) {
         ByteBufUtils.writeOFHeader(MESSAGE_TYPE, message, outBuffer, EncodeConstants.EMPTY_LENGTH);
         outBuffer.writeShort(message.getPortNo().getValue().intValue());
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(message.getHwAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(message.getHwAddress()));
         outBuffer.writeInt(createPortConfigBitmask(message.getConfigV10()));
         outBuffer.writeInt(createPortConfigBitmask(message.getMaskV10()));
         outBuffer.writeInt(createPortFeaturesBitmask(message.getAdvertiseV10()));
index b0629a61ace7792a3b4aa241490b000a814522b4..58bc147d9a69690a90a719e5562b4c296f2d46a4 100644 (file)
@@ -36,7 +36,7 @@ public class OF10PortStatusMessageFactory implements OFSerializer<PortStatusMess
         outBuffer.writeByte(message.getReason().getIntValue());
         outBuffer.writeZero(PADDING);
         outBuffer.writeShort(message.getPortNo().intValue());
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(message.getHwAddr()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(message.getHwAddr()));
         writeName(message.getName(), outBuffer);
         writePortConfig(message.getConfigV10(), outBuffer);
         writePortState(message.getStateV10(), outBuffer);
index b3a173b5be069fd5e6aa51f7dc883527dc43d53b..bded4b0246055f7a4a7377c4e97341b6078390f4 100644 (file)
@@ -36,7 +36,7 @@ public class PortModInputMessageFactory implements OFSerializer<PortMod$G> {
         ByteBufUtils.writeOFHeader(MESSAGE_TYPE, message, outBuffer, EncodeConstants.EMPTY_LENGTH);
         outBuffer.writeInt(message.getPortNo().getValue().intValue());
         outBuffer.writeZero(PADDING_IN_PORT_MOD_MESSAGE_01);
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(message.getHwAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(message.getHwAddress()));
         outBuffer.writeZero(PADDING_IN_PORT_MOD_MESSAGE_02);
         outBuffer.writeInt(createPortConfigBitmask(message.getConfig()));
         outBuffer.writeInt(createPortConfigBitmask(message.getMask()));
index 305be6ce7a440f95ee245d3fb73e87b56d38c11c..ed021061aec12daf0d346df064bf84009ae55d36 100644 (file)
@@ -39,7 +39,7 @@ public class PortStatusMessageFactory implements OFSerializer<PortStatusMessage>
         outBuffer.writeZero(PADDING);
         outBuffer.writeInt(message.getPortNo().intValue());
         outBuffer.writeZero(PORT_PADDING_1);
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(message.getHwAddr()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(message.getHwAddr()));
         outBuffer.writeZero(PORT_PADDING_2);
         writeName(message.getName(), outBuffer);
         writePortConfig(message.getConfig(), outBuffer);
index 764ade36d974d698a33ee3bcdf6d69924a8fe7ed..d76af0810646a3d65cf967a5cdbd964e63fd85b5 100644 (file)
@@ -33,6 +33,6 @@ public abstract class AbstractOxmIpv4AddressSerializer extends AbstractOxmMatchE
     }
 
     protected static void writeIpv4Address(final Ipv4Address address, final ByteBuf out) {
-        out.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(address));
+        out.writeBytes(IetfInetUtil.ipv4AddressBytes(address));
     }
 }
index 4dcd0e2b322e50bdd751d1bd1bbdf5aa6ad18ea6..906e4b7f27b500a4f5e89192604760890dc4313f 100644 (file)
@@ -19,6 +19,6 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.
 public abstract class AbstractOxmMacAddressSerializer extends AbstractOxmMatchEntrySerializer {
 
     protected void writeMacAddress(final MacAddress address, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(address)); // 48 b + mask [OF 1.3.2 spec]
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(address)); // 48 b + mask [OF 1.3.2 spec]
     }
 }
index 32771c0a659d9a3e15ad4b11217febaa2fd70156..91dbf47528c1c6f2f44b692a7573e8d4edd85726 100644 (file)
@@ -37,8 +37,8 @@ public class OF10MatchSerializer implements OFSerializer<MatchV10> {
         outBuffer.writeInt(encodeWildcards(match.getWildcards(), match.getNwSrcMask().toJava(),
                     match.getNwDstMask().toJava()));
         outBuffer.writeShort(match.getInPort().toJava());
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(match.getDlSrc()));
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(match.getDlDst()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(match.getDlSrc()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(match.getDlDst()));
         outBuffer.writeShort(match.getDlVlan().toJava());
         outBuffer.writeByte(match.getDlVlanPcp().toJava());
         outBuffer.writeZero(PADDING_IN_MATCH);
@@ -46,8 +46,8 @@ public class OF10MatchSerializer implements OFSerializer<MatchV10> {
         outBuffer.writeByte(match.getNwTos().toJava());
         outBuffer.writeByte(match.getNwProto().toJava());
         outBuffer.writeZero(PADDING_IN_MATCH_2);
-        outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(match.getNwSrc()));
-        outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(match.getNwDst()));
+        outBuffer.writeBytes(IetfInetUtil.ipv4AddressBytes(match.getNwSrc()));
+        outBuffer.writeBytes(IetfInetUtil.ipv4AddressBytes(match.getNwDst()));
         outBuffer.writeShort(match.getTpSrc().toJava());
         outBuffer.writeShort(match.getTpDst().toJava());
     }
index f80dce40aeaf2afc0702eb2a20d512f38fe0ed0c..972970603a60cacd2615931a241786b34ac0a884 100644 (file)
@@ -12,6 +12,9 @@ import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
 
 import com.google.common.util.concurrent.ListenableFuture;
 import java.net.InetAddress;
@@ -19,11 +22,13 @@ import java.net.UnknownHostException;
 import java.util.List;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
+import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.Mock;
 import org.mockito.junit.MockitoJUnitRunner;
 import org.opendaylight.infrautils.diagstatus.DiagStatusService;
+import org.opendaylight.infrautils.diagstatus.ServiceRegistration;
 import org.opendaylight.openflowjava.protocol.api.connection.SwitchConnectionHandler;
 import org.opendaylight.openflowjava.protocol.api.connection.TlsConfiguration;
 import org.opendaylight.openflowjava.protocol.api.connection.TlsConfigurationImpl;
@@ -51,11 +56,15 @@ public class SwitchConnectionProviderImplTest {
     private SwitchConnectionProviderImpl provider;
     private ConnectionConfigurationImpl config;
 
+    @Before
+    public void before() {
+        doReturn(mock(ServiceRegistration.class)).when(diagStatus).register(any());
+    }
+
     /**
      * Creates new {@link SwitchConnectionProvider} instance for each test.
      * @param protocol communication protocol
      */
-
     public void startUp(final TransportProtocol protocol) throws UnknownHostException {
         config = null;
         if (protocol != null) {
index 6245637bcedebec2bc8f51af1d9f88b22ab78b5f..5585072ad460df05da7ac746cad4e755d9d6485c 100644 (file)
@@ -481,7 +481,7 @@ public class MultipartReplyMessageFactoryTest {
         byte[] address = new byte[6];
         serializedBuffer.readBytes(address);
         Assert.assertEquals("Wrong MacAddress", port.getHwAddr().getValue().toLowerCase(),
-                IetfYangUtil.INSTANCE.macAddressFor(address).getValue().toLowerCase());
+                IetfYangUtil.macAddressFor(address).getValue().toLowerCase());
         serializedBuffer.skipBytes(2);
         byte[] name = new byte[16];
         serializedBuffer.readBytes(name);
index e0437435d1122f33a738291e2b9e1f2a09782b2b..3da16400b318f4ae0450f848d22ed8616ea322e9 100644 (file)
@@ -80,7 +80,7 @@ public class OF10FeaturesReplyMessageFactoryTest {
         byte[] address = new byte[6];
         serializedBuffer.readBytes(address);
         Assert.assertEquals("Wrong MacAddress", port.getHwAddr().getValue().toLowerCase(),
-                IetfYangUtil.INSTANCE.macAddressFor(address).getValue().toLowerCase());
+                IetfYangUtil.macAddressFor(address).getValue().toLowerCase());
         byte[] name = new byte[16];
         serializedBuffer.readBytes(name);
         Assert.assertEquals("Wrong name", port.getName(), new String(name).trim());
index 71733cf71233b58bd384fb322609d90bdb2498bc..876ba9fa10e853620ce6cdde0cbf3e907c5951fd 100644 (file)
@@ -70,7 +70,7 @@ public class OF10PortModInputMessageFactoryTest {
         Assert.assertEquals("Wrong PortNo", message.getPortNo().getValue().longValue(), out.readUnsignedShort());
         byte[] address = new byte[6];
         out.readBytes(address);
-        Assert.assertEquals("Wrong MacAddress", message.getHwAddress(), IetfYangUtil.INSTANCE.macAddressFor(address));
+        Assert.assertEquals("Wrong MacAddress", message.getHwAddress(), IetfYangUtil.macAddressFor(address));
         Assert.assertEquals("Wrong config", 21, out.readUnsignedInt());
         Assert.assertEquals("Wrong mask", 98, out.readUnsignedInt());
         Assert.assertEquals("Wrong advertise", 652, out.readUnsignedInt());
index ad2510373a4d13a795363355d1c03f2542abdc06..0e4ffc2d2287f80db066a46826ca2d8b3499695e 100644 (file)
@@ -74,7 +74,7 @@ public class OF10PortStatusMessageFactoryTest {
         byte[] address = new byte[6];
         serializedBuffer.readBytes(address);
         Assert.assertEquals("Wrong MacAddress", message.getHwAddr().getValue().toLowerCase(),
-                IetfYangUtil.INSTANCE.macAddressFor(address).getValue().toLowerCase());
+                IetfYangUtil.macAddressFor(address).getValue().toLowerCase());
         byte[] name = new byte[16];
         serializedBuffer.readBytes(name);
         Assert.assertEquals("Wrong name", message.getName(), new String(name).trim());
index 1ffdc3541674e51770101c2ff1b4017bd7774b04..f6503b56a9b4d98946f4c6636bd736883e3c8c0b 100644 (file)
@@ -89,7 +89,7 @@ public class PortModInputMessageFactoryTest {
         out.skipBytes(PADDING_IN_PORT_MOD_MESSAGE_01);
         byte[] address = new byte[6];
         out.readBytes(address);
-        Assert.assertEquals("Wrong MacAddress", message.getHwAddress(), IetfYangUtil.INSTANCE.macAddressFor(address));
+        Assert.assertEquals("Wrong MacAddress", message.getHwAddress(), IetfYangUtil.macAddressFor(address));
         out.skipBytes(PADDING_IN_PORT_MOD_MESSAGE_02);
         Assert.assertEquals("Wrong config", message.getConfig(), createPortConfig(out.readInt()));
         Assert.assertEquals("Wrong mask", message.getMask(), createPortConfig(out.readInt()));
index 13305df7d0337f29704ed34134f4f04f89ac5a4c..f351f86a186aed3f81465c6d83db19b440d59ff9 100644 (file)
@@ -80,7 +80,7 @@ public class PortStatusMessageFactoryTest {
         byte[] address = new byte[6];
         serializedBuffer.readBytes(address);
         Assert.assertEquals("Wrong MacAddress", message.getHwAddr().getValue().toLowerCase(),
-                IetfYangUtil.INSTANCE.macAddressFor(address).getValue().toLowerCase());
+                IetfYangUtil.macAddressFor(address).getValue().toLowerCase());
         serializedBuffer.skipBytes(PORT_PADDING_2);
         byte[] name = new byte[16];
         serializedBuffer.readBytes(name);
index 8a9df364d5af224dff8cf3db735c0b64574cff6c..fc78e954dbdd38e9d97b7686a1955a9a14e6bb1d 100644 (file)
@@ -7,6 +7,11 @@
  */
 package org.opendaylight.openflowjava.protocol.it.integration;
 
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.doAnswer;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
+
 import java.net.InetAddress;
 import java.util.ArrayList;
 import java.util.Deque;
@@ -20,9 +25,9 @@ import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.ArgumentMatchers;
 import org.mockito.Mock;
-import org.mockito.Mockito;
 import org.mockito.junit.MockitoJUnitRunner;
 import org.opendaylight.infrautils.diagstatus.DiagStatusService;
+import org.opendaylight.infrautils.diagstatus.ServiceRegistration;
 import org.opendaylight.openflowjava.protocol.api.connection.TlsConfiguration;
 import org.opendaylight.openflowjava.protocol.api.connection.TlsConfigurationImpl;
 import org.opendaylight.openflowjava.protocol.impl.clients.ClientEvent;
@@ -79,8 +84,8 @@ public class IntegrationTest {
 
     public void setUp(final TransportProtocol protocol) throws Exception {
         LOGGER.debug("\n starting test -------------------------------");
-        Mockito.doAnswer(invocation -> {
-            ((Runnable)invocation.getArguments()[0]).run();
+        doAnswer(invocation -> {
+            invocation.getArgument(0, Runnable.class).run();
             return null;
         }).when(executorService).execute(ArgumentMatchers.any());
 
@@ -99,7 +104,10 @@ public class IntegrationTest {
         connConfig.setTransferProtocol(protocol);
         mockPlugin = new MockPlugin(executorService);
 
-        switchConnectionProvider = new SwitchConnectionProviderImpl(Mockito.mock(DiagStatusService.class), connConfig);
+        final var diagStatusService = mock(DiagStatusService.class);
+        doReturn(mock(ServiceRegistration.class)).when(diagStatusService).register(any());
+
+        switchConnectionProvider = new SwitchConnectionProviderImpl(diagStatusService, connConfig);
         switchConnectionProvider.setSwitchConnectionHandler(mockPlugin);
         switchConnectionProvider.startup().get(CONNECTION_TIMEOUT, TimeUnit.MILLISECONDS);
         if (protocol.equals(TransportProtocol.TCP) || protocol.equals(TransportProtocol.TLS)) {
index d5d325fcea5c65743cffdd0b3ec3d36537e9999a..59005e9cbc14ef3c8cd06fa43042356c44cc445e 100644 (file)
@@ -231,19 +231,19 @@ public abstract class ByteBufUtils {
     }
 
     public static Ipv4Address readIetfIpv4Address(final ByteBuf buf) {
-        return IetfInetUtil.INSTANCE.ipv4AddressFor(buf.readInt());
+        return IetfInetUtil.ipv4AddressFor(buf.readInt());
     }
 
     public static Ipv6Address readIetfIpv6Address(final ByteBuf buf) {
         final byte[] tmp = new byte[16];
         buf.readBytes(tmp);
-        return IetfInetUtil.INSTANCE.ipv6AddressFor(tmp);
+        return IetfInetUtil.ipv6AddressFor(tmp);
     }
 
     public static MacAddress readIetfMacAddress(final ByteBuf buf) {
         final byte[] tmp = new byte[EncodeConstants.MAC_ADDRESS_LENGTH];
         buf.readBytes(tmp);
-        return IetfYangUtil.INSTANCE.macAddressFor(tmp);
+        return IetfYangUtil.macAddressFor(tmp);
     }
 
     public static byte[] serializeList(final List<Short> list) {
index 8fc170bb1036781ced7494f3e7bf819294594f36..13fe6317ea1c80a1802fd45dcbb500931fada91b 100644 (file)
             <artifactId>jakarta.annotation-api</artifactId>
             <optional>true</optional>
         </dependency>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <optional>true</optional>
+        </dependency>
         <dependency>
             <groupId>com.guicedee.services</groupId>
             <artifactId>javax.inject</artifactId>
index f92ffef558356dee35bb10024cbdaa22b58734bc..a84fed96c9d0eae342f949810b3f31c8dcfba2d8 100644 (file)
@@ -7,8 +7,6 @@
  */
 package org.opendaylight.openflowplugin.impl;
 
-import static java.util.Objects.requireNonNull;
-
 import javax.annotation.PreDestroy;
 import javax.inject.Inject;
 import javax.inject.Singleton;
@@ -29,14 +27,11 @@ public final class DefaultDiagStatusProvider implements DiagStatusProvider {
     private static final Logger LOG = LoggerFactory.getLogger(DefaultDiagStatusProvider.class);
     private static final String OPENFLOW_SERVICE_NAME = "OPENFLOW";
 
-    private final DiagStatusService diagStatusService;
-
     private ServiceRegistration reg;
 
     @Inject
     @Activate
     public DefaultDiagStatusProvider(@Reference final DiagStatusService diagStatusService) {
-        this.diagStatusService = requireNonNull(diagStatusService);
         reg = diagStatusService.register(OPENFLOW_SERVICE_NAME);
     }
 
@@ -44,26 +39,26 @@ public final class DefaultDiagStatusProvider implements DiagStatusProvider {
     @Deactivate
     public void close() {
         if (reg != null) {
-            reg.unregister();
+            reg.close();
             reg = null;
         }
     }
 
     @Override
-    public void reportStatus(ServiceState serviceState) {
+    public void reportStatus(final ServiceState serviceState) {
         LOG.debug("reporting status as {} for {}", serviceState, OPENFLOW_SERVICE_NAME);
-        diagStatusService.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, serviceState));
+        reg.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, serviceState));
     }
 
     @Override
-    public void reportStatus(ServiceState serviceState, Throwable throwable) {
+    public void reportStatus(final ServiceState serviceState, final Throwable throwable) {
         LOG.debug("reporting status as {} for {}", serviceState, OPENFLOW_SERVICE_NAME);
-        diagStatusService.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, throwable));
+        reg.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, throwable));
     }
 
     @Override
-    public void reportStatus(ServiceState serviceState, String description) {
+    public void reportStatus(final ServiceState serviceState, final String description) {
         LOG.debug("reporting status as {} for {}", serviceState, OPENFLOW_SERVICE_NAME);
-        diagStatusService.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, serviceState, description));
+        reg.report(new ServiceDescriptor(OPENFLOW_SERVICE_NAME, serviceState, description));
     }
 }
index 2169e1605cc7c311f3c25075c8118601798c021f..a74e9a16441dc4634a5d5cf33361efe12a200d81 100644 (file)
@@ -29,7 +29,7 @@ public abstract class AbstractMacAddressFilterEntrySerializer<E extends MacAddre
     protected final void serializeEntry(final E entry, final MacAddress mask, final ByteBuf outBuffer) {
         writeMacAddress(entry.getAddress(), outBuffer);
         if (mask != null) {
-            writeMask(IetfYangUtil.INSTANCE.macAddressBytes(mask), outBuffer, EncodeConstants.MAC_ADDRESS_LENGTH);
+            writeMask(IetfYangUtil.macAddressBytes(mask), outBuffer, EncodeConstants.MAC_ADDRESS_LENGTH);
         }
     }
 }
index 733301f079125979672f15cdcc28aa050e243882..aeecb7f7ded3e2c21fc95a4772d872fc3196bb5b 100644 (file)
@@ -174,7 +174,7 @@ public abstract class AbstractMatchEntrySerializer<E, M> implements MatchEntrySe
      * @param outBuffer output buffer
      */
     protected static void writeIpv4Address(final Ipv4Address address, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(address));
+        outBuffer.writeBytes(IetfInetUtil.ipv4AddressBytes(address));
     }
 
     /**
@@ -184,7 +184,7 @@ public abstract class AbstractMatchEntrySerializer<E, M> implements MatchEntrySe
      * @param outBuffer output buffer
      */
     protected static void writeIpv6Address(final Ipv6Address address, final ByteBuf outBuffer) {
-        outBuffer.writeBytes(IetfInetUtil.INSTANCE.ipv6AddressBytes(address));
+        outBuffer.writeBytes(IetfInetUtil.ipv6AddressBytes(address));
     }
 
     /**
@@ -195,7 +195,7 @@ public abstract class AbstractMatchEntrySerializer<E, M> implements MatchEntrySe
      */
     protected static void writeMacAddress(final MacAddress address, final ByteBuf outBuffer) {
         // 48 b + mask [OF 1.3.2 spec]
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(address));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(address));
     }
 
     /**
@@ -207,7 +207,7 @@ public abstract class AbstractMatchEntrySerializer<E, M> implements MatchEntrySe
     protected static void writeIpv4Prefix(final @NonNull Ipv4Prefix prefix, final @Nullable Integer mask,
             final @NonNull ByteBuf outBuffer) {
         // Write address part of prefix
-        writeIpv4Address(IetfInetUtil.INSTANCE.ipv4AddressFrom(prefix), outBuffer);
+        writeIpv4Address(IetfInetUtil.ipv4AddressFrom(prefix), outBuffer);
 
         // If prefix had mask, also write prefix
         if (mask != null) {
index f6610d9e8bd45b319fefaf8dbbe23b3f2c6615e3..ac8a8c87f1dbfd340d92f238885c9a0d2ac2c3ce 100644 (file)
@@ -37,7 +37,7 @@ public class PortMessageSerializer extends AbstractMessageSerializer<PortMessage
         outBuffer.writeInt(OpenflowPortsUtil
                 .getProtocolPortNumber(OpenflowVersion.OF13, message.getPortNumber()).intValue());
         outBuffer.writeZero(PADDING_IN_PORT_MOD_MESSAGE_01);
-        outBuffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(message.getHardwareAddress()));
+        outBuffer.writeBytes(IetfYangUtil.macAddressBytes(message.getHardwareAddress()));
         outBuffer.writeZero(PADDING_IN_PORT_MOD_MESSAGE_02);
         outBuffer.writeInt(createPortConfigBitMask(message.getConfiguration()));
         outBuffer.writeInt(requireNonNullElse(createPortConfigBitMask(message.getMask()), DEFAULT_PORT_CONFIG_MASK));
index b309f9566b56df1b5a7301dcaeee3c5028f74f7c..4dcb4b99618b2f293adb8f240bcf8111ae66a241 100644 (file)
@@ -70,7 +70,7 @@ public final class AddressNormalizationUtil {
             return null;
         }
 
-        final byte[] address = IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Prefix));
+        final byte[] address = IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Prefix));
         final byte[] mask =
                 IpConversionUtil.convertIpv6PrefixToByteArray(IpConversionUtil.extractIpv6Prefix(ipv6Prefix));
         return normalizeIpv6Address(address, mask);
@@ -89,7 +89,7 @@ public final class AddressNormalizationUtil {
             return null;
         }
 
-        final byte[] address = IetfInetUtil.INSTANCE.ipv6AddressBytes(ipv6Address);
+        final byte[] address = IetfInetUtil.ipv6AddressBytes(ipv6Address);
         final byte[] mask = IpConversionUtil.convertIpv6ArbitraryMaskToByteArray(ipv4Mask);
         return normalizeIpv6Address(address, mask);
     }
@@ -116,7 +116,7 @@ public final class AddressNormalizationUtil {
             return null;
         }
 
-        final byte[] address = IetfInetUtil.INSTANCE.ipv4AddressBytes(IpConversionUtil.extractIpv4Address(ipv4Prefix));
+        final byte[] address = IetfInetUtil.ipv4AddressBytes(IpConversionUtil.extractIpv4Address(ipv4Prefix));
         final byte[] mask =
                 IpConversionUtil.convertArbitraryMaskToByteArray(IpConversionUtil.extractIpv4AddressMask(ipv4Prefix));
         return normalizeIpv4Address(address, mask);
@@ -135,7 +135,7 @@ public final class AddressNormalizationUtil {
             return null;
         }
 
-        final byte[] address = IetfInetUtil.INSTANCE.ipv4AddressBytes(ipv4Address);
+        final byte[] address = IetfInetUtil.ipv4AddressBytes(ipv4Address);
         final byte[] mask = IpConversionUtil.convertArbitraryMaskToByteArray(ipv4Mask);
         return normalizeIpv4Address(address, mask);
     }
index d4a4e9dd7ceadc4bfab8074e75362067cc119a94..1c1613db9cc8bfed778acb3607eab9c2de4b3e6d 100644 (file)
@@ -103,7 +103,7 @@ public final class DeviceInitializationUtil {
         if (address == null) {
             return null;
         }
-        final IpAddress ipAddress = IetfInetUtil.INSTANCE.ipAddressFor(address.getAddress());
+        final IpAddress ipAddress = IetfInetUtil.ipAddressFor(address.getAddress());
         LOG.info("IP address of the node {} is: {}", node, ipAddress);
         return ipAddress;
     }
index da639f9efc94b7bc18f2be90b78f2f8e4c9cdf6c..3c10e637adcd28f75c54200ab52b427fd7641efa 100644 (file)
@@ -12,7 +12,6 @@ import com.google.common.base.Function;
 import com.google.common.base.Preconditions;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
@@ -36,9 +35,9 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.flows.service.rev160314.Upd
 import org.opendaylight.yang.gen.v1.urn.opendaylight.flows.service.rev160314.batch.flow.output.list.grouping.BatchFailedFlowsOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.flows.service.rev160314.batch.flow.output.list.grouping.BatchFailedFlowsOutputBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node;
-import org.opendaylight.yangtools.yang.binding.Identifiable;
-import org.opendaylight.yangtools.yang.binding.Identifier;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
+import org.opendaylight.yangtools.yang.binding.Key;
+import org.opendaylight.yangtools.yang.binding.KeyAware;
 import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier;
 import org.opendaylight.yangtools.yang.binding.util.BindingMap;
 import org.opendaylight.yangtools.yang.common.RpcError;
@@ -50,7 +49,7 @@ import org.opendaylight.yangtools.yang.common.Uint8;
 public final class FlowUtil {
 
     private static final RpcResultBuilder<List<BatchFailedFlowsOutput>> SUCCESSFUL_FLOW_OUTPUT_RPC_RESULT =
-            RpcResultBuilder.success(Collections.emptyList());
+            RpcResultBuilder.success(List.of());
 
     /**
      * Attach barrier response to given {@link RpcResult}&lt;RemoveFlowsBatchOutput&gt;.
@@ -121,7 +120,7 @@ public final class FlowUtil {
         // Hidden on purpose
     }
 
-    static <K extends Identifier<V>, V extends Identifiable<K>> Map<K, V> index(final List<V> list) {
+    static <K extends Key<V>, V extends KeyAware<K>> Map<K, V> index(final List<V> list) {
         return list == null ? null : BindingMap.ordered(list);
     }
 
index 8d9c81e14266f742d85162cb4dcf90ee266324ad..b05af98df53a284a2be6ec3c4801bf191204f39e 100644 (file)
@@ -28,7 +28,7 @@ public class ArpSourceHardwareAddressEntryDeserializerTest extends AbstractMatch
         final MacAddress arpSourceHardwareAddressMask = new MacAddress("00:00:00:00:00:00");
 
         writeHeader(in, false);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpSourceHardwareAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(arpSourceHardwareAddress));
 
         assertEquals(arpSourceHardwareAddress.getValue(),
                 ((ArpMatch) deserialize(in).getLayer3Match()).getArpSourceHardwareAddress().getAddress()
@@ -36,8 +36,8 @@ public class ArpSourceHardwareAddressEntryDeserializerTest extends AbstractMatch
         assertEquals(0, in.readableBytes());
 
         writeHeader(in, true);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpSourceHardwareAddress));
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpSourceHardwareAddressMask));
+        in.writeBytes(IetfYangUtil.macAddressBytes(arpSourceHardwareAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(arpSourceHardwareAddressMask));
 
         final ArpSourceHardwareAddress desAddress =
                 ((ArpMatch) deserialize(in).getLayer3Match()).getArpSourceHardwareAddress();
index 6cdbf47a7f2042ceb381aab2b966564bb1557627..a2d5165c74407d28bc1a0c521476ba9e710b6a3f 100644 (file)
@@ -30,7 +30,7 @@ public class ArpSourceTransportAddressEntryDeserializerTest extends AbstractMatc
 
         writeHeader(in, false);
         Iterator<String> addressParts = IpConversionUtil.splitToParts(arpSourceTransportAddressNoMask);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
+        in.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
 
         assertEquals(arpSourceTransportAddressNoMask.getValue(),
                 ((ArpMatch) deserialize(in).getLayer3Match()).getArpSourceTransportAddress().getValue());
@@ -38,7 +38,7 @@ public class ArpSourceTransportAddressEntryDeserializerTest extends AbstractMatc
 
         writeHeader(in, true);
         addressParts = IpConversionUtil.splitToParts(arpSourceTransportAddress);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
+        in.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
         in.writeBytes(MatchConvertorUtil.extractIpv4Mask(addressParts));
 
         final Ipv4Prefix desAddress =
index a70be6589d360f0f2e750e4eae6c7514d3ce599e..35f20af237757ddf7fc162e88f2eb941810e30b8 100644 (file)
@@ -28,7 +28,7 @@ public class ArpTargetHardwareAddressEntryDeserializerTest extends AbstractMatch
         final MacAddress arpTargetHardwareAddressMask = new MacAddress("00:00:00:00:00:00");
 
         writeHeader(in, false);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpTargetHardwareAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(arpTargetHardwareAddress));
 
         assertEquals(arpTargetHardwareAddress.getValue(),
                 ((ArpMatch) deserialize(in).getLayer3Match()).getArpTargetHardwareAddress().getAddress()
@@ -36,8 +36,8 @@ public class ArpTargetHardwareAddressEntryDeserializerTest extends AbstractMatch
         assertEquals(0, in.readableBytes());
 
         writeHeader(in, true);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpTargetHardwareAddress));
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(arpTargetHardwareAddressMask));
+        in.writeBytes(IetfYangUtil.macAddressBytes(arpTargetHardwareAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(arpTargetHardwareAddressMask));
 
         final ArpTargetHardwareAddress desAddress =
                 ((ArpMatch) deserialize(in).getLayer3Match()).getArpTargetHardwareAddress();
index 4b9757f5de7b97cad9be903af45192bfa56a0556..a8dec037ce1f157178b290696599d3060aa608b3 100644 (file)
@@ -30,7 +30,7 @@ public class ArpTargetTransportAddressEntryDeserializerTest extends AbstractMatc
 
         writeHeader(in, false);
         Iterator<String> addressParts = IpConversionUtil.splitToParts(arpTargetTransportAddressNoMask);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
+        in.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
 
         assertEquals(arpTargetTransportAddressNoMask.getValue(),
                 ((ArpMatch) deserialize(in).getLayer3Match()).getArpTargetTransportAddress().getValue());
@@ -38,7 +38,7 @@ public class ArpTargetTransportAddressEntryDeserializerTest extends AbstractMatc
 
         writeHeader(in, true);
         addressParts = IpConversionUtil.splitToParts(arpTargetTransportAddress);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
+        in.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
         in.writeBytes(MatchConvertorUtil.extractIpv4Mask(addressParts));
 
         final Ipv4Prefix desAddress =
index 93e95cbae1593ad2fa404d773956bd0177c9019a..495c8d7745fa7ead89c411ac12cd12b0f4d490a9 100644 (file)
@@ -27,15 +27,15 @@ public class EthernetDestinationEntryDeserializerTest extends AbstractMatchEntry
         final MacAddress ethernetDestinationAddressMask = new MacAddress("00:00:00:00:00:00");
 
         writeHeader(in, false);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethernetDestinationAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(ethernetDestinationAddress));
 
         assertEquals(ethernetDestinationAddress.getValue(), deserialize(in).getEthernetMatch().getEthernetDestination()
                 .getAddress().getValue());
         assertEquals(0, in.readableBytes());
 
         writeHeader(in, true);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethernetDestinationAddress));
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethernetDestinationAddressMask));
+        in.writeBytes(IetfYangUtil.macAddressBytes(ethernetDestinationAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(ethernetDestinationAddressMask));
 
         final EthernetDestination desAddress = deserialize(in).getEthernetMatch().getEthernetDestination();
         assertEquals(ethernetDestinationAddress.getValue(), desAddress.getAddress().getValue());
index f47ee760bc07c2001c829e8e4f1c2d35f8f6b477..ec83e7e4d8b3f31acf8f62ea7186b803c57860f2 100644 (file)
@@ -27,15 +27,15 @@ public class EthernetSourceEntryDeserializerTest extends AbstractMatchEntryDeser
         final MacAddress ethernetSourceAddressMask = new MacAddress("00:00:00:00:00:00");
 
         writeHeader(in, false);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethernetSourceAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(ethernetSourceAddress));
 
         assertEquals(ethernetSourceAddress.getValue(),
                 deserialize(in).getEthernetMatch().getEthernetSource().getAddress().getValue());
         assertEquals(0, in.readableBytes());
 
         writeHeader(in, true);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethernetSourceAddress));
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ethernetSourceAddressMask));
+        in.writeBytes(IetfYangUtil.macAddressBytes(ethernetSourceAddress));
+        in.writeBytes(IetfYangUtil.macAddressBytes(ethernetSourceAddressMask));
 
         final EthernetSource desAddress = deserialize(in).getEthernetMatch().getEthernetSource();
         assertEquals(ethernetSourceAddress.getValue(), desAddress.getAddress().getValue());
index 79baddf8e3506583914caef174c511f21301eb09..fc712b947ebd2d0796ce43b68d5b896ce1b61dad 100644 (file)
@@ -32,7 +32,7 @@ public class Ipv4DestinationEntryDeserializerTest extends AbstractMatchEntryDese
         final Iterator<String> addressParts = IpConversionUtil.splitToParts(address);
 
         writeHeader(in, true);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
+        in.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
         in.writeBytes(MatchConvertorUtil.extractIpv4Mask(addressParts));
 
         final Ipv4Match match = (Ipv4Match) deserialize(in).getLayer3Match();
index 71d4a24d222b8df92a2584b2bfc1bca3e2310e73..757513650b5ade258a628a60865bff835763a9c6 100644 (file)
@@ -32,7 +32,7 @@ public class Ipv4SourceEntryDeserializerTest extends AbstractMatchEntryDeseriali
         final Iterator<String> addressParts = IpConversionUtil.splitToParts(address);
 
         writeHeader(in, true);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
+        in.writeBytes(IetfInetUtil.ipv4AddressBytes(new Ipv4Address(addressParts.next())));
         in.writeBytes(MatchConvertorUtil.extractIpv4Mask(addressParts));
 
         final Ipv4Match match = (Ipv4Match) deserialize(in).getLayer3Match();
index 9d7c7d1b98701748121bf56deaeafd6a683aaeef..46923b985ffc1b2d4b3435521bdcf146b2113bb0 100644 (file)
@@ -28,7 +28,7 @@ public class Ipv6DestinationEntryDeserializerTest extends AbstractMatchEntryDese
         final Ipv6Prefix address = new Ipv6Prefix("fe80::200:f8ff:fe21:67cf/30");
 
         writeHeader(in, true);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(address)));
+        in.writeBytes(IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(address)));
         in.writeBytes(IpConversionUtil.convertIpv6PrefixToByteArray(IpConversionUtil.extractIpv6Prefix(address)));
 
         final Ipv6Match match = (Ipv6Match) deserialize(in).getLayer3Match();
index 70f6f042d2602a6e55163c65ea3c1b67471d25f7..845aa94bcd015cf86ddadd858ab7ba392fb32aa9 100644 (file)
@@ -26,7 +26,7 @@ public class Ipv6NdSllEntryDeserializerTest extends AbstractMatchEntryDeserializ
         final MacAddress address = new MacAddress("00:01:02:03:04:05");
 
         writeHeader(in, false);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(address));
+        in.writeBytes(IetfYangUtil.macAddressBytes(address));
 
         Ipv6Match match = (Ipv6Match) deserialize(in).getLayer3Match();
         assertEquals(address.getValue(), match.getIpv6NdSll().getValue());
index 3869b3f8393bf4de27dac8500d0e929835107598..8e242dd5dc8820cc7e5ee7ce1971a8473aaefd5a 100644 (file)
@@ -28,12 +28,12 @@ public class Ipv6NdTargetEntryDeserializerTest extends AbstractMatchEntryDeseria
         final Ipv6Address address = new Ipv6Address("2001:db8::");
 
         writeHeader(in, false);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv6AddressBytes(address));
+        in.writeBytes(IetfInetUtil.ipv6AddressBytes(address));
 
         Ipv6Match match = (Ipv6Match) deserialize(in).getLayer3Match();
         assertArrayEquals(
-                IetfInetUtil.INSTANCE.ipv6AddressBytes(address),
-                IetfInetUtil.INSTANCE.ipv6AddressBytes(match.getIpv6NdTarget()));
+                IetfInetUtil.ipv6AddressBytes(address),
+                IetfInetUtil.ipv6AddressBytes(match.getIpv6NdTarget()));
         assertEquals(0, in.readableBytes());
     }
 
index 6860936da9f4dd4e7f4594f2e7bf212412b029db..0e11d57fef6a68ebe4beef4b56acdb470504b35f 100644 (file)
@@ -26,7 +26,7 @@ public class Ipv6NdTllEntryDeserializerTest extends AbstractMatchEntryDeserializ
         final MacAddress address = new MacAddress("00:01:02:03:04:05");
 
         writeHeader(in, false);
-        in.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(address));
+        in.writeBytes(IetfYangUtil.macAddressBytes(address));
 
         Ipv6Match match = (Ipv6Match) deserialize(in).getLayer3Match();
         assertEquals(address.getValue(), match.getIpv6NdTll().getValue());
index 25c4b25a6504e861aea8a55a6aa11fd506fa7e11..abca2f3e33085e6a3c9319ad5ea0485d3bfa3a69 100644 (file)
@@ -28,7 +28,7 @@ public class Ipv6SourceEntryDeserializerTest extends AbstractMatchEntryDeseriali
         final Ipv6Prefix address = new Ipv6Prefix("fe80::200:f8ff:fe21:67cf/30");
 
         writeHeader(in, true);
-        in.writeBytes(IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(address)));
+        in.writeBytes(IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(address)));
         in.writeBytes(IpConversionUtil.convertIpv6PrefixToByteArray(IpConversionUtil.extractIpv6Prefix(address)));
 
         final Ipv6Match match = (Ipv6Match) deserialize(in).getLayer3Match();
index 6d8172fedf18c9bbbea3ee4cd9a3f50edfa2093f..dc66b6efe3188eec108a013933ecd7ceba48772c 100644 (file)
@@ -65,7 +65,7 @@ public class PortMessageDeserializerTest extends AbstractDeserializerTest {
         buffer.writeInt(XID);
         buffer.writeInt(PORT);
         buffer.writeZero(PADDING_IN_PORT_MOD_MESSAGE_1);
-        buffer.writeBytes(IetfYangUtil.INSTANCE.macAddressBytes(ADDRESS));
+        buffer.writeBytes(IetfYangUtil.macAddressBytes(ADDRESS));
 
         final int config = ByteBufUtils.fillBitMaskFromMap(ImmutableMap
                 .<Integer, Boolean>builder()
index 930bb521d333f07d003458f885a4c1b045dda0af..19b4b5b1a4bc5249b6b1e324efcc0c2c49b01522 100644 (file)
@@ -31,7 +31,7 @@ public class SetDlDstActionSerializerTest extends AbstractSetFieldActionSerializ
         assertAction(action, out -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index 70942af696b0f891bb26ecbedc0a777923204e44..16426c6799539c288bcc389c9fcaa4d05cd5622e 100644 (file)
@@ -31,7 +31,7 @@ public class SetDlSrcActionSerializerTest extends AbstractSetFieldActionSerializ
         assertAction(action, out -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index c5d4c005445230e127d92e1b79782ad67fcca572..4ef1ae91a0f13db782dd3d7c88ea8d9b3a3cbd55 100644 (file)
@@ -37,11 +37,11 @@ public class ArpSourceHardwareAddressEntrySerializerTest extends AbstractMatchEn
         assertMatch(arpShaMatch, true, (out) -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
 
             byte[] maskBytes = new byte[6];
             out.readBytes(maskBytes);
-            assertEquals(mask, IetfYangUtil.INSTANCE.macAddressFor(maskBytes));
+            assertEquals(mask, IetfYangUtil.macAddressFor(maskBytes));
         });
 
         final Match arpShaMatchNoMask = new MatchBuilder()
@@ -55,7 +55,7 @@ public class ArpSourceHardwareAddressEntrySerializerTest extends AbstractMatchEn
         assertMatch(arpShaMatchNoMask, false, (out) -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index e1796eeabbec0944d9c5ae91e8680fcc8a4b2ead..3e46e0844ba7e76dcb82289f4f9929c0a7c10ce0 100644 (file)
@@ -37,11 +37,11 @@ public class ArpTargetHardwareAddressEntrySerializerTest extends AbstractMatchEn
         assertMatch(arpShaMatch, true, (out) -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
 
             byte[] maskBytes = new byte[6];
             out.readBytes(maskBytes);
-            assertEquals(mask, IetfYangUtil.INSTANCE.macAddressFor(maskBytes));
+            assertEquals(mask, IetfYangUtil.macAddressFor(maskBytes));
         });
 
         final Match arpShaMatchNoMask = new MatchBuilder()
@@ -55,7 +55,7 @@ public class ArpTargetHardwareAddressEntrySerializerTest extends AbstractMatchEn
         assertMatch(arpShaMatchNoMask, false, (out) -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index 3680a43b85139bd5aa9d0eed3b5930e780169fae..cab5b786d5ca163999e0d7a0789f654e4984a7cd 100644 (file)
@@ -37,11 +37,11 @@ public class EthernetDestinationEntrySerializerTest extends AbstractMatchEntrySe
         assertMatch(match, true, (out) -> {
             final byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
 
             final byte[] maskBytes = new byte[6];
             out.readBytes(maskBytes);
-            assertEquals(mask, IetfYangUtil.INSTANCE.macAddressFor(maskBytes));
+            assertEquals(mask, IetfYangUtil.macAddressFor(maskBytes));
         });
 
         final Match matchNoMask = new MatchBuilder()
@@ -55,7 +55,7 @@ public class EthernetDestinationEntrySerializerTest extends AbstractMatchEntrySe
         assertMatch(matchNoMask, false, (out) -> {
             final byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index 5640eb0ab8b36db9d20aac13134a9300fd3f50ab..e0339cd6df2b3908d767a3b2b1ea0c3349a6ac61 100644 (file)
@@ -37,11 +37,11 @@ public class EthernetSourceEntrySerializerTest extends AbstractMatchEntrySeriali
         assertMatch(match, true, (out) -> {
             final byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
 
             final byte[] maskBytes = new byte[6];
             out.readBytes(maskBytes);
-            assertEquals(mask, IetfYangUtil.INSTANCE.macAddressFor(maskBytes));
+            assertEquals(mask, IetfYangUtil.macAddressFor(maskBytes));
         });
 
         final Match matchNoMask = new MatchBuilder()
@@ -55,7 +55,7 @@ public class EthernetSourceEntrySerializerTest extends AbstractMatchEntrySeriali
         assertMatch(matchNoMask, false, (out) -> {
             final byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(address, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(address, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index d08a52950899fd54fc87d146e79a93a112ccd01a..aa093a63d3a4d8a614a7704d9d83f48e210639e3 100644 (file)
@@ -38,7 +38,7 @@ public class Ipv6ArbitraryBitMaskDestinationEntrySerializerTest extends Abstract
         assertMatch(ipv6abmMatch, true, (out) -> {
             byte[] address = new byte[16];
             out.readBytes(address);
-            assertArrayEquals(address, IetfInetUtil.INSTANCE.ipv6AddressBytes(ipv6Address));
+            assertArrayEquals(address, IetfInetUtil.ipv6AddressBytes(ipv6Address));
 
             byte[] mask = new byte[16];
             out.readBytes(mask);
@@ -54,7 +54,7 @@ public class Ipv6ArbitraryBitMaskDestinationEntrySerializerTest extends Abstract
         assertMatch(ipv6abmMatchNoMask, false, (out) -> {
             byte[] address = new byte[16];
             out.readBytes(address);
-            assertArrayEquals(address, IetfInetUtil.INSTANCE.ipv6AddressBytes(ipv6Address));
+            assertArrayEquals(address, IetfInetUtil.ipv6AddressBytes(ipv6Address));
         });
     }
 
index 178187c3d80f42db16da15687782a1d70621a94e..76c49b65b7f4ac0d4d6a467364a35aee0269cbd4 100644 (file)
@@ -38,7 +38,7 @@ public class Ipv6ArbitraryBitMaskSourceEntrySerializerTest extends AbstractMatch
         assertMatch(ipv6abmMatch, true, (out) -> {
             byte[] address = new byte[16];
             out.readBytes(address);
-            assertArrayEquals(address, IetfInetUtil.INSTANCE.ipv6AddressBytes(ipv6Address));
+            assertArrayEquals(address, IetfInetUtil.ipv6AddressBytes(ipv6Address));
 
             byte[] mask = new byte[16];
             out.readBytes(mask);
@@ -54,7 +54,7 @@ public class Ipv6ArbitraryBitMaskSourceEntrySerializerTest extends AbstractMatch
         assertMatch(ipv6abmMatchNoMask, false, (out) -> {
             byte[] address = new byte[16];
             out.readBytes(address);
-            assertArrayEquals(address, IetfInetUtil.INSTANCE.ipv6AddressBytes(ipv6Address));
+            assertArrayEquals(address, IetfInetUtil.ipv6AddressBytes(ipv6Address));
         });
     }
 
index 065cbc148a874457865edf57ce90ae199a35d277..76aa76fa1a19a2c8fdd5447764aebec5f6b563d1 100644 (file)
@@ -36,7 +36,7 @@ public class Ipv6DestinationEntrySerializerTest extends AbstractMatchEntrySerial
             byte[] address = new byte[16];
             out.readBytes(address);
             assertArrayEquals(address,
-                    IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
+                    IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
 
             byte[] mask = new byte[16];
             out.readBytes(mask);
@@ -59,7 +59,7 @@ public class Ipv6DestinationEntrySerializerTest extends AbstractMatchEntrySerial
             byte[] address = new byte[16];
             out.readBytes(address);
             assertArrayEquals(address,
-                    IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
+                    IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
         });
     }
 
index ad983ee0f6944331d72a3f751c0041c380ea2f06..f1d450aad224e942bb50294855d14c99f1d58d13 100644 (file)
@@ -30,7 +30,7 @@ public class Ipv6NdSllEntrySerializerTest extends AbstractMatchEntrySerializerTe
         assertMatch(ipv6NdSllMatch, false, (out) -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(ipv6NdSll, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(ipv6NdSll, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index b8789305bad2d4a6038f0fb8daa2123879f4e2b7..caae491dcd3706f338681d377685eb835cc7d8c7 100644 (file)
@@ -34,7 +34,7 @@ public class Ipv6NdTargetEntrySerializerTest extends AbstractMatchEntrySerialize
         assertMatch(ipv6NdTargetMatch, false, (out) -> {
             byte[] addressBytes = new byte[16];
             out.readBytes(addressBytes);
-            assertArrayEquals(addressBytes, IetfInetUtil.INSTANCE.ipv6AddressBytes(ipv6NdTarget));
+            assertArrayEquals(addressBytes, IetfInetUtil.ipv6AddressBytes(ipv6NdTarget));
         });
     }
 
index a7bd39a8baf12914c5bbe62c5e074470daa0172b..49472df266e1ef1b55605d26fcd1aeeda9edb610 100644 (file)
@@ -30,7 +30,7 @@ public class Ipv6NdTllEntrySerializerTest extends AbstractMatchEntrySerializerTe
         assertMatch(ipv6NdTllMatch, false, (out) -> {
             byte[] addressBytes = new byte[6];
             out.readBytes(addressBytes);
-            assertEquals(ipv6NdTll, IetfYangUtil.INSTANCE.macAddressFor(addressBytes));
+            assertEquals(ipv6NdTll, IetfYangUtil.macAddressFor(addressBytes));
         });
     }
 
index 847950031cf7d325cb8cbf3c3705a4f537d2c8eb..78aac1fa26663f99b00127471b9b7dfaf1a96272 100644 (file)
@@ -36,7 +36,7 @@ public class Ipv6SourceEntrySerializerTest extends AbstractMatchEntrySerializerT
             byte[] address = new byte[16];
             out.readBytes(address);
             assertArrayEquals(address,
-                    IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
+                    IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
 
             byte[] mask = new byte[16];
             out.readBytes(mask);
@@ -59,7 +59,7 @@ public class Ipv6SourceEntrySerializerTest extends AbstractMatchEntrySerializerT
             byte[] address = new byte[16];
             out.readBytes(address);
             assertArrayEquals(address,
-                    IetfInetUtil.INSTANCE.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
+                    IetfInetUtil.ipv6AddressBytes(IpConversionUtil.extractIpv6Address(ipv6Address)));
         });
     }
 
index 7d9df9858803c778aaeba8d45f080cf74f78aa5f..6604f6590579365676cbf07629cdc4e0c8ea51c6 100644 (file)
@@ -116,7 +116,7 @@ public class PortMessageSerializerTest extends AbstractSerializerTest {
         out.skipBytes(PADDING_IN_PORT_MOD_MESSAGE_01);
         byte[] address = new byte[6];
         out.readBytes(address);
-        assertEquals(new MacAddress(MAC_ADDRESS), IetfYangUtil.INSTANCE.macAddressFor(address));
+        assertEquals(new MacAddress(MAC_ADDRESS), IetfYangUtil.macAddressFor(address));
         out.skipBytes(PADDING_IN_PORT_MOD_MESSAGE_02);
 
         // Port config
index 1b6b3dc25978bdced0affdabe1d8358e81160564..1bb82362389c09c58e719ae5263ee13db7d2646c 100644 (file)
@@ -103,7 +103,7 @@ public class DeviceInitializationUtilTest {
     @Test
     public void getIpAddress() {
         final IpAddress ipAddress = DeviceInitializationUtil.getIpAddress(connectionContext, NODE_II);
-        assertEquals(ipAddress, IetfInetUtil.INSTANCE.ipAddressFor(INET_SOCKET_ADDRESS.getAddress()));
+        assertEquals(ipAddress, IetfInetUtil.ipAddressFor(INET_SOCKET_ADDRESS.getAddress()));
     }
 
     @Test
index aded30d4d77e2bed1d8e36c85417c44c683f67f2..4b00647d5216e80fbe9b3ae4eb4d0ffe88abac91 100644 (file)
     <artifactId>openflowplugin</artifactId>
     <packaging>bundle</packaging>
 
-    <build>
-        <plugins>
-            <plugin>
-                <groupId>org.apache.felix</groupId>
-                <artifactId>maven-bundle-plugin</artifactId>
-            </plugin>
-        </plugins>
-    </build>
     <dependencies>
         <dependency>
             <groupId>${project.groupId}</groupId>
             <artifactId>openflow-protocol-api</artifactId>
         </dependency>
 
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <optional>true</optional>
+        </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
         </dependency>
 
-        <dependency>
-            <groupId>org.slf4j</groupId>
-            <artifactId>slf4j-simple</artifactId>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>${project.groupId}.openflowjava</groupId>
             <artifactId>openflowjava-util</artifactId>
index 0743a09c4388cca562e52223ea94de14b0c0c997..7aa60281d6f909f61165a0fb1b552a6d232085a8 100644 (file)
@@ -114,7 +114,7 @@ public final class IpConversionUtil {
     */
 
     public static Ipv4Prefix createPrefix(final Ipv4Address ipv4Address) {
-        return IetfInetUtil.INSTANCE.ipv4PrefixFor(ipv4Address);
+        return IetfInetUtil.ipv4PrefixFor(ipv4Address);
     }
 
     public static Ipv4Prefix createPrefix(final Ipv4Address ipv4Address, final String mask) {
@@ -132,7 +132,7 @@ public final class IpConversionUtil {
     }
 
     public static Ipv4Prefix createPrefix(final Ipv4Address ipv4Address, final int intmask) {
-        return IetfInetUtil.INSTANCE.ipv4PrefixFor(ipv4Address, intmask);
+        return IetfInetUtil.ipv4PrefixFor(ipv4Address, intmask);
     }
 
     public static Ipv4Prefix createPrefix(final Ipv4Address ipv4Address, final byte [] bytemask) {
@@ -140,11 +140,11 @@ public final class IpConversionUtil {
             return createPrefix(ipv4Address);
         }
 
-        return IetfInetUtil.INSTANCE.ipv4PrefixFor(ipv4Address, countBits(bytemask));
+        return IetfInetUtil.ipv4PrefixFor(ipv4Address, countBits(bytemask));
     }
 
     public static Ipv6Prefix createPrefix(final Ipv6Address ipv6Address) {
-        return IetfInetUtil.INSTANCE.ipv6PrefixFor(ipv6Address);
+        return IetfInetUtil.ipv6PrefixFor(ipv6Address);
     }
 
     public static Ipv6Prefix createPrefix(final Ipv6Address ipv6Address, final String mask) {
@@ -162,7 +162,7 @@ public final class IpConversionUtil {
     }
 
     public static Ipv6Prefix createPrefix(final Ipv6Address ipv6Address, final int intmask) {
-        return IetfInetUtil.INSTANCE.ipv6PrefixFor(ipv6Address, intmask);
+        return IetfInetUtil.ipv6PrefixFor(ipv6Address, intmask);
     }
 
     public static Ipv6Prefix createPrefix(final Ipv6Address ipv6Address, final byte [] bytemask) {
@@ -170,7 +170,7 @@ public final class IpConversionUtil {
             return createPrefix(ipv6Address);
         }
 
-        return IetfInetUtil.INSTANCE.ipv6PrefixFor(ipv6Address, countBits(bytemask));
+        return IetfInetUtil.ipv6PrefixFor(ipv6Address, countBits(bytemask));
     }
 
     public static DottedQuad createArbitraryBitMask(final byte[] bitmask)  {
@@ -207,11 +207,11 @@ public final class IpConversionUtil {
     }
 
     public static Integer extractPrefix(final Ipv4Prefix ipv4Prefix) {
-        return IetfInetUtil.INSTANCE.splitIpv4Prefix(ipv4Prefix).getValue();
+        return IetfInetUtil.splitIpv4Prefix(ipv4Prefix).getValue();
     }
 
     public static Integer extractPrefix(final Ipv6Prefix ipv6Prefix) {
-        return IetfInetUtil.INSTANCE.splitIpv6Prefix(ipv6Prefix).getValue();
+        return IetfInetUtil.splitIpv6Prefix(ipv6Prefix).getValue();
     }
 
     public static Integer extractPrefix(final Ipv4Address ipv4Prefix) {
@@ -656,11 +656,11 @@ public final class IpConversionUtil {
     }
 
     public static Ipv6Address extractIpv6Address(final Ipv6Prefix ipv6Prefix) {
-        return IetfInetUtil.INSTANCE.ipv6AddressFrom(ipv6Prefix);
+        return IetfInetUtil.ipv6AddressFrom(ipv6Prefix);
     }
 
     public static Ipv4Address extractIpv4Address(final Ipv4Prefix ipv4Prefix) {
-        return IetfInetUtil.INSTANCE.ipv4AddressFrom(ipv4Prefix);
+        return IetfInetUtil.ipv4AddressFrom(ipv4Prefix);
     }
 
     public static DottedQuad extractIpv4AddressMask(final Ipv4Prefix ipv4Prefix) {
@@ -692,7 +692,7 @@ public final class IpConversionUtil {
     }
 
     public static Integer extractIpv6Prefix(final Ipv6Prefix ipv6Prefix) {
-        return IetfInetUtil.INSTANCE.splitIpv6Prefix(ipv6Prefix).getValue();
+        return IetfInetUtil.splitIpv6Prefix(ipv6Prefix).getValue();
     }
 
     public static int countBits(final byte[] mask) {
index 20b111f1da9011437beb64191fc30498c9a3a918..ca59d2542b48b84c4c1663e2bcc324d21d24b2d5 100644 (file)
@@ -403,7 +403,7 @@ public class MatchConvertor extends Convertor<Match, List<MatchEntry>, VersionCo
             boolean hasMask = ethernetDestination.getMask() != null;
 
             if (hasMask) {
-                ethDstBuilder.setMask(IetfYangUtil.INSTANCE.macAddressBytes(ethernetDestination.getMask()));
+                ethDstBuilder.setMask(IetfYangUtil.macAddressBytes(ethernetDestination.getMask()));
             }
 
             ethDstCaseBuilder.setEthDst(ethDstBuilder.build());
@@ -423,7 +423,7 @@ public class MatchConvertor extends Convertor<Match, List<MatchEntry>, VersionCo
             boolean hasMask = ethernetSource.getMask() != null;
 
             if (hasMask) {
-                ethDstBuilder.setMask(IetfYangUtil.INSTANCE.macAddressBytes(ethernetSource.getMask()));
+                ethDstBuilder.setMask(IetfYangUtil.macAddressBytes(ethernetSource.getMask()));
             }
 
             ethSrcCaseBuilder.setEthSrc(ethDstBuilder.build());
index bd1eacc3b1b14433cc4732a8607cb0b3a72aed64..662ff0d3a04fc41355881fccffc6244d92f2eef9 100644 (file)
@@ -115,13 +115,13 @@ public class MatchV10ResponseConvertor extends Convertor<MatchV10, MatchBuilder,
         if (!source.getWildcards().getDLTYPE() && source.getNwSrc() != null) {
             final Ipv4Prefix prefix;
             if (source.getNwSrcMask() != null) {
-                prefix = IetfInetUtil.INSTANCE.ipv4PrefixFor(source.getNwSrc(), source.getNwSrcMask().toJava());
+                prefix = IetfInetUtil.ipv4PrefixFor(source.getNwSrc(), source.getNwSrcMask().toJava());
             } else {
                 //Openflow Spec : 1.3.2
                 //An all-one-bits oxm_mask is equivalent to specifying 0 for oxm_hasmask and omitting oxm_mask.
                 // So when user specify 32 as a mast, switch omit that mast and we get null as a mask in flow
                 // statistics response.
-                prefix = IetfInetUtil.INSTANCE.ipv4PrefixFor(source.getNwSrc());
+                prefix = IetfInetUtil.ipv4PrefixFor(source.getNwSrc());
             }
             if (!NO_IP.equals(prefix.getValue())) {
                 ipv4MatchBuilder.setIpv4Source(prefix);
@@ -131,13 +131,13 @@ public class MatchV10ResponseConvertor extends Convertor<MatchV10, MatchBuilder,
         if (!source.getWildcards().getDLTYPE() && source.getNwDst() != null) {
             final Ipv4Prefix prefix;
             if (source.getNwDstMask() != null) {
-                prefix = IetfInetUtil.INSTANCE.ipv4PrefixFor(source.getNwDst(), source.getNwDstMask().toJava());
+                prefix = IetfInetUtil.ipv4PrefixFor(source.getNwDst(), source.getNwDstMask().toJava());
             } else {
                 //Openflow Spec : 1.3.2
                 //An all-one-bits oxm_mask is equivalent to specifying 0 for oxm_hasmask and omitting oxm_mask.
                 // So when user specify 32 as a mast, switch omit that mast and we get null as a mask in flow
                 // statistics response.
-                prefix = IetfInetUtil.INSTANCE.ipv4PrefixFor(source.getNwDst());
+                prefix = IetfInetUtil.ipv4PrefixFor(source.getNwDst());
             }
             if (!NO_IP.equals(prefix.getValue())) {
                 ipv4MatchBuilder.setIpv4Destination(prefix);
index 7d9e1476caf2e87dfb7b2960c23827dc1aabdd8d..d40a766a1a2ee079003375c8fa36c9ce37cad4ad 100644 (file)
@@ -41,7 +41,7 @@ public class OfToSalArpShaCase extends ConvertorCase<ArpShaCase, MatchBuilder, M
             byte[] mask = arpSha.getMask();
 
             if (mask != null) {
-                arpSourceHardwareAddressBuilder.setMask(IetfYangUtil.INSTANCE.macAddressFor(mask));
+                arpSourceHardwareAddressBuilder.setMask(IetfYangUtil.macAddressFor(mask));
             }
 
             arpMatchBuilder.setArpSourceHardwareAddress(arpSourceHardwareAddressBuilder.build());
index ea67f59a310a3dd1685b66ff1b0e0d5ed0925079..f60c3d6e654a28e196c8905e5cf1efa50b68b5f7 100644 (file)
@@ -41,7 +41,7 @@ public class OfToSalArpThaCase extends ConvertorCase<ArpThaCase, MatchBuilder, M
             byte[] mask = arpTha.getMask();
 
             if (mask != null) {
-                arpTargetHardwareAddressBuilder.setMask(IetfYangUtil.INSTANCE.macAddressFor(mask));
+                arpTargetHardwareAddressBuilder.setMask(IetfYangUtil.macAddressFor(mask));
             }
 
             arpMatchBuilder.setArpTargetHardwareAddress(arpTargetHardwareAddressBuilder.build());
index 23d886baf87d47be25920721363cb09fb0f8c9ab..e2e4c0a22b31c1ba462b9c12ee496b363cdd2711 100644 (file)
@@ -39,7 +39,7 @@ public class OfToSalEthDstCase extends ConvertorCase<EthDstCase, MatchBuilder, M
             byte[] destinationMask = ethDstCase.getMask();
 
             if (destinationMask != null) {
-                ethDestinationBuilder.setMask(IetfYangUtil.INSTANCE.macAddressFor(destinationMask));
+                ethDestinationBuilder.setMask(IetfYangUtil.macAddressFor(destinationMask));
             }
 
             ethMatchBuilder.setEthernetDestination(ethDestinationBuilder.build());
index f737c4a0a509dd5020f03ec144bfe48a43f8313b..98fec7913cfed498446f78ee2d323f91cade738a 100644 (file)
@@ -39,7 +39,7 @@ public class OfToSalEthSrcCase extends ConvertorCase<EthSrcCase, MatchBuilder, M
             byte[] mask = ethSrcCase.getMask();
 
             if (mask != null) {
-                ethSourceBuilder.setMask(IetfYangUtil.INSTANCE.macAddressFor(mask));
+                ethSourceBuilder.setMask(IetfYangUtil.macAddressFor(mask));
             }
 
             ethMatchBuilder.setEthernetSource(ethSourceBuilder.build());
index e2ce21f88db17d43b5c1016ae8961501b06b4911..2090d65fb1e7d84c53d9222d17f65bef90c0c440 100644 (file)
@@ -126,7 +126,7 @@ public class SalToOfArpMatchCase extends ConvertorCase<ArpMatch, List<MatchEntry
             arpShaBuilder.setMacAddress(arpSourceHardwareAddress.getAddress());
             boolean hasMask = false;
             if (null != arpSourceHardwareAddress.getMask()) {
-                arpShaBuilder.setMask(IetfYangUtil.INSTANCE.macAddressBytes(arpSourceHardwareAddress.getMask()));
+                arpShaBuilder.setMask(IetfYangUtil.macAddressBytes(arpSourceHardwareAddress.getMask()));
                 hasMask = true;
             }
             arpShaCaseBuilder.setArpSha(arpShaBuilder.build());
@@ -146,7 +146,7 @@ public class SalToOfArpMatchCase extends ConvertorCase<ArpMatch, List<MatchEntry
             arpThaBuilder.setMacAddress(arpTargetHardwareAddress.getAddress());
             boolean hasMask = false;
             if (null != arpTargetHardwareAddress.getMask()) {
-                arpThaBuilder.setMask(IetfYangUtil.INSTANCE.macAddressBytes(arpTargetHardwareAddress.getMask()));
+                arpThaBuilder.setMask(IetfYangUtil.macAddressBytes(arpTargetHardwareAddress.getMask()));
                 hasMask = true;
             }
             arpThaCaseBuilder.setArpTha(arpThaBuilder.build());
index 95d78893f303d5bd6a617d4cf80728d6e37e4a7b..dd357ea1f6644c85e998810aaa418da01a490cdb 100644 (file)
@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>binding-parent</artifactId>
-        <version>11.0.7</version>
+        <version>12.0.0</version>
         <relativePath/>
     </parent>
 
             <dependency>
                 <groupId>org.opendaylight.controller</groupId>
                 <artifactId>controller-artifacts</artifactId>
-                <version>7.0.4</version>
+                <version>8.0.0</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
             <dependency>
                 <groupId>org.opendaylight.infrautils</groupId>
                 <artifactId>infrautils-artifacts</artifactId>
-                <version>5.0.3</version>
+                <version>6.0.1</version>
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
index f541746d5cdc00b53da9bee770993ac4b7d1ba80..7a2dbfb9af34a47108435e6ef1711b13933879dc 100644 (file)
@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.opendaylight.odlparent</groupId>
         <artifactId>odlparent-lite</artifactId>
-        <version>12.0.3</version>
+        <version>13.0.3</version>
         <relativePath/>
     </parent>
 
index 0a1ff72cd94393d580d1c158f8f0e208d976c87a..d7703b5e6c505b037ed97bb8e056ef16d1cc06a8 100644 (file)
@@ -172,7 +172,7 @@ abstract class AbstractDropTest implements Listener<PacketReceived>, AutoCloseab
             //TODO: use HEX, use binary form
             //Hex.decodeHex("000000000001".toCharArray());
 
-            ethSourceBuilder.setAddress(IetfYangUtil.INSTANCE.macAddressFor(srcMac));
+            ethSourceBuilder.setAddress(IetfYangUtil.macAddressFor(srcMac));
             ethernetMatch.setEthernetSource(ethSourceBuilder.build());
             match.setEthernetMatch(ethernetMatch.build());
 
index c2ecea00b0917a3f87514955a4b10989b789c113..5f2b9d7acae7594701c0d9f7538fee0911ca7c2d 100644 (file)
         <developerConnection>scm:git:ssh://git.opendaylight.org:29418/openflowplugin.git</developerConnection>
     </scm>
     <dependencies>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <optional>true</optional>
+        </dependency>
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>