From: Matej Perina <matej.perina@pantheon.sk>
Date: Wed, 13 Apr 2016 13:11:10 +0000 (+0200)
Subject: BUG 5028 - Test GBP and SFC against Li OFP
X-Git-Tag: release/boron~88
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=1191a338e675d0628eb376837135e6835cc34589;p=groupbasedpolicy.git

BUG 5028 - Test GBP and SFC against Li OFP

migration to openflowplugin Li

Change-Id: Iafc78069780aa04ed06935fa567e2ee86ccd158f
Signed-off-by: Matej Perina <matej.perina@pantheon.sk>
---

diff --git a/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/OfWriter.java b/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/OfWriter.java
index 5a8c77dd6..72fff6c0d 100755
--- a/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/OfWriter.java
+++ b/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/OfWriter.java
@@ -40,6 +40,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.Fl
 import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.tables.Table;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.tables.TableBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.tables.table.Flow;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.tables.table.FlowBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.flow.MatchBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.group.types.rev131018.GroupId;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.group.types.rev131018.GroupTypes;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.group.types.rev131018.group.BucketsBuilder;
@@ -168,6 +170,9 @@ public class OfWriter {
         Preconditions.checkNotNull(flow);
         Preconditions.checkNotNull(nodeId);
 
+        if (flow.getMatch() == null) {
+            flow = new FlowBuilder(flow).setMatch(new MatchBuilder().build()).build();
+        }
         TableBuilder tableBuilder = this.getTableBuilderForNode(nodeId, tableId);
         // transforming List<Flow> to Set (with customized equals/hashCode) to eliminate duplicate entries
         List<Flow> flows = tableBuilder.getFlow();
@@ -264,6 +269,9 @@ public class OfWriter {
             for (Equivalence.Wrapper<Flow> wf : additions) {
                 Flow f = wf.get();
                 if (f != null) {
+                    if (f.getMatch() == null) {
+                        f = new FlowBuilder(f).setMatch(new MatchBuilder().build()).build();
+                    }
                     t.put(LogicalDatastoreType.CONFIGURATION,
                             FlowUtils.createFlowPath(tableIid, f.getId()), f, true);
                 }
diff --git a/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/FlowTable.java b/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/FlowTable.java
index 743c519cd..51e756a08 100755
--- a/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/FlowTable.java
+++ b/renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/FlowTable.java
@@ -51,7 +51,8 @@ public abstract class FlowTable extends OfTable {
             .setTableId(getTableId())
             .setBarrier(false)
             .setHardTimeout(0)
-            .setIdleTimeout(0);
+            .setIdleTimeout(0)
+            .setMatch(new MatchBuilder().build());
     }
 
     /**