commit 306633a0ce13254be6beb24499667f99dc9c5284
author Sangho Shin <sangho@onlab.us> Mon Oct 20 14:26:55 2014 -0700
committer Sangho Shin <sangho@onlab.us> Mon Oct 20 14:26:55 2014 -0700
tree adbbd065ab85973fe3fbf05120c4cfe36f45c185
parent 816554403360a1135f7752239294e397c5312fd7

Modify the return types of the policy related public functions following the SRM interface.

Change-Id: I2801394629e9efcb0223d06d053e527f37cbe51d

src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java

diff --git a/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java b/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java
index 5f3e11b..74f2a92 100644
--- a/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java
+++ b/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java
@@ -1085,8 +1085,9 @@
      *
      * @param sw
      * @param mplsLabel
+     * @return
      */
-    public void createPolicy(String pid, MACAddress srcMac, MACAddress dstMac,
+    public boolean createPolicy(String pid, MACAddress srcMac, MACAddress dstMac,
             Short etherType, IPv4Net srcIp, IPv4Net dstIp, Byte ipProto,
             Short srcTcpPort, Short dstTcpPort, int priority, String tid) {
 
@@ -1137,12 +1138,15 @@
                     sw13.pushFlow(maEntry);
                 } catch (IOException e) {
                     e.printStackTrace();
+                    return false;
                 }
             }
         }
 
         PolicyInfo policyInfo = new PolicyInfo(pid, policyMatch, priority, tid);
         policyTable.put(pid, policyInfo);
+
+        return true;
     }
 
     /**
@@ -1252,8 +1256,9 @@
      * @param srcTcpPort
      * @param dstTcpPort
      * @param tid
+     * @return
      */
-    public void removePolicy(String pid) {
+    public boolean removePolicy(String pid) {
         PolicyInfo policyInfo =  policyTable.get(pid);
         PacketMatch policyMatch = policyInfo.match;
         String tid = policyInfo.tunnelId;
@@ -1289,18 +1294,19 @@
                 } catch (IOException e) {
                     e.printStackTrace();
                     log.debug("policy remove failed due to pushFlow() exception");
-                    return;
+                    return false;
                 }
             }
         }
 
         log.debug("Policy {} is removed.", pid);
+        return true;
     }
 
 
-    public void removeTunnel(String tunnelId) {
+    public boolean removeTunnel(String tunnelId) {
 
-
+        return true;
     }
 
     // ************************************