Jonathan Hart

Compare ConnectPoints rather than Interfaces.

Now we can have multiple Interfaces per port, and the old code results in an
intent using the same port for ingress and egress.

Change-Id: I9f9ad396a0c6e7d31d654896f31c59a34240f82f
...@@ -598,7 +598,7 @@ public class Router implements RouteListener { ...@@ -598,7 +598,7 @@ public class Router implements RouteListener {
598 Set<ConnectPoint> ingressPorts = new HashSet<>(); 598 Set<ConnectPoint> ingressPorts = new HashSet<>();
599 599
600 for (Interface intf : interfaceService.getInterfaces()) { 600 for (Interface intf : interfaceService.getInterfaces()) {
601 - if (!intf.equals(egressInterface)) { 601 + if (!intf.connectPoint().equals(egressInterface.connectPoint())) {
602 ConnectPoint srcPort = intf.connectPoint(); 602 ConnectPoint srcPort = intf.connectPoint();
603 ingressPorts.add(srcPort); 603 ingressPorts.add(srcPort);
604 } 604 }
......