Hi Paulo, On 12/11/25 11:20 PM, Mark Michelson via dev wrote: > Wow Paulo, this is fantastic! I think this provides a great > jumping-off point for developing further incremental processing in > ovn-ic.
Indeed, thanks a lot for working on this! > > Acked-by: Mark Michelson <[email protected]> > > On Wed, Dec 10, 2025 at 7:28 AM Paulo Guilherme Silva via dev > <[email protected]> wrote: >> >> Initial implementation adds a single node (ic). This single >> node executes the ovn-ic processing pipeline but does not do so >> incrementally. >> >> In order to develop incremental processing for ovn-ic, the code >> will be organised with a .c/.h file for each I-P node following >> the naming convention en-<node name>.c/.h. These files will >> contain definition of the node data, the main node processing >> functions and change handlers (if any). The purpose of these nodes >> will be coordination of the nodes work and implemention of the >> relevant interfaces to plug into the I-P framework. The actual >> work that will be executed by the node will be organised into >> a companion file or files. Ideally this file will follow the >> naming convention of the node: e.g. en-<node name>.c is >> associated with <node name>.c. >> >> Initial node topology sees the ic node dependent on all DB >> nodes. This structure and approach was heavily influenced >> by the way I+P was developed and evolved in the northd daemon. >> >> Signed-off-by: Paulo Guilherme Silva <[email protected]> >> >> --- Commits 2a9e7e2e87db ("northd: controller: Support dumping a visual representation of the I-P graph.") [0] and afcb8c99f4d8 ("docs: Add OVN Incremental Processing Engine documentation.") [1] added and documented support for getting a visual representation of the I-P engine in ovn-northd and ovn-controller [2]. With your patch ovn-ic starts using the I-P engine so it might be nice if we add an implementation of inc_proc_graph_dump() in ovn-ic too. Hopefully that helps us later when/if we try to add finer grain incremental processing to ovn-ic. What do you think? Regards, Dumitru [0] https://github.com/ovn-org/ovn/commit/2a9e7e2 [1] https://github.com/ovn-org/ovn/commit/afcb8c9 [2] https://github.com/ovn-org/ovn/blob/main/Documentation/topics/incremental-processing/incremental-processing.rst#graph-visualization _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
