[ovs-dev] [PATCH v2] ovn-controller: Add physical/logical translation for containers.

[Ben Pfaff]

On Mon, May 18, 2015 at 06:03:41AM -0700, Gurucharan Shetty wrote:
> When containers are running inside VMs and the openflow flows
> are added in the hypervisor, the physical to logical translation
> (and vice versa) needs to handle the VLAN tags that the packet
> comes with.
> 
> Signed-off-by: Gurucharan Shetty <shettyg at nicira.com>
> ---
> v1->v2:
> * For container traffic that are tagged, strip the vlan in a rule
> that has a higher priority than the rule used for VMs.

Thanks for the update!

In this kind of situation I'd tend to just write 'tag ? 150 : 100'
for the priority.
> +            if (tag) {
> +                ofctrl_add_flow(0, 150, &match, &ofpacts);
> +            } else {
> +                ofctrl_add_flow(0, 100, &match, &ofpacts);
> +            }

Acked-by: Ben Pfaff <blp at nicira.com>