From: Jarno Rajahalme Date: Mon, 26 Aug 2013 20:39:37 +0000 (-0700) Subject: ofproto-dpif.at: Remove push_vlan from an OF1.0 test. X-Git-Tag: sliver-openvswitch-2.0.90-1~18^2~15 X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=78cef7fdf84169a4031696f7bc00aa11ca5c1392;p=sliver-openvswitch.git 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 --- 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