Skip to content

[action] [PR:3797] [Techsupport]Add show route-map command to frr dumps in techsupport#3803

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/3797
Mar 13, 2025
Merged

[action] [PR:3797] [Techsupport]Add show route-map command to frr dumps in techsupport#3803
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/3797

Conversation

@mssonicbld
Copy link
Collaborator

What I did

Adding show route-map commamnd to frr dump in techsupport. This will provide detailed information about route-map across different daemons which is useful for debugging.

Example

ZEBRA:
route-map: TO_BGP_PEER_V4 Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 100 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
    Call CHECK_IDF_ISOLATION
  Action:
    Exit routemap
BGP:
route-map: TO_BGP_PEER_V4 Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 100 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
    Call CHECK_IDF_ISOLATION
  Action:
    Exit routemap

How I did it

Adding the command to list of commands dumped from frr in techsupport.

How to verify it

Running techsupport and verifying

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)

<!--
    Please make sure you've read and understood our contributing guidelines:
    https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

    ** Make sure all your commits include a signature generated with `git commit -s` **

    If this is a bug fix, make sure your description includes "closes #xxxx",
    "fixes #xxxx" or "resolves #xxxx" so that GitHub automatically closes the related
    issue when the PR is merged.

    If you are adding/modifying/removing any command or utility script, please also
    make sure to add/modify/remove any unit tests from the tests
    directory as appropriate.

    If you are modifying or removing an existing 'show', 'config' or 'sonic-clear'
    subcommand, or you are adding a new subcommand, please make sure you also
    update the Command Line Reference Guide (doc/Command-Reference.md) to reflect
    your changes.

    Please provide the following information:
-->

#### What I did
Adding show route-map commamnd to frr dump in techsupport. This will provide detailed information about route-map across different daemons which is useful for debugging.

Example
```
ZEBRA:
route-map: TO_BGP_PEER_V4 Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 100 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
    Call CHECK_IDF_ISOLATION
  Action:
    Exit routemap
BGP:
route-map: TO_BGP_PEER_V4 Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 100 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
    Call CHECK_IDF_ISOLATION
  Action:
    Exit routemap
```

#### How I did it
Adding the command to list of commands dumped from frr in techsupport.

#### How to verify it
Running techsupport and verifying

#### 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)
@mssonicbld
Copy link
Collaborator Author

Original PR: #3797

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 04bc01d into sonic-net:202411 Mar 13, 2025
7 checks passed
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-utilities that referenced this pull request Mar 31, 2025
sonic-net#151)

```<br>* 93d21bc1 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-utilities into 202412 (2025-03-13) [Sonic Automation]
* 04bc01d - (origin/202411) [Techsupport]Add show route-map command to frr dumps in techsupport (sonic-net#3803) (2025-03-13) [mssonicbld]
* 7a3f4ad - Fix the interface name length validation for subinterface (sonic-net#3799) (2025-03-11) [mssonicbld]
* 357f95d - enable pfcwd for backplane ports (sonic-net#3796) (2025-03-07) [mssonicbld]
* 7f316e7 - revert YANG check in db_migrator (sonic-net#3794) (2025-03-06) [mssonicbld]
* c01a4f5 - Add namespace support for show interface neighbor expected (sonic-net#3790) (2025-03-03) [mssonicbld]
* 17a7b18 - Fix `vnet_route_check` for active and inactive routes, add `--all` option (sonic-net#3780) (2025-02-25) [mssonicbld]
* 5cc1012 - [202411][FC] remove FC delay field (sonic-net#3768) (2025-02-21) [Stepan Blyshchak]
* 8ceba38 - [QOS] Skip showing unnecessary warning message (sonic-net#3762) (2025-02-11) [mssonicbld]
* 04b60e2 - Fix call for spanning-tree commands in dump script (sonic-net#3752) (2025-02-07) [mssonicbld]
* bbe29ae - [show][interface] Add changes for show interface errors command (sonic-net#3749) (2025-02-04) [mssonicbld]<br>```
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-utilities that referenced this pull request Mar 31, 2025
sonic-net#152)

```<br>* e45cc495 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-utilities into 202412 (2025-03-14) [Sonic Automation]
* b197a61 - (origin/202411) [db_migrator] Migrate tunnel table (sonic-net#3806) (2025-03-14) [mssonicbld]
* 5b765f3 - [GCU] Add data acl table and rule check (sonic-net#3807) (2025-03-14) [mssonicbld]
* 04bc01d - [Techsupport]Add show route-map command to frr dumps in techsupport (sonic-net#3803) (2025-03-13) [mssonicbld]
* 7a3f4ad - Fix the interface name length validation for subinterface (sonic-net#3799) (2025-03-11) [mssonicbld]
* 357f95d - enable pfcwd for backplane ports (sonic-net#3796) (2025-03-07) [mssonicbld]
* 7f316e7 - revert YANG check in db_migrator (sonic-net#3794) (2025-03-06) [mssonicbld]
* c01a4f5 - Add namespace support for show interface neighbor expected (sonic-net#3790) (2025-03-03) [mssonicbld]
* 17a7b18 - Fix `vnet_route_check` for active and inactive routes, add `--all` option (sonic-net#3780) (2025-02-25) [mssonicbld]
* 5cc1012 - [202411][FC] remove FC delay field (sonic-net#3768) (2025-02-21) [Stepan Blyshchak]
* 8ceba38 - [QOS] Skip showing unnecessary warning message (sonic-net#3762) (2025-02-11) [mssonicbld]
* 04b60e2 - Fix call for spanning-tree commands in dump script (sonic-net#3752) (2025-02-07) [mssonicbld]
* bbe29ae - [show][interface] Add changes for show interface errors command (sonic-net#3749) (2025-02-04) [mssonicbld]<br>```
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-utilities that referenced this pull request Mar 31, 2025
sonic-net#154)

