Control plane redirect for OSPF traffic

Change-Id: I1d31bceadce6f67070a14afaaebdeab07d16f40a

Control plane redirect for OSPF traffic
Change-Id: I1d31bceadce6f67070a14afaaebdeab07d16f40a
Jonathan Hart
Commit ea49238329426d8e212e27892fb1591d134114b7 ea492383 1 parent 38f561c1
Showing 2 changed files with 84 additions and 25 deletions
apps/routing-api/src/main/java/org/onosproject/routing/config/RouterConfig.java
apps/routing/src/main/java/org/onosproject/routing/impl/ControlPlaneRedirectManager.java
--- a/apps/routing-api/src/main/java/org/onosproject/routing/config/RouterConfig.java
View file @ea49238
+++ b/apps/routing-api/src/main/java/org/onosproject/routing/config/RouterConfig.java
View file @ea49238
@@ -25,7 +25,9 @@ import org.onosproject.net.config.Config;
  */
 public class RouterConfig extends Config<ApplicationId> {
-    public static final String CP_CONNECT_POINT = "controlPlaneConnectPoint";
+    private static final String CP_CONNECT_POINT = "controlPlaneConnectPoint";
+    private static final String OSPF_ENABLED = "ospfEnabled";
+    private static final String PIM_ENABLED = "pimEnabled";
     /**
      * Returns the routing control plane connect point.
@@ -35,4 +37,22 @@ public class RouterConfig extends Config<ApplicationId> {
     public ConnectPoint getControlPlaneConnectPoint() {
         return ConnectPoint.deviceConnectPoint(object.path(CP_CONNECT_POINT).asText());
     }
+
+    /**
+     * Returns whether OSPF is enabled on this router.
+     *
+     * @return true if OSPF is enabled, otherwise false
+     */
+    public boolean getOspfEnabled() {
+        return object.path(OSPF_ENABLED).asBoolean(false);
+    }
+
+    /**
+     * Returns whether PIM is enabled on this router.
+     *
+     * @return true if PIM is enabled, otherwise false
+     */
+    public boolean pimEnabled() {
+        return object.path(PIM_ENABLED).asBoolean(false);
+    }
 }
--- a/apps/routing/src/main/java/org/onosproject/routing/impl/ControlPlaneRedirectManager.java
View file @ea49238
+++ b/apps/routing/src/main/java/org/onosproject/routing/impl/ControlPlaneRedirectManager.java
View file @ea49238
@@ -59,11 +59,13 @@ public class ControlPlaneRedirectManager {
     private final Logger log = getLogger(getClass());
     private static final int PRIORITY = 40001;
+    private static final int OSPF_IP_PROTO = 0x59;
     private static final String APP_NAME = "org.onosproject.cpredirect";
     private ApplicationId appId;
     private ConnectPoint controlPlaneConnectPoint;
+    private boolean ospfEnabled = false;
     @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY)
     protected CoreService coreService;
@@ -112,27 +114,31 @@ public class ControlPlaneRedirectManager {
             return;
         }
-        if (!config.getControlPlaneConnectPoint().equals(controlPlaneConnectPoint)) {
+        controlPlaneConnectPoint = config.getControlPlaneConnectPoint();
-            this.controlPlaneConnectPoint = config.getControlPlaneConnectPoint();
+        ospfEnabled = config.getOspfEnabled();
-        }
+
+        updateDevice();
+    }
+    private void updateDevice() {
         if (controlPlaneConnectPoint != null &&
                 deviceService.isAvailable(controlPlaneConnectPoint.deviceId())) {
-            notifySwitchAvailable();
+            DeviceId deviceId = controlPlaneConnectPoint.deviceId();
-        }
-    }
-    private void notifySwitchAvailable() {
+            interfaceService.getInterfaces().stream()
-        DeviceId deviceId = controlPlaneConnectPoint.deviceId();
+                    .filter(intf -> intf.connectPoint().deviceId().equals(deviceId))
+                    .forEach(this::provisionInterface);
-        interfaceService.getInterfaces().stream()
+            log.info("Set up interfaces on {}", controlPlaneConnectPoint.deviceId());
-                .filter(intf -> intf.connectPoint().deviceId().equals(deviceId))
+        }
-                .forEach(this::addInterfaceForwarding);
+    }
-        log.info("Sent interface objectives to {}", controlPlaneConnectPoint.deviceId());
+    private void provisionInterface(Interface intf) {
+        addBasicInterfaceForwarding(intf);
+        updateOspfForwarding(intf);
     }
