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

Reply via email to