| Message ID | 20260901135638.290353-9-jtanenba@redhat.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | ovn-sbctl: Review of ovn-sbctl documentation. | expand |
| Context | Check | Description |
|---|---|---|
| ovsrobot/apply-robot | success | apply and check: success |
| ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
| ovsrobot/github-robot-_ovn-kubernetes | success | github build: passed |
Hi Jacob. The commit message says that this fixes the --may-exist option, but the actual doc change is fixing the documentation of the "--if-exists" option. On Tue, Sep 1, 2026 at 11:57 AM Jacob Tanenbaum via dev <ovs-dev@openvswitch.org> wrote: > > Add a missing comma in the description of --may-exist > > Assisted-by: Claude Opus 4.6, Claude Code > Fixes: 0b70e8a83965 ("ovn-sbctl: Add daemon support.") > Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com> > --- > utilities/ovn-sbctl.8.xml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/utilities/ovn-sbctl.8.xml b/utilities/ovn-sbctl.8.xml > index 9ab5419e5..27b6acc63 100644 > --- a/utilities/ovn-sbctl.8.xml > +++ b/utilities/ovn-sbctl.8.xml > @@ -330,8 +330,8 @@ > > <p> > Without <code>--if-exists</code>, attempting to delete a chassis that > - does not exist is an error. With <code>--if-exists</code> attempting > - to delete a chassis that does not exist has no effect. > + does not exist is an error. With <code>--if-exists</code>, > + attempting to delete a chassis that does not exist has no effect. > </p> > </dd> > </dl> > -- > 2.55.0 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
diff --git a/utilities/ovn-sbctl.8.xml b/utilities/ovn-sbctl.8.xml index 9ab5419e5..27b6acc63 100644 --- a/utilities/ovn-sbctl.8.xml +++ b/utilities/ovn-sbctl.8.xml @@ -330,8 +330,8 @@ <p> Without <code>--if-exists</code>, attempting to delete a chassis that - does not exist is an error. With <code>--if-exists</code> attempting - to delete a chassis that does not exist has no effect. + does not exist is an error. With <code>--if-exists</code>, + attempting to delete a chassis that does not exist has no effect. </p> </dd> </dl>
Add a missing comma in the description of --may-exist Assisted-by: Claude Opus 4.6, Claude Code Fixes: 0b70e8a83965 ("ovn-sbctl: Add daemon support.") Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com> --- utilities/ovn-sbctl.8.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)