On 12/2/25 5:38 PM, Mairtin O'Loingsigh wrote: > On Thu, Nov 27, 2025 at 06:20:34PM +0100, Dumitru Ceara via dev wrote: >> This commit introduces the ability to dump a (partial) representation >> of the incremental processing graph. It's currently supported for >> ovn-northd and ovn-controller through a command-line argument, >> --dump-inc-proc-graph[=<i-p-node>]. >> >> If the command line argument is present the binary (ovn-northd or >> ovn-controller) just dumps a representation of the incremental >> processing graph in DOT format to stdout. The binary then exits. >> >> If the '<i-p-node>' optional argument value is present the >> representation only includes nodes up to (and including) '<i-p-node>'. >> >> The goal of the feature is to make lives of the developers/maintainers >> easier when writing or debugging features that require incremental >> processing. >> >> Reported-at: https://issues.redhat.com/browse/FDP-2646 >> Assisted-by: gemini-cli, with model: gemini-3-pro-preview >> Signed-off-by: Dumitru Ceara <[email protected]> >> --- >> build-aux/initial-tab-whitelist | 1 + >> controller/ovn-controller.c | 28 ++ >> lib/inc-proc-eng.c | 64 +++- >> lib/inc-proc-eng.h | 23 +- >> northd/ovn-northd.c | 25 ++ >> tests/automake.mk | 3 +- >> tests/ovn-inc-proc-graph-dump.at | 513 +++++++++++++++++++++++++++++++ >> tests/testsuite.at | 1 + >> 8 files changed, 646 insertions(+), 12 deletions(-) >> create mode 100644 tests/ovn-inc-proc-graph-dump.at >>
[...] >> > > Looks good to me. > > Acked-by: Mairtin O'Loingsigh <[email protected]> > Hi Mairtin, Ales, Thanks for the reviews! I rebased this patch and applied it to main. Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