```<br>* 94af3eab - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-utilities into 202412 (2025-03-15) [Sonic Automation]
* b197a61 - (origin/202411) [db_migrator] Migrate tunnel table (sonic-net#3806) (2025-03-14) [mssonicbld]
* 5b765f3 - [GCU] Add data acl table and rule check (sonic-net#3807) (2025-03-14) [mssonicbld]
* 04bc01d - [Techsupport]Add show route-map command to frr dumps in techsupport (sonic-net#3803) (2025-03-13) [mssonicbld]
* 7a3f4ad - Fix the interface name length validation for subinterface (sonic-net#3799) (2025-03-11) [mssonicbld]
* 357f95d - enable pfcwd for backplane ports (sonic-net#3796) (2025-03-07) [mssonicbld]
* 7f316e7 - revert YANG check in db_migrator (sonic-net#3794) (2025-03-06) [mssonicbld]
* c01a4f5 - Add namespace support for show interface neighbor expected (sonic-net#3790) (2025-03-03) [mssonicbld]
* 17a7b18 - Fix `vnet_route_check` for active and inactive routes, add `--all` option (sonic-net#3780) (2025-02-25) [mssonicbld]
* 5cc1012 - [202411][FC] remove FC delay field (sonic-net#3768) (2025-02-21) [Stepan Blyshchak]
* 8ceba38 - [QOS] Skip showing unnecessary warning message (sonic-net#3762) (2025-02-11) [mssonicbld]
* 04b60e2 - Fix call for spanning-tree commands in dump script (sonic-net#3752) (2025-02-07) [mssonicbld]
* bbe29ae - [show][interface] Add changes for show interface errors command (sonic-net#3749) (2025-02-04) [mssonicbld]<br>```
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-utilities that referenced this pull request Mar 31, 2025
sonic-net#155)

```<br>* 4e7f14b1 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-utilities into 202412 (2025-03-16) [Sonic Automation]
* b197a61 - (origin/202411) [db_migrator] Migrate tunnel table (sonic-net#3806) (2025-03-14) [mssonicbld]
* 5b765f3 - [GCU] Add data acl table and rule check (sonic-net#3807) (2025-03-14) [mssonicbld]
* 04bc01d - [Techsupport]Add show route-map command to frr dumps in techsupport (sonic-net#3803) (2025-03-13) [mssonicbld]
* 7a3f4ad - Fix the interface name length validation for subinterface (sonic-net#3799) (2025-03-11) [mssonicbld]
* 357f95d - enable pfcwd for backplane ports (sonic-net#3796) (2025-03-07) [mssonicbld]
* 7f316e7 - revert YANG check in db_migrator (sonic-net#3794) (2025-03-06) [mssonicbld]
* c01a4f5 - Add namespace support for show interface neighbor expected (sonic-net#3790) (2025-03-03) [mssonicbld]
* 17a7b18 - Fix `vnet_route_check` for active and inactive routes, add `--all` option (sonic-net#3780) (2025-02-25) [mssonicbld]
* 5cc1012 - [202411][FC] remove FC delay field (sonic-net#3768) (2025-02-21) [Stepan Blyshchak]
* 8ceba38 - [QOS] Skip showing unnecessary warning message (sonic-net#3762) (2025-02-11) [mssonicbld]
* 04b60e2 - Fix call for spanning-tree commands in dump script (sonic-net#3752) (2025-02-07) [mssonicbld]
* bbe29ae - [show][interface] Add changes for show interface errors command (sonic-net#3749) (2025-02-04) [mssonicbld]<br>```
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-utilities that referenced this pull request Mar 31, 2025
```<br>* dcf6214 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-utilities into 202412 (2025-03-17) [Sonic Automation]
* b197a61 - (origin/202411) [db_migrator] Migrate tunnel table (sonic-net#3806) (2025-03-14) [mssonicbld]
* 5b765f3 - [GCU] Add data acl table and rule check (sonic-net#3807) (2025-03-14) [mssonicbld]
* 04bc01d - [Techsupport]Add show route-map command to frr dumps in techsupport (sonic-net#3803) (2025-03-13) [mssonicbld]
* 7a3f4ad - Fix the interface name length validation for subinterface (sonic-net#3799) (2025-03-11) [mssonicbld]
* 357f95d - enable pfcwd for backplane ports (sonic-net#3796) (2025-03-07) [mssonicbld]
* 7f316e7 - revert YANG check in db_migrator (sonic-net#3794) (2025-03-06) [mssonicbld]
* c01a4f5 - Add namespace support for show interface neighbor expected (sonic-net#3790) (2025-03-03) [mssonicbld]
* 17a7b18 - Fix `vnet_route_check` for active and inactive routes, add `--all` option (sonic-net#3780) (2025-02-25) [mssonicbld]
* 5cc1012 - [202411][FC] remove FC delay field (sonic-net#3768) (2025-02-21) [Stepan Blyshchak]
* 8ceba38 - [QOS] Skip showing unnecessary warning message (sonic-net#3762) (2025-02-11) [mssonicbld]
* 04b60e2 - Fix call for spanning-tree commands in dump script (sonic-net#3752) (2025-02-07) [mssonicbld]
* bbe29ae - [show][interface] Add changes for show interface errors command (sonic-net#3749) (2025-02-04) [mssonicbld]<br>```
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.

1 participant