On Mon, May 11, 2026 at 8:58 PM Mark Michelson via dev < [email protected]> wrote:
> Fix the --n-threads option format in the manpage (was missing the > "--" prefix). Add the undocumented --dump-inc-proc-graph option to > both the manpage and usage() output. > > Fix runtime management command names: set-n-threads and get-n-threads > were documented with incorrect names; the actual registered commands > are parallel-build/set-n-threads and parallel-build/get-n-threads. > > Add missing runtime management commands to the manpage: > nb-connection-status, sb-connection-status, > debug/chassis-features-list, inc-engine/recompute, > inc-engine/compute-log-timeout, and inc-engine/list-stopwatches. > > Mark the counter_name argument of inc-engine/show-stats as optional, > matching the actual command registration. > > Reported-at: https://redhat.atlassian.net/browse/FDP-3054 > Assisted-by: Claude Code (Claude Opus 4.6) > Signed-off-by: Mark Michelson <[email protected]> > --- > northd/ovn-northd.8.xml | 57 +++++++++++++++++++++++++++++++++++++---- > northd/ovn-northd.c | 2 ++ > 2 files changed, 54 insertions(+), 5 deletions(-) > > diff --git a/northd/ovn-northd.8.xml b/northd/ovn-northd.8.xml > index e5d460382..c49ec1b37 100644 > --- a/northd/ovn-northd.8.xml > +++ b/northd/ovn-northd.8.xml > @@ -44,7 +44,7 @@ > Commands</code> below. > </p> > </dd> > - <dt><code>n-threads N</code></dt> > + <dt><code>--n-threads=<var>N</var></code></dt> > <dd> > <p> > In certain situations, it may be desirable to enable > parallelization > @@ -62,6 +62,15 @@ > enabled (with 256 threads) and a warning is logged. > </p> > </dd> > + > <dt><code>--dump-inc-proc-graph</code>[<code>=<var>node</var></code>]</dt> > + <dd> > + <p> > + Dump the incremental processing engine graph representation in > + DOT format to stdout at startup and then exit. If > + <var>node</var> is specified, only the subgraph rooted at that > + engine node is printed. > + </p> > + </dd> > </dl> > <p> > <var>database</var> in the above options must be an OVSDB active or > @@ -157,7 +166,7 @@ > </dd> > > > - <dt><code>set-n-threads N</code></dt> > + <dt><code>parallel-build/set-n-threads</code> <var>N</var></dt> > <dd> > <p> > Set the number of threads used for building logical flows. > @@ -165,17 +174,36 @@ > When N is 1 parallelization is disabled. > When N is less than 1 or more than 256, an error is returned. > If ovn-northd fails to start parallelization (e.g. fails to setup > - semaphores, parallelization is disabled and an error is returned. > + semaphores), parallelization is disabled and an error is returned. > </p> > </dd> > > - <dt><code>get-n-threads</code></dt> > + <dt><code>parallel-build/get-n-threads</code></dt> > <dd> > <p> > Return the number of threads used for building logical flows. > </p> > </dd> > > + <dt><code>nb-connection-status</code></dt> > + <dd> > + Prints whether the connection to the OVN Northbound database is > + currently connected or not. > + </dd> > + > + <dt><code>sb-connection-status</code></dt> > + <dd> > + Prints whether the connection to the OVN Southbound database is > + currently connected or not. > + </dd> > + > + <dt><code>debug/chassis-features-list</code></dt> > + <dd> > + Lists the chassis feature flags and their current values as > + determined by <code>ovn-northd</code>. These features reflect > + the capabilities reported by the connected chassis. > + </dd> > + > <dt><code>inc-engine/show-stats</code></dt> > <dd> > <p> > @@ -195,7 +223,7 @@ > </p> > </dd> > > - <dt><code>inc-engine/show-stats <var>engine_node_name</var> > <var>counter_name</var></code></dt> > + <dt><code>inc-engine/show-stats</code> <var>engine_node_name</var> > [<var>counter_name</var>]</dt> > <dd> > <p> > Display the <code>ovn-northd</code> engine counter(s) for the > specified > @@ -210,6 +238,25 @@ > <p> Reset <code>ovn-northd</code> engine counters. </p> > </dd> > > + <dt><code>inc-engine/recompute</code></dt> > + <dd> > + Triggers a full recompute of the incremental processing engine > + on the next iteration. > + </dd> > + > + <dt><code>inc-engine/compute-log-timeout</code> > <var>msecs</var></dt> > + <dd> > + Sets the timeout in milliseconds for logging engine compute > + events. > + </dd> > + > + <dt><code>inc-engine/list-stopwatches</code> [<var>node</var>]</dt> > + <dd> > + Lists the engine nodes and their change handler names. If > + <var>node</var> is specified, only the handlers for that engine > + node are listed. > + </dd> > + > </dl> > </p> > > diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c > index 0ed2eb17a..7e0f888c1 100644 > --- a/northd/ovn-northd.c > +++ b/northd/ovn-northd.c > @@ -619,6 +619,8 @@ Options:\n\ > (default: %s)\n\ > --dry-run start in paused state (do not commit db > changes)\n\ > --n-threads=N specify number of threads\n\ > + --dump-inc-proc-graph[=NODE]\n\ > + dump incremental processing graph and exit\n\ > --unixctl=SOCKET override default control socket name\n\ > -h, --help display this help message\n\ > -o, --options list available options\n\ > -- > 2.52.0 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Thank you Mark, applied to main. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
