commit 74ec40636b900812788bfa1eadef6be3209d030c
author Hyunsun Moon <hyunsun.moon@gmail.com> Wed Dec 09 10:58:32 2015 -0800
committer Gerrit Code Review <gerrit@onlab.us> Wed Dec 09 20:10:10 2015 +0000
tree 207dd85d537de8d6747df1cd161bd52e611cf3a1
parent 17ed48b5d11e740f5ed37b93583f9aa150ca23f4

Renamed some methods and removed unnecessary L3 flow rules

Change-Id: I44911e4370ac65724716f8c3869c9f8696ef18b0

apps/openstackswitching/app/src/main/java/org/onosproject/openstackswitching/OpenstackSwitchingRulePopulator.java

diff --git a/apps/openstackswitching/app/src/main/java/org/onosproject/openstackswitching/OpenstackSwitchingRulePopulator.java b/apps/openstackswitching/app/src/main/java/org/onosproject/openstackswitching/OpenstackSwitchingRulePopulator.java
index bba8f77..ed89479 100644
--- a/apps/openstackswitching/app/src/main/java/org/onosproject/openstackswitching/OpenstackSwitchingRulePopulator.java
+++ b/apps/openstackswitching/app/src/main/java/org/onosproject/openstackswitching/OpenstackSwitchingRulePopulator.java
@@ -97,7 +97,7 @@
      * @param port port for the VM created
      */
     public void populateSwitchingRules(Device device, Port port) {
-        populateFlowRulesForVMSetTunnelTag(device, port);
+        populateFlowRulesForTunnelTag(device, port);
         populateFlowRulesForTrafficToSameCnode(device, port);
         populateFlowRulesForTrafficToDifferentCnode(device, port);
     }
@@ -108,13 +108,13 @@
      * @param device device to put the rules
      * @param port port info of the VM
      */
-    private void populateFlowRulesForVMSetTunnelTag(Device device, Port port) {
+    private void populateFlowRulesForTunnelTag(Device device, Port port) {
         Ip4Address vmIp = getFixedIpAddressForPort(port.annotations().value("portName"));
         String portName = port.annotations().value("portName");
         String vni = getVniForPort(portName);
 
         if (vmIp != null) {
-            setFlowRuleForVMSetTunnelTag(device.id(), port, vni);
+            setFlowRuleForTunnelTag(device.id(), port, vni);
         }
     }
 
@@ -253,7 +253,7 @@
         return port.macAddress();
     }
 
-    private void setFlowRuleForVMSetTunnelTag(DeviceId deviceId, Port port, String vni) {
+    private void setFlowRuleForTunnelTag(DeviceId deviceId, Port port, String vni) {
 
         TrafficSelector.Builder sBuilder = DefaultTrafficSelector.builder();
         TrafficTreatment.Builder tBuilder = DefaultTrafficTreatment.builder();
@@ -306,26 +306,6 @@
                 .add();
 
         flowObjectiveService.forward(id, fo);
-
-        //For L3 Ease-West Routing Case
-        sBuilder = DefaultTrafficSelector.builder();
-        tBuilder = DefaultTrafficTreatment.builder();
-
-        sBuilder.matchEthType(Ethernet.TYPE_IPV4)
-                .matchIPDst(ip4Address.toIpPrefix());
-
-        tBuilder.setEthDst(vmMacAddress)
-                .setOutput(port.number());
-
-        fo = DefaultForwardingObjective.builder()
-                .withSelector(sBuilder.build())
-                .withTreatment(tBuilder.build())
-                .withPriority(EAST_WEST_ROUTING_RULE_PRIORITY)
-                .withFlag(ForwardingObjective.Flag.SPECIFIC)
-                .fromApp(appId)
-                .add();
-
-        flowObjectiveService.forward(id, fo);
     }