You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
liveness is only registering register_status_func, and not register_status_reason_func, so only status keeps coherent, status_reason when liveness is down would be empty:
In terms of handling link_down|link_up notifications it's still correct, but if any application were relying only on "status_reason" here it would be a problem. That's why it's also recommended to always check if status is coherent too.
Overtime we added this option to register the reason, but it ended up left behind here for of_lldp
The text was updated successfully, but these errors were encountered:
liveness
is only registeringregister_status_func
, and notregister_status_reason_func
, so onlystatus
keeps coherent,status_reason
when liveness is down would be empty:In terms of handling link_down|link_up notifications it's still correct, but if any application were relying only on
"status_reason"
here it would be a problem. That's why it's also recommended to always check ifstatus
is coherent too.Overtime we added this option to register the reason, but it ended up left behind here for
of_lldp
The text was updated successfully, but these errors were encountered: