Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Bfd diagnostic code for when neighbor signaled session down. #1233

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Add Bfd diagnostic code for when neighbor signaled session down. #1233

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
fc7bd12
Add Bfd diagnostic code for when neighbor signaled session down.
AnilKR123 Dec 16, 2024
52ddf09
Move NEIGHBOR_DOWN to the end of the enums list to preserve the origi…
AnilKR123 Dec 17, 2024
63bd254
Add comment about RFC5880 deviation.
AnilKR123 Dec 17, 2024
0041e2b
Merge branch 'master' into Bfd
AnilKR123 Jan 15, 2025
77fd5c3
Merge branch 'master' into Bfd
dplore Feb 11, 2025
b3229cb
Bump version number.
AnilKR123 Feb 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 18 additions & 5 deletions release/models/bfd/openconfig-bfd.yang
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,13 @@ module openconfig-bfd {

oc-ext:openconfig-version "0.3.0";

revision "2024-12-16" {
description
"Add Bfd diagnostic code for when
neighbor signaled session down. Specified in RFC5880.";
reference "0.3.1";
}

revision "2024-03-05" {
description
"Add configuration of min interval, multiplier when
Expand Down Expand Up @@ -157,35 +164,41 @@ module openconfig-bfd {
"The BFD echo function failed - echo packets have not been
received for the required period of time.";
}
enum FORWARDING_RESET {
enum NEIGHBOR_DOWN {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To keep this as a non-breaking change, the original ordering of the enums must be preserved. You can only add enums to the list.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realize the enum value won't match the value in a packet on the wire, but that is not required.

value 3;
description
"The Bfd neighbor signaled session down - Bfd packet was received with
neighbor state down.";
}
enum FORWARDING_RESET {
value 4;
description
"The forwarding plane in the local system was reset - such
that the remote system cannot rely on the forwarding state of
the device specifying this error code.";
}
enum PATH_DOWN {
value 4;
value 5;
description
"Signalling outside of BFD specified that the path underlying
this session has failed.";
}
enum CONCATENATED_PATH_DOWN {
value 5;
value 6;
description
"When a BFD session runs over a series of path segments, this
error code indicates that a subsequent path segment (i.e.,
one in the transmit path between the source and destination
of the session) has failed.";
}
enum ADMIN_DOWN {
value 6;
value 7;
description
"The BFD session has been administratively disabled by the
peer.";
}
enum REVERSE_CONCATENATED_PATH_DOWN {
value 7;
value 8;
description
"In the case that a BFD session is running over a series of
path segments, this error code indicates that a path segment
Expand Down
Loading