Skip to content

Commit

Permalink
ovn-controller: Allocate ct zones for localnet ports.
Browse files Browse the repository at this point in the history
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 <russell@ovn.org>
Acked-by: Han Zhou <zhouhan@gmail.com>
Acked-by: Ben Pfaff <blp@ovn.org>
  • Loading branch information
russellb committed Feb 3, 2016
1 parent e90aeb5 commit 5a4bee8
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions ovn/controller/binding.c
Expand Up @@ -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);
}
}

Expand Down

0 comments on commit 5a4bee8

Please sign in to comment.