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.

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 show command for BFD sessions #1942

Merged
merged 2 commits into from
Nov 25, 2021
Merged

Add show command for BFD sessions #1942

merged 2 commits into from
Nov 25, 2021

Conversation

shi-su
Copy link
Contributor

@shi-su shi-su commented Nov 23, 2021

What I did

Add show command for BFD sessions.

How I did it

Add show command for BFD session and unit tests.

How to verify it

Verify the show command locally and unit test passes.

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

Example output:

>> show bfd summary
Total number of BFD sessions: 3
Peer Addr    Interface    Vrf      State    Type          Local Addr      TX Interval    RX Interval    Multiplier  Multihop
-----------  -----------  -------  -------  ------------  ------------  -------------  -------------  ------------  ----------
10.0.1.1     default      default  DOWN     async_active  10.0.0.1                300            500             3  true
10.0.2.1     Ethernet12   default  UP       async_active  10.0.0.1                200            600             3  false
2000::10:1   default      default  UP       async_active  2000::1                 100            700             3  false
>> show bfd peer 10.0.1.1
Total number of BFD sessions for peer IP 10.0.1.1: 1
Peer Addr    Interface    Vrf      State    Type          Local Addr      TX Interval    RX Interval    Multiplier  Multihop
-----------  -----------  -------  -------  ------------  ------------  -------------  -------------  ------------  ----------
10.0.1.1     default      default  DOWN     async_active  10.0.0.1                300            500             3  true

prsunny
prsunny previously approved these changes Nov 23, 2021
Copy link
Contributor

@prsunny prsunny left a comment

Choose a reason for hiding this comment

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

lgtm, please update the CLI command reference

@shi-su shi-su merged commit 30e4654 into sonic-net:master Nov 25, 2021
abdosi pushed a commit that referenced this pull request Dec 8, 2021
What I did
Add show command for BFD sessions.

How I did it
Add show command for BFD session and unit tests.

How to verify it
Verify the show command locally and unit test passes.
shi-su added a commit to shi-su/sonic-utilities that referenced this pull request Jan 6, 2022
What I did
Add show command for BFD sessions.

How I did it
Add show command for BFD session and unit tests.

How to verify it
Verify the show command locally and unit test passes.
shi-su added a commit that referenced this pull request Jan 26, 2022
What I did
Cherry-pick changes in #1942 to 202012 branch
Add show command for BFD sessions.

How I did it
Add show command for BFD session and unit tests.

How to verify it
Verify the show command locally and unit test passes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants