On 1/12/26 2:18 PM, Paulo Guilherme (MGC URPE SDN) wrote:
> Hi Dumitru,
> 

Hi Paulo,

> I have another question about our patch series inc-processing_ovn-ic. In
> this moment, i have the following local patches:
> 
> dbc8f637e ovn-ic: Introduces I+P engine for gateway.
> 393db49dd ovn-ic: Introduces I+P engine for route.
> 8e287043d ovn-ic: Introduces I+P engine for port-binding.
> 1a4d2825b ovn-ic: Introduces I+P engine for Enum-Datapaths.
> 9b9ce18c3 ovn-ic: Introduce structs ic_data and ic_input.
> 2735f3c6b docs: Update OVN I+P Engine documentation.
> 82bb5e944 ovn-ic: Support dumping a visual representation of the I-P graph.
> 0470c8bba ovn-ic: Introduce incremental processing.
> 053690db7 ovn: Move run_idl_loop function to lib/ovn-util.
> 
> We haven't finished the feature yet.
> 
> So the question is, for you and the OVN community, Is it better to

In general, for OVN community related discussions, including patch
reviews/discussion/debugging, please try to keep [email protected] in
CC.  It's more beneficial to the community if more people can choose to
participate in the discussion.

> complete the entire feature and submit one unique complete patch series
> or submit each new patch in patch series format?
> 

I'm not sure I understand the question completely.  Do you mean to say
that your current implementation does some partial incremental
processing in ovn-ic, for some of types database resource changes, and
falls back to recompute if other changes changes are detected?

If so, then it's probably acceptable to incorporate this version (after
review and approval in the community).  It makes sense in my opinion to
add incremental processing support (which normally requires more complex
code and sometimes increases memory usage) only for the most common
types of changes, potentially in multiple releases, to maximize the
impact vs cost aspect.

Regards,
Dumitru

> Regards, 
> Paulo
> 
> Em seg., 5 de jan. de 2026 às 08:49, Dumitru Ceara <[email protected]
> <mailto:[email protected]>> escreveu:
> 
>     On 12/22/25 9:51 PM, Paulo Guilherme (MGC URPE SDN) wrote:
>     > Hi Dumitru,
>     >
> 
>     Hi Paulo,
> 
>     > I apologize for bringing this up again.
>     >
> 
>     No problem!
> 
>     > Could you help me with this commit?
>     >
>     > I submitted the v4 of patch ovn-ic:incremental processing, but the
>     Build
>     > and Test step is depriving. In my fork of ovn, Build and Test step
>     > approved. I tried to execute de process again with 'Recheck-request:
>     > github-robot-_Build_and_Test', but I noticed the pipeline doesn't
>     start
>     > again. What could I do to resolve this?
>     >
> 
>     I don't see any email reply to that patch with 'Recheck-request':
> 
>     https://patchwork.ozlabs.org/project/ovn/
>     patch/[email protected]/
>     <https://patchwork.ozlabs.org/project/ovn/
>     patch/[email protected]/>
> 
>     Maybe you didn't reply to the patch with ovs-dev in cc?
> 
>     The failure you hit:
> 
>     "628: ovn.at:43274 <http://ovn.at:43274>     
>      lb_force_snat_ip=router_ip select correct
>     network for snat -- parallelization=yes -- ovn_monitor_all=yes"
> 
>     is due to a known bug:
>     https://issues.redhat.com/browse/FDP-2870 <https://
>     issues.redhat.com/browse/FDP-2870>
> 
>     I'll trigger recheck on the series as the failure on patch 4/4 seems to
>     be an infrastructure issue:
> 
>     Error: stat /tmp/image.tar: no such file or directory
> 
>     Regards,
>     Dumitru
> 
>     > Regards,
>     > Paulo
>     >
>     > Em sex., 19 de dez. de 2025 às 11:27, Paulo Guilherme (MGC URPE SDN) <
>     > [email protected]
>     <mailto:[email protected]>> escreveu:
>     >
>     >> Recheck-request: github-robot-_Build_and_Test
>     >>
>     >> Em qua., 17 de dez. de 2025 às 06:45, 0-day Robot
>     <[email protected] <mailto:[email protected]>>
>     >> escreveu:
>     >>
>     >>> From: [email protected] <mailto:[email protected]>
>     >>>
>     >>> Test-Label: github-robot: Build and Test
>     >>> Test-Status: fail
>     >>> http://patchwork.ozlabs.org/api/patches/2174959/ <http://
>     patchwork.ozlabs.org/api/patches/2174959/>
>     >>>
>     >>> _github build: failed_
>     >>> Build URL: https://github.com/ovsrobot/ovn/actions/
>     runs/20282240556 <https://github.com/ovsrobot/ovn/actions/
>     runs/20282240556>
>     >>>
>     >>
>     >
> 
> 
> 
> /‘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

Reply via email to