[ovs-dev] [PATCH ovn v2 2/2] controller: ipv6_pd: properly update ipv6_ra_pd_list pb option in sb db

[Mark Gray]

On 06/08/2021 10:01, Lorenzo Bianconi wrote:
> Clear ipv6_ra_pd_list option in port_binding option column in order to
> properly update IPv6 prefix info since otherwise if options smap already
> contains a stale value for "ipv6_ra_pd_list" key, it will not be updated
> with the new one.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1983862
> 
> Fixes: e3a398e914 ("controller: Add ipv6 prefix delegation state machine")
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi at redhat.com>
> ---
>  controller/pinctrl.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/controller/pinctrl.c b/controller/pinctrl.c
> index 503d5686c..a5ca5b16a 100644
> --- a/controller/pinctrl.c
> +++ b/controller/pinctrl.c
> @@ -1258,6 +1258,7 @@ fill_ipv6_prefix_state(struct ovsdb_idl_txn *ovnsb_idl_txn,
>              pfd->last_complete = time_msec();
>              pfd->next_announce = pfd->last_complete + pfd->t1;
>              smap_clone(&options, &pb->options);
> +            smap_remove(&options, "ipv6_ra_pd_list");
>              smap_add_format(&options, "ipv6_ra_pd_list", "%d:%s/%d",
>                              pfd->aid, prefix_str, pfd->plen);
>              sbrec_port_binding_set_options(pb, &options);
> 
Acked-by: Mark D. Gray <mark.d.gray at redhat.com>