Skip to content

[202012][show] add new CLI to show tunnel route objects #2255#2659

Merged
zjswhhh merged 4 commits intosonic-net:202012from
zjswhhh:show_mux_tunnel_202012
Feb 8, 2023
Merged

[202012][show] add new CLI to show tunnel route objects #2255#2659
zjswhhh merged 4 commits intosonic-net:202012from
zjswhhh:show_mux_tunnel_202012

Conversation

@zjswhhh
Copy link
Contributor

@zjswhhh zjswhhh commented Feb 7, 2023

What I did

Add new CLI support to show tunnel route objects in APP DB.

sign-off: Jing Zhang [email protected]

How I did it

  • Check if tunnel route object exists for server_ipv4, server_ipv6. If existing, print it out.
  • If not specifying port name, print all tunnel route objects.

How to verify it

  • Added unit tests.
  • Tested on dual testbeds:
    • show mux tunnel-route Ethernet140
       PORT         DEST_TYPE    DEST_ADDRESS    
       -----------  -----------  ---------------  
       Ethernet140  server_ipv4  192.168.0.29/32  
      
    • show mux tunnel-route --json Ethernet140
       {
           "TUNNEL_ROUTE": {
               "Ethernet140": {
                   "server_ipv4": {
                       "DEST": "192.168.0.29/32"
                   }
               }
           }
       }
      

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)

What I did
Add new CLI support to show tunnel route objects in ASIC DB.
sign-off: Jing Zhang [email protected]

How I did it
Check if tunnel route object exists for server_ipv4, server_ipv6, soc_ipv4. If existing, print it out.
If not specifying port name, print all tunnel route objects.

How to verify it
Added unit tests.
Tested on dual testbed.
@zjswhhh zjswhhh marked this pull request as draft February 7, 2023 23:21
@zjswhhh zjswhhh marked this pull request as ready for review February 8, 2023 00:03
@zjswhhh
Copy link
Contributor Author

zjswhhh commented Feb 8, 2023

Hi @theasianpianist - can you help review? It's to manually pick up PR #2255.

@zjswhhh zjswhhh merged commit b3ebba2 into sonic-net:202012 Feb 8, 2023
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