[ovs-dev] [PATCH 3/3] ovn-northd: Propagate dynamic addresses to port group address sets.
Jakub Sitnicki
jkbs at redhat.com
Mon Jul 30 14:40:12 UTC 2018
On Sat, 28 Jul 2018 23:54:32 -0700
Han Zhou <zhouhan at gmail.com> wrote:
> On Thu, Jul 26, 2018 at 3:51 AM, Jakub Sitnicki <jkbs at redhat.com> wrote:
> >
> > If a logical switch port belongs to a port group and has dynamic
> > addresses assigned, propagate the addresses to the auto-generated
> > address sets for the port group.
> >
> > Signed-off-by: Jakub Sitnicki <jkbs at redhat.com>
> > ---
(...)
> Thanks for the fix! Just a minor problem:
>
> It would be better to check if the address is dynamic using
> is_dynamic_lsp_address() before calling split_addresses(). Otherwise, there
> seems to be annoying INFO log: "invalid syntax '%s' in address" printed if
> the address is "dynamic".
>
> Acked-by: Han Zhou <hzhou8 at ebay.com>
Thanks a lot for the review, Han. I've posted v2 with the suggested fix
and kept your acks:
https://patchwork.ozlabs.org/project/openvswitch/list/?series=58323
-Jakub
More information about the dev
mailing list