Hi Jacob, I have a couple of comments below
On Fri, Aug 21, 2026 at 3:58 PM Jacob Tanenbaum via dev
<[email protected]> wrote:
>
> When e3a398 added documentation for handle_dhcpv6_reply two entries
> where added for it in the Logical_Flow.actions. This patch removes one
s/where/were/
> of those entries.
>
> Fixes: e3a398e9146e ("controller: Add ipv6 prefix delegation state machine")
> Assisted-by: Claude Opus 4.6, Claude Code
> Signed-off-by: Jacob Tanenbaum <[email protected]>
> ---
> ovn-sb.xml | 11 ++---------
> 1 file changed, 2 insertions(+), 9 deletions(-)
>
> diff --git a/ovn-sb.xml b/ovn-sb.xml
> index 50a0a3419..e95663e2d 100644
> --- a/ovn-sb.xml
> +++ b/ovn-sb.xml
> @@ -2623,7 +2623,8 @@ tcp.flags = RST;
> <dd>
> <p>
> Handle DHCPv6 prefix delegation advertisements/replies from
> - a IPv6 delegation server. <code>ovn-controller</code> will
> + an IPv6 delegation server and manages the prefix delegation state
> + machine. <code>ovn-controller</code> will
The verbs in this sentence are of different forms. It should either be
"Handles ... and manages" or "Handle ... and manage"
> add an entry <code>ipv6_ra_pd_list</code> in the
> <ref table="Port_Binding" column="options"/> table for each
> prefix received from the delegation server
> @@ -2673,14 +2674,6 @@ tcp.flags = RST;
> </p>
> </dd>
>
> - <dt><code>handle_dhcpv6_reply;</code></dt>
> - <dd>
> - <p>
> - This action is used to parse DHCPv6 replies from IPv6
> - Delegation Router and managed IPv6 Prefix delegation state
> machine
> - </p>
> - </dd>
> -
> <dt><code><var>R</var> = chk_lb_hairpin();</code></dt>
> <dd>
> <p>
> --
> 2.55.0
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
