Make Netty-3 dependency optional

Akka is using an long-obsolete version of Netty in for classic remoting.
While this is not a real problem by default, as the user needs to
configure it for the classes to be loaded, it is tripping up static
analysis tools.

This patch makes the org.jboss.netty.* import optional and removes our
packaging of Netty-3. This means that Akka Classic Remoting cannot be
configured unless the user also packages Netty-3 themselves.

JIRA: CONTROLLER-2119
Change-Id: I0a1bf611f140f2d5b6b83700893d0d69a3ba6115
Signed-off-by: Peter Suna <peter.suna@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/akka/repackaged-akka/pom.xml b/akka/repackaged-akka/pom.xml
index cc222188b55f9b8d12988b0ea9314409c978d864..f15478d0e6b3a7a7d12901a013c800726e8f6740 100644 (file)
--- a/akka/repackaged-akka/pom.xml
+++ b/akka/repackaged-akka/pom.xml
@@ -54,11 +54,6 @@
             <groupId>io.aeron</groupId>
             <artifactId>aeron-driver</artifactId>
         </dependency>
-        <dependency>
-            <groupId>io.netty</groupId>
-            <artifactId>netty</artifactId>
-            <version>3.10.6.Final</version>
-        </dependency>
         <dependency>
             <groupId>org.agrona</groupId>
             <artifactId>agrona</artifactId>
@@ -223,6 +218,7 @@
                             org.fusesource.leveldbjni;resolution:=optional,
                             org.iq80.leveldb;resolution:=optional,
                             org.iq80.leveldb.impl;resolution:=optional,
+                            org.jboss.netty.*;resolution:=optional,
                             *
                         </Import-Package>
                     </instructions>

diff --git a/features/odl-controller-akka/src/main/history/dependencies.xml b/features/odl-controller-akka/src/main/history/dependencies.xml
index 4e7493fb0bb0dda0bde9b8d180a3f709c78d5bf8..05a9908e3544816c57add48abe29585ce87a94b6 100644 (file)
--- a/features/odl-controller-akka/src/main/history/dependencies.xml
+++ b/features/odl-controller-akka/src/main/history/dependencies.xml
@@ -6,7 +6,6 @@
         <bundle>mvn:com.typesafe/ssl-config-core_2.13/0.4.3</bundle>
         <bundle>mvn:io.aeron/aeron-client/1.38.1</bundle>
         <bundle>mvn:io.aeron/aeron-driver/1.38.1</bundle>
-        <bundle>mvn:io.netty/netty/3.10.6.Final</bundle>
         <bundle>mvn:org.agrona/agrona/1.15.2</bundle>
         <bundle>mvn:org.opendaylight.controller/repackaged-akka/${project.version}</bundle>
         <bundle>mvn:org.reactivestreams/reactive-streams/1.0.4</bundle>