[ovs-dev] [PATCH v6 2/2] ovn: Add address_set() support for ACLs.

Zong Kai LI zealokii at gmail.com
Thu Jun 30 03:21:58 UTC 2016


I've tested this serial patches, it works as expected.

Only one thing I'm not sure:

diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c
> index a12b247..d85efd5 100644
> --- a/ovn/utilities/ovn-sbctl.c
> +++ b/ovn/utilities/ovn-sbctl.c
> @@ -779,6 +779,10 @@ static const struct ctl_table_class tables[] = {
>       {{&sbrec_table_mac_binding, &sbrec_mac_binding_col_logical_port,
> NULL},
>        {NULL, NULL, NULL}}},
>
> +    {&sbrec_table_address_set,
> +     {{&sbrec_table_address_set, &sbrec_address_set_col_name, NULL},
> +      {NULL, NULL, NULL}}},
> +
>      {NULL, {{NULL, NULL, NULL}, {NULL, NULL, NULL}}}
>  };
>

since Address_Set table is identical to OVN NB DB and OVN SB DB, and
ovn-northd will update data in this table from north to south. So I think
it will make less sense to open Address_Set table to ovn-sbctl.



More information about the dev mailing list