commit 77e18553fb01fba590c18af09f712a56cf33bdde
author David Glantz <david.glantz@windstream.com> Tue Sep 21 11:21:36 2021 -0500
committer David Glantz <david.glantz@windstream.com> Fri Oct 01 13:21:46 2021 +0000
tree 1b66cd96c556e46ae6beec8ee96b9c4bc90d041f
parent ff63198d93e4fa8e3061a815a57a41dda9525b99

Remove logic to create a link when network configuration is added.

Change-Id: I9e26308c0f5ce9c5b70ccdbc37d97e4f480ff227
(cherry picked from commit a4d40f37bfd342c64595fea6c7357d303dd55870)

core/net/src/main/java/org/onosproject/net/link/impl/LinkManager.java

diff --git a/core/net/src/main/java/org/onosproject/net/link/impl/LinkManager.java b/core/net/src/main/java/org/onosproject/net/link/impl/LinkManager.java
index 326020b..ef09ee7 100644
--- a/core/net/src/main/java/org/onosproject/net/link/impl/LinkManager.java
+++ b/core/net/src/main/java/org/onosproject/net/link/impl/LinkManager.java
@@ -370,20 +370,14 @@
             Link link = getLink(src, dst);
             LinkDescription desc;
 
-            if (link == null) {
-                // TODO Revisit this behaviour.
-                // config alone probably should not be adding a link,
-                // netcfg provider should be the one.
-                desc = BasicLinkOperator.descriptionOf(src, dst, cfg);
-            } else {
+            if (link != null) {
                 desc = BasicLinkOperator.combine(cfg,
                         BasicLinkOperator.descriptionOf(src, dst, link));
+                ProviderId pid = Optional.ofNullable(link)
+                        .map(Link::providerId)
+                        .orElse(ProviderId.NONE);
+                store.createOrUpdateLink(pid, desc);
             }
-
-            ProviderId pid = Optional.ofNullable(link)
-                    .map(Link::providerId)
-                    .orElse(ProviderId.NONE);
-            store.createOrUpdateLink(pid, desc);
         }
     }
 }