Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Properly log recompute reason.

2022-03-29 Thread Dumitru Ceara
On 3/29/22 16:24, Mark Michelson wrote: > I pushed this change to main, branch-22.03, and branch-21.12. Thanks > Dumitru and Lorenzo. > Thanks! ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Properly log recompute reason.

2022-03-29 Thread Mark Michelson
I pushed this change to main, branch-22.03, and branch-21.12. Thanks Dumitru and Lorenzo. On 3/29/22 05:28, Lorenzo Bianconi wrote: The 'reason' was computed only when logging at level DBG. However, if a node takes more than 'engine_compute_log_timeout_msec' to recompute, the log is generated

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Properly log recompute reason.

2022-03-29 Thread Lorenzo Bianconi
> The 'reason' was computed only when logging at level DBG. However, if a > node takes more than 'engine_compute_log_timeout_msec' to recompute, the > log is generated at level INFO, resulting in: > > inc_proc_eng|INFO|node: northd, recompute ((null)) > > We now unconditionally compute 'reason

[ovs-dev] [PATCH ovn] inc-proc-eng: Properly log recompute reason.

2022-03-29 Thread Dumitru Ceara
The 'reason' was computed only when logging at level DBG. However, if a node takes more than 'engine_compute_log_timeout_msec' to recompute, the log is generated at level INFO, resulting in: inc_proc_eng|INFO|node: northd, recompute ((null)) We now unconditionally compute 'reason'. In theory