On 12/15/25 2:21 PM, Paulo Guilherme (MGC URPE SDN) wrote: > Hi Dumitru, >
Hi Paulo, > Yes, I'm preparing a v3 with those I-P graph visualization changes > included. > In fact, it's already ready, I just need to complete the unit tests and > submit them. > Thanks for letting me know! I'll move v2 to "Changes Requested" in patchwork. Looking forward to v3. Regards, Dumitru > Regards, > > Paul > > Em seg., 15 de dez. de 2025 às 07:48, Dumitru Ceara <[email protected] > <mailto:[email protected]>> escreveu: > > On 12/12/25 7:53 PM, Paulo Guilherme (MGC URPE SDN) wrote: > > Hi Dumitru and Mark, > > > > Yes, I agree with you. The next few days I will study these > patches and > > try backporting of them adapting to ovn-ic, thanks! > > > > Hi Paulo, > > Just making sure, do you mean you'll be preparing a v3 with those I-P > graph visualization changes included or are you planning on doing that > as a follow up patch? > > Thanks, > Dumitru > > > Regards > > > > Paulo > > > > Em sex., 12 de dez. de 2025 às 04:54, Dumitru Ceara > <[email protected] <mailto:[email protected]> > > <mailto:[email protected] <mailto:[email protected]>>> escreveu: > > > > 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] > <mailto:[email protected]> > > <mailto:[email protected] <mailto:[email protected]>>> > > > > > > On Wed, Dec 10, 2025 at 7:28 AM Paulo Guilherme Silva via dev > > > <[email protected] <mailto:[email protected]> > <mailto:[email protected] <mailto:[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] > <mailto:[email protected]> > <mailto:[email protected] > <mailto:[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 <https:// > github.com/ovn-org/ovn/commit/2a9e7e2> <https:// > > github.com/ovn-org/ovn/commit/2a9e7e2 <http://github.com/ovn- > org/ovn/commit/2a9e7e2>> > > [1] https://github.com/ovn-org/ovn/commit/afcb8c9 <https:// > github.com/ovn-org/ovn/commit/afcb8c9> <https:// > > github.com/ovn-org/ovn/commit/afcb8c9 <http://github.com/ovn- > org/ovn/commit/afcb8c9>> > > [2] > > https://github.com/ovn-org/ovn/blob/main/Documentation/topics/ > <https://github.com/ovn-org/ovn/blob/main/Documentation/topics/> > > incremental-processing/incremental-processing.rst#graph- > > visualization <https://github.com/ovn-org/ovn/blob/main/ > <https://github.com/ovn-org/ovn/blob/main/> > > Documentation/topics/incremental-processing/incremental- > > processing.rst#graph-visualization> > > > > > > > > > > /‘Esta mensagem é direcionada apenas para os endereços constantes no > > cabeçalho inicial. Se você não está listado nos endereços > constantes no > > cabeçalho, pedimos-lhe que desconsidere completamente o conteúdo dessa > > mensagem e cuja cópia, encaminhamento e/ou execução das ações citadas > > estão imediatamente anuladas e proibidas’./ > > > > / //‘Apesar do Magazine Luiza tomar todas as precauções razoáveis para > > assegurar que nenhum vírus esteja presente nesse e-mail, a empresa não > > poderá aceitar a responsabilidade por quaisquer perdas ou danos > causados > > por esse e-mail ou por seus anexos’./ > > > > > > /‘Esta mensagem é direcionada apenas para os endereços constantes no > cabeçalho inicial. Se você não está listado nos endereços constantes no > cabeçalho, pedimos-lhe que desconsidere completamente o conteúdo dessa > mensagem e cuja cópia, encaminhamento e/ou execução das ações citadas > estão imediatamente anuladas e proibidas’./ > > / //‘Apesar do Magazine Luiza tomar todas as precauções razoáveis para > assegurar que nenhum vírus esteja presente nesse e-mail, a empresa não > poderá aceitar a responsabilidade por quaisquer perdas ou danos causados > por esse e-mail ou por seus anexos’./ > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
