On 3/8/24 11:22, Eelco Chaudron wrote: > > > On 5 Mar 2024, at 21:37, Timothy Redaelli wrote: > >> A log message like this one: >> >> 2024-01-09T06:45:17.201Z|00071|bfd(handler2)|INFO|ovn-0af536-0: BFD state >> change: down->up "Neighbor Signaled Session Down"->"Neighbor Signaled Session >> Down". >> >> can be hard to read since '->' usually represents a status change, but >> in this case the diagnostic code stays constant. Update the log message to >> avoid such ambiguity. The log message for the above event become: >> >> 2024-01-09T06:45:16.211Z|00026|bfd(handler3)|INFO|ovn-0af536-0: BFD state >> change: (bfd.SessionState: down, bfd.LocalDiag: "Neighbor Signaled Session >> Down") -> (bfd.SessionState: up, bfd.LocalDiag: "Neighbor Signaled Session >> Down") >> >> Reported-by: Alex Stupnikov <[email protected]> >> Reported-at: https://bugzilla.redhat.com/2258496 >> Signed-off-by: Timothy Redaelli <[email protected]> > > Thanks for following up, the change looks good to me! > > Acked-by: Eelco Chaudron <[email protected]> >
Thanks, Timothy and Eelco! Applied and backported down to 2.17 as previously discussed. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
