From: Russell Bryant Date: Mon, 25 Jan 2016 21:54:06 +0000 (-0500) Subject: ovn-controller: Allocate ct zones for localnet ports. X-Git-Url: http://git.cascardo.eti.br/?a=commitdiff_plain;h=5a4bee8a022ff99ff4efe18c74412d41926146fa;hp=e90aeb578ffc0cbae377b6251c2d956de98dacad;p=cascardo%2Fovs.git ovn-controller: Allocate ct zones for localnet ports. Previously, all ct() actions applied to localnet ports used the default conntrack zone. We should allocate a ct zone ID for all localnet ports just like we do for all local VIFs so that none of our connection tracking interferes with any base system connection tracking in the default zone. Signed-off-by: Russell Bryant Acked-by: Han Zhou Acked-by: Ben Pfaff --- diff --git a/ovn/controller/binding.c b/ovn/controller/binding.c index c9649ebf8..ce9cccfd4 100644 --- a/ovn/controller/binding.c +++ b/ovn/controller/binding.c @@ -189,6 +189,13 @@ binding_run(struct controller_ctx *ctx, const struct ovsrec_bridge *br_int, sbrec_port_binding_set_chassis(binding_rec, chassis_rec); } else if (binding_rec->chassis == chassis_rec) { sbrec_port_binding_set_chassis(binding_rec, NULL); + } else if (!binding_rec->chassis + && !strcmp(binding_rec->type, "localnet")) { + /* localnet ports will never be bound to a chassis, but we want + * to list them in all_lports because we want to allocate + * a conntrack zone ID for each one, as we'll be creating + * a patch port for each one. */ + sset_add(&all_lports, binding_rec->logical_port); } }