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 <astup...@redhat.com>
> Reported-at: https://bugzilla.redhat.com/2258496
> Signed-off-by: Timothy Redaelli <tredae...@redhat.com>

Thanks for following up, the change looks good to me!

Acked-by: Eelco Chaudron <echau...@redhat.com>

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to