bfd: always export remote_state and remote_diagnostic to OVSDB
authorAndy Zhou <azhou@nicira.com>
Thu, 22 Oct 2015 17:29:56 +0000 (10:29 -0700)
committerAndy Zhou <azhou@nicira.com>
Fri, 23 Oct 2015 02:53:45 +0000 (19:53 -0700)
commit2979b9158bff4d6fcbddf7e09131a8cfeb9ec746
treed90d692d341111dad443e33740b642df4d85b95f
parent67794d5617bff0b116b61f2bc8edde24624ebca7
bfd: always export remote_state and remote_diagnostic to OVSDB

RFC 5880 specified bfd.RemoteSessionState as one of the state
variables.  In OVS implementation, this value is exported to OVSDB's
BFD status column of the interface table, as one of the map elements,
with the key of 'remote_state'.

It can be surprising when the 'remote_state' map element disappears
when BFD is in the 'DOWN' state, but otherwise always exported.
Change to always exporting it, to make it more predictable for
applications that monitors the BFD status column.

While at it, make the same change to 'remote_diagnostic', so that it
is also always exported to OVSDB for consistency.

VMWare-BZ: 1535979
Reported-by: Mihir Gangar <gangarm@vmware.com>
Signed-off-by: Andy Zhou <azhou@nicira.com>
Acked-by: Justin Pettit <jpettit@nicira.com>
lib/bfd.c