Use FlowManager's flow ID service rather thanhave Forwarding generate its own IDs
diff --git a/src/main/java/net/onrc/onos/ofcontroller/forwarding/Forwarding.java b/src/main/java/net/onrc/onos/ofcontroller/forwarding/Forwarding.java
index b9124e8..686bee0 100644
--- a/src/main/java/net/onrc/onos/ofcontroller/forwarding/Forwarding.java
+++ b/src/main/java/net/onrc/onos/ofcontroller/forwarding/Forwarding.java
@@ -1,6 +1,5 @@
package net.onrc.onos.ofcontroller.forwarding;
-import java.util.Collection;
import java.util.Iterator;
import net.floodlightcontroller.core.FloodlightContext;
@@ -45,9 +44,6 @@
private IDeviceStorage deviceStorage;
private TopologyManager topologyService;
- // TODO Flow IDs should be globally managed
- private long currentId = 1;
-
public Forwarding() {
}
@@ -155,7 +151,7 @@
dataPath.setSrcPort(srcSwitchPort);
dataPath.setDstPort(dstSwitchPort);
- FlowId flowId = new FlowId(currentId++); //dummy flow ID
+ FlowId flowId = new FlowId(flowService.getNextFlowEntryId());
FlowPath flowPath = new FlowPath();
flowPath.setFlowId(flowId);
flowPath.setInstallerId(new CallerId("Forwarding"));