On Thu, Nov 27, 2025 at 6:21 PM Dumitru Ceara via dev < [email protected]> wrote:
> This patch series aims to enhance the developer experience when > working with the I-P engine. > > The first two patches in this series were manually coded, while the > last two were generated using vibe coding. Patch 3 specifically > introduces the capability to dump the I-P engine's Directed Acyclic > Graph (DAG) into a human-consumable DOT file. This feature is > implemented for ovn-northd and ovn-controller, which are the only > components currently utilizing the I-P engine. The DOT file can then > be easily transformed into a PNG or similar image format, aiding in > visualization and debugging. > > The AI experience for code generation was satisfactory, requiring only > minor tweaks to the generated code. I have reviewed the AI-assisted > generated code and it does not look very different from what would > have been manually coded. However, generating test cases with AI did > not work well, so all test cases have been fully manually coded. > > NOTE: the cover letter contents above have also been "vibe-coded" :) > Assisted-by: gemini-cli, with model: gemini-3-pro-preview > > Dumitru Ceara (4): > controller: Make I-P engine node definitions global. > controller: Move I-P engine initialization to separate function. > northd: controller: Support dumping a visual representation of the I-P > graph. > docs: Add OVN Incremental Processing Engine documentation. > > Documentation/automake.mk | 4 + > .../datapath-sync-graph.png | Bin 0 -> 94992 bytes > .../incremental-processing/evpn-arp-graph.png | Bin 0 -> 285440 bytes > .../incremental-processing.rst | 211 +++ > .../topics/incremental-processing/index.rst | 31 + > Documentation/topics/index.rst | 1 + > build-aux/initial-tab-whitelist | 1 + > controller/ovn-controller.c | 1203 +++++++++-------- > 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 + > 14 files changed, 1511 insertions(+), 569 deletions(-) > create mode 100644 > Documentation/topics/incremental-processing/datapath-sync-graph.png > create mode 100644 > Documentation/topics/incremental-processing/evpn-arp-graph.png > create mode 100644 > Documentation/topics/incremental-processing/incremental-processing.rst > create mode 100644 Documentation/topics/incremental-processing/index.rst > create mode 100644 tests/ovn-inc-proc-graph-dump.at > > -- > 2.51.1 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Thank you Dumitru, the whole series looks good to me, thanks. Acked-by: Ales Musil <[email protected]> Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
