From 78cef7fdf84169a4031696f7bc00aa11ca5c1392 Mon Sep 17 00:00:00 2001 From: Jarno Rajahalme Date: Mon, 26 Aug 2013 13:39:37 -0700 Subject: [PATCH] ofproto-dpif.at: Remove push_vlan from an OF1.0 test. Remove push_vlan from an OF1.0 test, as it requires OF1.1+ support, but was silently discarded. A later patch will make this test to fail due to validation of usable OpenFlow protocol versions while parsing actions. It should be noted that existing controllers may depend on the silently discarded push_vlan being accepted during the action validation code at the switch, as OpenFlow 1.0 supports setting a vlan ID, which will implicitly push a vlan header, if it did not exist already. Signed-off-by: Jarno Rajahalme Signed-off-by: Ben Pfaff --- tests/ofproto-dpif.at | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ofproto-dpif.at b/tests/ofproto-dpif.at index af19672ac..7f01a2dfd 100644 --- a/tests/ofproto-dpif.at +++ b/tests/ofproto-dpif.at @@ -266,7 +266,7 @@ cookie=0x6 table=4 in_port=83 actions=load:4->NXM_NX_REG3[[]],mod_nw_src:83.83.8 cookie=0x7 table=5 in_port=84 actions=load:5->NXM_NX_REG4[[]],load:6->NXM_NX_TUN_ID[[]],mod_nw_dst:84.84.84.84,controller,resubmit(85,6) cookie=0x8 table=6 in_port=85 actions=mod_tp_src:85,controller,resubmit(86,7) cookie=0x9 table=7 in_port=86 actions=mod_tp_dst:86,controller,controller -cookie=0xa dl_src=40:44:44:44:44:41 actions=push_vlan:0x8100,mod_vlan_vid:99,mod_vlan_pcp:1,controller +cookie=0xa dl_src=40:44:44:44:44:41 actions=mod_vlan_vid:99,mod_vlan_pcp:1,controller cookie=0xa dl_src=40:44:44:44:44:42 actions=push_mpls:0x8847,load:10->OXM_OF_MPLS_LABEL[[]],load:3->OXM_OF_MPLS_TC[[]],controller cookie=0xa dl_src=40:44:44:44:44:43 actions=push_mpls:0x8847,load:10->OXM_OF_MPLS_LABEL[[]],load:3->OXM_OF_MPLS_TC[[]],controller cookie=0xa dl_src=40:44:44:44:44:44 actions=push_mpls:0x8847,load:10->OXM_OF_MPLS_LABEL[[]],load:3->OXM_OF_MPLS_TC[[]],controller -- 2.20.1