Workaround for flow mod semantic differences between OF 1.0 and
1.3.
The intent code always used to use MODIFY_STRICT flow mods to install flows,
however MODIFY* flows in OF 1.3 don't create the flow if it doesn't already
exist.
We're testing using ADD flow mods instead.
Change-Id: Ida709f27cf104b3a18a10e542f8f735706800168
diff --git a/src/main/java/net/onrc/onos/core/intent/FlowEntry.java b/src/main/java/net/onrc/onos/core/intent/FlowEntry.java
index 53a8c5a..379426f 100644
--- a/src/main/java/net/onrc/onos/core/intent/FlowEntry.java
+++ b/src/main/java/net/onrc/onos/core/intent/FlowEntry.java
@@ -151,7 +151,11 @@
switch (operator) {
case ADD:
- builder = factory.buildFlowModifyStrict();
+ // XXX Workaround for semantic differences between OF1.0 and 1.3.
+ // The original code for 1.0 used MODIFY_STRICT but in 1.3 a
+ // MODIFY* command won't create the flow entry if it doesn't
+ // already exist.
+ builder = factory.buildFlowAdd();
break;
case REMOVE:
builder = factory.buildFlowDeleteStrict();