-    private void addInterfaceForwarding(Interface intf) {
+    private void addBasicInterfaceForwarding(Interface intf) {
         log.debug("Adding interface objectives for {}", intf);
         DeviceId deviceId = controlPlaneConnectPoint.deviceId();
@@ -153,7 +159,7 @@ public class ControlPlaneRedirectManager {
                     .build();
             flowObjectiveService.forward(deviceId,
-                    buildForwardingObjective(toSelector, toTreatment));
+                    buildForwardingObjective(toSelector, toTreatment, true));
             // IPv4 from router
             TrafficSelector fromSelector = DefaultTrafficSelector.builder()
@@ -169,8 +175,7 @@ public class ControlPlaneRedirectManager {
                     .build();
             flowObjectiveService.forward(deviceId,
-                    buildForwardingObjective(fromSelector, intfTreatment));
+                    buildForwardingObjective(fromSelector, intfTreatment, true));
-
             // ARP to router
             toSelector = DefaultTrafficSelector.builder()
@@ -185,7 +190,7 @@ public class ControlPlaneRedirectManager {
                     .build();
             flowObjectiveService.forward(deviceId,
-                    buildForwardingObjective(toSelector, toTreatment));
+                    buildForwardingObjective(toSelector, toTreatment, true));
             // ARP from router
             fromSelector = DefaultTrafficSelector.builder()
@@ -201,22 +206,53 @@ public class ControlPlaneRedirectManager {
                     .build();
             flowObjectiveService.forward(deviceId,
-                    buildForwardingObjective(fromSelector, intfTreatment));
+                    buildForwardingObjective(fromSelector, intfTreatment, true));
         }
     }
+    private void updateOspfForwarding(Interface intf) {
+        // OSPF to router
+        TrafficSelector toSelector = DefaultTrafficSelector.builder()
+                .matchInPort(intf.connectPoint().port())
+                .matchEthType(EthType.EtherType.IPV4.ethType().toShort())
+                .matchVlanId(intf.vlan())
+                .matchIPProtocol((byte) OSPF_IP_PROTO)
+                .build();
+
+        TrafficTreatment toTreatment = DefaultTrafficTreatment.builder()
+                .setOutput(controlPlaneConnectPoint.port())
+                .build();
+
+        flowObjectiveService.forward(controlPlaneConnectPoint.deviceId(),
+                buildForwardingObjective(toSelector, toTreatment, ospfEnabled));
+    }
+
+    /**
+     * Builds a forwarding objective from the given selector and treatment.
+     *
+     * @param selector selector
+     * @param treatment treatment
+     * @param add true to create an add objective, false to create a remove
+     *            objective
+     * @return forwarding objective
+     */
     private ForwardingObjective buildForwardingObjective(TrafficSelector selector,
-                                                         TrafficTreatment treatment) {
+                                                         TrafficTreatment treatment,
+                                                         boolean add) {
-        return DefaultForwardingObjective.builder()
+        ForwardingObjective.Builder fobBuilder = DefaultForwardingObjective.builder()
                 .withSelector(selector)
                 .withTreatment(treatment)
                 .fromApp(appId)
                 .withPriority(PRIORITY)
-                .withFlag(ForwardingObjective.Flag.VERSATILE)
+                .withFlag(ForwardingObjective.Flag.VERSATILE);
-                .add();
+
+        return add ? fobBuilder.add() : fobBuilder.remove();
     }
+    /**
+     * Listener for device events.
+     */
     private class InternalDeviceListener implements DeviceListener {
         @Override
         public void event(DeviceEvent event) {
@@ -227,7 +263,7 @@ public class ControlPlaneRedirectManager {
                 case DEVICE_AVAILABILITY_CHANGED:
                     if (deviceService.isAvailable(event.subject().id())) {
                         log.info("Device connected {}", event.subject().id());
-                        notifySwitchAvailable();
+                        updateDevice();
                     }
                     break;
@@ -244,10 +280,13 @@ public class ControlPlaneRedirectManager {
         }
     }
+    /**
+     * Listener for network config events.
+     */
     private class InternalNetworkConfigListener implements NetworkConfigListener {
         @Override
         public void event(NetworkConfigEvent event) {
-            if (event.subject().equals(RoutingService.ROUTER_CONFIG_CLASS)) {
+            if (event.configClass().equals(RoutingService.ROUTER_CONFIG_CLASS)) {
                 switch (event.type()) {
                 case CONFIG_ADDED:
                 case CONFIG_UPDATED: