[ovs-dev] [PATCH 1/2] ofproto-dpif.at: Remove push_vlan from an OF1.0 test.
Jarno Rajahalme
jrajahalme at nicira.com
Mon Aug 26 20:10:34 UTC 2013
On Aug 26, 2013, at 10:45 AM, Ben Pfaff <blp at nicira.com> wrote:
> On Tue, Aug 20, 2013 at 06:41:44PM -0700, Jarno Rajahalme wrote:
>> Remove push_vlan from an OF1.0 test, as it requires OF1.1+ support, but was
>> silently discarded.
>>
>> Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
>
> I agree that we should not expect OF1.1+ features to work in OF1.0,
> but I think that this test result demonstrates a bug somewhere in OVS:
> it is incorrect to silently discard an action. I would prefer to fix
> that, rather than to just stop trying it.
The later patch does that, as it does "usable protocol" analysis while parsing the actions (and matches). After applying the patch 2/2 the test modified here will fail, hence the modification.
The issue is a bit more subtle, as silently ignoring a single vlan push kind of works, as a subsequent setting of a vlan ID will implicitly add the vlan header if it did not exist previously.
Jarno
More information about the dev
mailing list