On Fri, Jul 29, 2016 at 11:18 AM, Russell Bryant <russ...@ovn.org> wrote:

> From: Babu Shanmugam <bscha...@redhat.com>
>
> Recent commits reorganizing bindings handling and also moving ct zone
> assignment to ovn-controller.c caused ct zone assignment to no longer
> work.  The code relies on an "all_lports" sset that should contain all
> logical ports that we should be assigning ct zones for.  Prior to this
> change, all_lports was always empty.
>
> Signed-off-by: Babu Shanmugam <bscha...@redhat.com>
> Co-authored-by: Russell Bryant <russ...@ovn.org>
> Signed-off-by: Russell Bryant <russ...@ovn.org>
> @@ -434,7 +437,9 @@ main(int argc, char *argv[])
>          if (chassis_id) {
>              chassis = chassis_run(&ctx, chassis_id);
>              encaps_run(&ctx, br_int, chassis_id);
> -            binding_run(&ctx, br_int, chassis_id, &local_datapaths);
> +            binding_run(&ctx, br_int, chassis_id, &local_datapaths,
> +                        &all_lports);
> +            VLOG_INFO("all_lports size: %lu", sset_count(&all_lports));
>

Consider this stray debug entry removed ...

-- 
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to