Hi Jacob, I have a couple of findings below.
On Tue, Sep 1, 2026 at 12:00 PM Jacob Tanenbaum via dev
<[email protected]> wrote:
>
> Load balancers where added to vflows and the documentation was not
s/where/were/
> updated to reflect that.
>
> Assisted-by: Claude Opus 4.6, Claude Code
> Fixes: 4da6881b6431 ("sbctl: Add Load Balancer support for vflows option.")
> Signed-off-by: Jacob Tanenbaum <[email protected]>
> ---
> utilities/ovn-sbctl.8.xml | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/utilities/ovn-sbctl.8.xml b/utilities/ovn-sbctl.8.xml
> index 3afcf5dc3..0b6aa4758 100644
> --- a/utilities/ovn-sbctl.8.xml
> +++ b/utilities/ovn-sbctl.8.xml
> @@ -425,8 +425,9 @@
> records directly used for generating OpenFlow flows are also
> listed. This includes: <var>port-bindings</var>,
> <var>mac-bindings</var>, <var>multicast-groups</var>,
> - <var>chassis</var>. The <code>--ovs</code> and
> <code>--stats</code>
> - can also be used in conjunction with <code>--vflows</code>.
> + <var>chassis</var> and <var>load-balancers</var>. The
> + <code>--ovs</code> and <code>--stats</code> can also be used in
> + conjunction with <code>--vflows</code>.
When rendered, this ends up saying "The --ovs and --stats can also be
used in conjunction with --vflows". To me, this reads a bit odd. I
think it can be changed to one of two options to make it sound more
natural:
* The --ovs and --stats options can also be used in conjunction with --vflows.
* --ovs and --stats can also be used in conjunction with --vflows.
> </p>
> </dd>
>
> --
> 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
