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

ospfd:fix show_ip_ospf_gr_helper #14710

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

teletajp
Copy link
Contributor

@teletajp teletajp commented Nov 1, 2023

Small fix for the command "show ip ospf vrf NAME graceful-restart helper detail". FRR did not show information by vrf's name.
Error description: If i have router ospf vrf red, vtysh's command show ip ospf vrf red graceful-restart helper will not show anything. But command show ip ospf vrf all graceful-restart helper will work normally. This fix fixes the display of information by vrf's name.

@teletajp teletajp force-pushed the fix_show_ospf_gr_helper_cmd branch from 0f5068c to 0c8c95f Compare November 1, 2023 10:09
@teletajp teletajp force-pushed the fix_show_ospf_gr_helper_cmd branch from 0c8c95f to 1666237 Compare November 1, 2023 10:20
@donaldsharp
Copy link
Member

ci:rerun

@ton31337
Copy link
Member

ton31337 commented Nov 2, 2023

@Mergifyio backport dev/9.1

Copy link

mergify bot commented Nov 2, 2023

backport dev/9.1

❌ No backport have been created

  • Backport to branch dev/9.1 failed

GitHub error: Branch not found

@ton31337
Copy link
Member

ton31337 commented Nov 7, 2023

Could you put a decent description in the commit message what is the actual fix here?

@ton31337 ton31337 added this to the 9.1 milestone Nov 7, 2023
@github-actions github-actions bot added the rebase PR needs rebase label Nov 7, 2023
@teletajp teletajp force-pushed the fix_show_ospf_gr_helper_cmd branch from 791ddd5 to e9924fc Compare November 7, 2023 20:27
@ton31337
Copy link
Member

ton31337 commented Nov 8, 2023

@teletajp please write a more detailed explanation (as in the PR's description) to the commit. With real outputs, it would be even better.

@ton31337 ton31337 removed this from the 9.1 milestone Nov 9, 2023
Fix for the command "show ip ospf vrf NAME graceful-restart helper".
FRR did not show information by vrf's name.
If i have router ospf vrf red, vtysh's command
'show ip ospf vrf red graceful-restart helper' will not show anything.
But command 'show ip ospf vrf all graceful-restart helper' will work
normally. This fix fixes the display of information by vrf's name.

Example:
frr1# show ip ospf vrf vrf-1 graceful-restart helper
VRF Name: vrf-1

       OSPF Router with ID (192.168.255.81)

 Graceful restart helper support enabled.
 Strict LSA check is enabled.
 Helper supported for Planned and Unplanned Restarts.
 Supported Graceful restart interval: 1800(in seconds).

Signed-off-by: teletajp <[email protected]>
@teletajp teletajp force-pushed the fix_show_ospf_gr_helper_cmd branch from e9924fc to 62754e1 Compare November 15, 2023 13:24
@teletajp
Copy link
Contributor Author

@teletajp please write a more detailed explanation (as in the PR's description) to the commit. With real outputs, it would be even better.

I added a description to the commit

@ton31337 ton31337 merged commit 77ff93b into FRRouting:master Nov 21, 2023
79 checks passed
@ton31337
Copy link
Member

@Mergifyio backport stable/9.1

Copy link

mergify bot commented Nov 21, 2023

backport stable/9.1

✅ Backports have been created

donaldsharp added a commit that referenced this pull request Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants