LldpLinkProvider: ignore logical port earlier
- No functional change.
When port link state went down,
old code attempted to remove logical port,
which has never been added to LinkDiscovery helper.
Change-Id: Ic6921d99944eecd72bcae02ad3d974836595e1a9
diff --git a/providers/lldp/src/main/java/org/onosproject/provider/lldp/impl/LldpLinkProvider.java b/providers/lldp/src/main/java/org/onosproject/provider/lldp/impl/LldpLinkProvider.java
index c923f78..d723ce1 100644
--- a/providers/lldp/src/main/java/org/onosproject/provider/lldp/impl/LldpLinkProvider.java
+++ b/providers/lldp/src/main/java/org/onosproject/provider/lldp/impl/LldpLinkProvider.java
@@ -353,6 +353,10 @@
* or calls {@link #removePort(Port)} otherwise.
*/
private void updatePort(LinkDiscovery discoverer, Port port) {
+ if (port.number().isLogical()) {
+ // silently ignore logical ports
+ return;
+ }
if (rules.isSuppressed(port)) {
log.trace("LinkDiscovery from {} disabled by configuration", port);
removePort(port);
@@ -365,9 +369,7 @@
return;
}
- if (!port.number().isLogical()) {
- discoverer.addPort(port);
- }
+ discoverer.addPort(port);
}
/**