Deprecate messagebus-netconf 28/95228/2
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 19 Feb 2021 06:07:23 +0000 (07:07 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 19 Feb 2021 06:17:33 +0000 (07:17 +0100)
messagebus-netconf is not used by anyone and has a good number
of deficiencies -- in its implementation as well as in the
framework (from controller).

Deprecate it for removal in the next release.

JIRA: NETCONF-761
Change-Id: I852b23fc30362c4ae609f04da9145580d887eca9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
15 files changed:
netconf/messagebus-netconf/pom.xml
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/ConnectionNotificationTopicRegistration.java
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSource.java
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSourceManager.java
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSourceMount.java
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSourceRegistration.java
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NotificationTopicRegistration.java
netconf/messagebus-netconf/src/main/java/org/opendaylight/netconf/messagebus/eventsources/netconf/StreamNotificationTopicRegistration.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/AbstractCodecTest.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/ConnectionNotificationTopicRegistrationTest.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSourceManagerTest.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSourceMountTest.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfEventSourceTest.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/NetconfTestUtils.java
netconf/messagebus-netconf/src/test/java/org/opendaylight/netconf/messagebus/eventsources/netconf/StreamNotificationTopicRegistrationTest.java

index c420904ffd1200d918735bc64c773b5d896d7597..0428d7ebbbef8e29e005d6da339463077dcfcde2 100644 (file)
@@ -18,9 +18,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
         <relativePath>../../parent</relativePath>
     </parent>
 
-    <groupId>org.opendaylight.netconf</groupId>
     <artifactId>messagebus-netconf</artifactId>
-    <version>1.10.0-SNAPSHOT</version>
     <name>${project.artifactId}</name>
     <packaging>bundle</packaging>
 
index 3999aa285065a51b54d8fde66ea85319d7664769..3cdd66052a2fcee63b62d89e93a03a5c04e007b9 100644 (file)
@@ -37,6 +37,7 @@ import org.w3c.dom.Element;
 /**
  * Topic registration on event-source-status-notification.
  */
+@Deprecated(forRemoval = true)
 class ConnectionNotificationTopicRegistration extends NotificationTopicRegistration {
     private static final Logger LOG = LoggerFactory.getLogger(ConnectionNotificationTopicRegistration.class);
 
index 7f13c7a75b78df35482b11a4c527adc55c7ecb34..72d27ff78ffead58e24fa3c568270a4f3f02ab13 100644 (file)
@@ -69,6 +69,7 @@ import org.w3c.dom.Element;
  * NetconfEventSource serves as proxy between nodes and messagebus. Subscribers can join topic stream from this source.
  * Then they will receive notifications from device that matches pattern specified by topic.
  */
+@Deprecated(forRemoval = true)
 public class NetconfEventSource implements EventSource, DOMNotificationListener {
 
     private static final Logger LOG = LoggerFactory.getLogger(NetconfEventSource.class);
index 6bad11f675b3d0af215964d07937858e543c3e8b..e9fcfc098036d9f06a79fab063159f4b014c68cc 100644 (file)
@@ -39,6 +39,7 @@ import org.slf4j.LoggerFactory;
  * NetconfEventSourceManager implements DataChangeListener. On topology changes, it manages creation,
  * updating and removing registrations of event sources.
  */
+@Deprecated(forRemoval = true)
 public final class NetconfEventSourceManager implements DataTreeChangeListener<Node>, AutoCloseable {
 
     private static final Logger LOG = LoggerFactory.getLogger(NetconfEventSourceManager.class);
index 28871bca6a7300847a89ed0071b962d5501b51de..5f99d1e370e06fb078617b49275a1c11186646e3 100644 (file)
@@ -48,6 +48,7 @@ import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 /**
  * Facade of mounted netconf device.
  */
+@Deprecated(forRemoval = true)
 class NetconfEventSourceMount {
     private static final YangInstanceIdentifier STREAMS_PATH = YangInstanceIdentifier.builder().node(Netconf.QNAME)
             .node(Streams.QNAME).build();
index dcb28acec04976d28e7a9c78e84c9cda5d6ed25c..ede15b0210b281b3af4fa9c3eaf45ad6112bada9 100644 (file)
@@ -32,6 +32,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Helper class to keep connection status of netconf node  and event source registration object.
  */
+@Deprecated(forRemoval = true)
 final class NetconfEventSourceRegistration implements AutoCloseable {
 
     private static final Logger LOG = LoggerFactory.getLogger(NetconfEventSourceRegistration.class);
index b3499a1ecbc34eaf08b8116c5f62e47a6a6eaa3c..286ac1db35c35a2dddb4d01d19ce2d8b59fe562f 100644 (file)
@@ -20,6 +20,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Notification topic registration.
  */
+@Deprecated(forRemoval = true)
 abstract class NotificationTopicRegistration implements AutoCloseable {
 
     private static final Logger LOG = LoggerFactory.getLogger(NotificationTopicRegistration.class);
index f2edf047aff1eb64712f6cfdf665955f542841d0..d6752179558e1000306c69293ff44c0724b8cf38 100644 (file)
@@ -25,6 +25,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Topic registration for notification with specified namespace from stream.
  */
+@Deprecated(forRemoval = true)
 class StreamNotificationTopicRegistration extends NotificationTopicRegistration {
 
     private static final Logger LOG = LoggerFactory.getLogger(StreamNotificationTopicRegistration.class);
index 50d9805d856f7021af6ae471bbc31fcaae016858..caf6f5f37a2677de58122e38ecf4bea8d66f5d67 100644 (file)
@@ -13,6 +13,7 @@ import org.opendaylight.mdsal.binding.dom.codec.impl.BindingCodecContext;
 import org.opendaylight.mdsal.binding.runtime.spi.BindingRuntimeHelpers;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.netmod.notification.rev080714.Netconf;
 
+@Deprecated(forRemoval = true)
 public abstract class AbstractCodecTest {
     static BindingCodecContext SERIALIZER;
 
index c9d0f3ebdc7850531a2ba0ea773452c19b572c3f..78b25847da91eb70d47711d9937fedfe59dff346 100644 (file)
@@ -30,6 +30,7 @@ import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.messagebus.even
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.DataContainerChild;
 
+@Deprecated(forRemoval = true)
 @RunWith(MockitoJUnitRunner.StrictStubs.class)
 public class ConnectionNotificationTopicRegistrationTest {
 
index f61ea9baaa6e44df43f181fc4eacaa28c5c7771e..df38c0904dc065b32cb6e73ef2000402c54938c2 100644 (file)
@@ -46,8 +46,8 @@ import org.opendaylight.yangtools.util.concurrent.FluentFutures;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 
+@Deprecated(forRemoval = true)
 public class NetconfEventSourceManagerTest extends AbstractCodecTest {
-
     private NetconfEventSourceManager netconfEventSourceManager;
     private ListenerRegistration<?> listenerRegistrationMock;
     private DOMMountPointService domMountPointServiceMock;
index a2d1adbcc1aaf1d9c284aa990ffedee00ae12255..ec06c223b412c9fc9bc08d6098a13cc869645749 100644 (file)
@@ -45,6 +45,7 @@ import org.opendaylight.yangtools.yang.data.api.schema.ContainerNode;
 import org.opendaylight.yangtools.yang.data.api.schema.LeafNode;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 
+@Deprecated(forRemoval = true)
 @RunWith(MockitoJUnitRunner.StrictStubs.class)
 public class NetconfEventSourceMountTest extends AbstractCodecTest {
     public static final String STREAM_1 = "stream-1";
index edf79ebb1f82d0146cacd6658bfc381166abe0d5..7ddbd9ea7bcbd0b090001135d1a8cdd11ed2a43f 100644 (file)
@@ -53,6 +53,7 @@ import org.opendaylight.yangtools.yang.model.api.NotificationDefinition;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.stmt.SchemaNodeIdentifier.Absolute;
 
+@Deprecated(forRemoval = true)
 @RunWith(MockitoJUnitRunner.StrictStubs.class)
 public class NetconfEventSourceTest {
     private static final Absolute NOTIFICATION_1_PATH = Absolute.of(QName.create("ns1", "1970-01-15", "not1"));
index 748963293a5ce64f1cc8d311294eaf4fb624b215..d6fcceab0de474f9349f896a744aa584224ccb91 100644 (file)
@@ -44,6 +44,7 @@ import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
 import org.opendaylight.yangtools.yang.data.impl.schema.Builders;
 import org.opendaylight.yangtools.yang.data.impl.schema.builder.api.CollectionNodeBuilder;
 
+@Deprecated(forRemoval = true)
 public final class NetconfTestUtils {
     public static final String NOTIFICATION_CAPABILITY_PREFIX = "(urn:ietf:params:xml:ns:netconf:notification";
 
index d1123f7476758a22e1f24612b85311154c4439b3..436245400a343752e9ecd801d4d0e198a080d02a 100644 (file)
@@ -37,6 +37,7 @@ import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeBuilder;
 import org.opendaylight.yangtools.concepts.ListenerRegistration;
 
+@Deprecated(forRemoval = true)
 @RunWith(MockitoJUnitRunner.StrictStubs.class)
 public class StreamNotificationTopicRegistrationTest {