Bug fix for ForwardingRulesManager: 89/289/1
authorKalvin Hom <kahom@cisco.com>
Fri, 3 May 2013 02:14:17 +0000 (19:14 -0700)
committerKalvin Hom <kahom@cisco.com>
Fri, 3 May 2013 02:14:17 +0000 (19:14 -0700)
addFlow and addFlowAsync are
reversed.

https://bugs.opendaylight.org/show_bug.cgi?id=16

Change-Id: If119f64e637401ea022b092af44192bdef3db986
Signed-off-by: Kalvin Hom <kahom@cisco.com>
opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/ForwardingRulesManagerImpl.java

index 110d338..eee08d3 100644 (file)
@@ -604,9 +604,9 @@ public class ForwardingRulesManagerImpl implements IForwardingRulesManager,
     private Status addEntriesInternal(FlowEntryInstall entry, boolean async) {
         // Install the flow on the network node
         Status status = (async)?
-                programmer.addFlow(entry.getNode(), entry.getInstall()
-                        .getFlow()) :
                 programmer.addFlowAsync(entry.getNode(), entry.getInstall()
+                        .getFlow()) :
+                programmer.addFlow(entry.getNode(), entry.getInstall()
                             .getFlow());