[show] vnet endpoint [ip/ipv6] command#2342
Merged
siqbal1986 merged 11 commits intosonic-net:masterfrom Oct 3, 2022
siqbal1986:CLI_vnet_endpoint_ip_addition
Merged
[show] vnet endpoint [ip/ipv6] command#2342siqbal1986 merged 11 commits intosonic-net:masterfrom siqbal1986:CLI_vnet_endpoint_ip_addition
siqbal1986 merged 11 commits intosonic-net:masterfrom
siqbal1986:CLI_vnet_endpoint_ip_addition
Conversation
The status may be active, inactive or not present at all. The status field cna have 3 values. Empty : Configured but not in ASIC Active : Configured and active in ASIC. Inacive : Configured but not in ASIC due to lack ofr BFD session.
show vnet endpoint [IPv4/IPv6] This command shows the status of a vxlan tunnel endpoint nad the number of prefixes pointing to it. Also this command can show the prefix details if endpoint ip is specified. This command checks the endpoint state using BFD monitoring session of the endpoint.
…status_addition' into CLI_vnet_endpoint_ip_addition
prsunny
reviewed
Sep 16, 2022
| - Usage: | ||
|
|
||
| ``` | ||
| show vnet endpoint <ipv4_address/ipv6_address> |
Contributor
There was a problem hiding this comment.
Can we also print the "endpoint_monitor" ip so we have a mapping? BFD session just have endpoint_monitor'
show/vnet.py
Outdated
| for idx, endpoint in enumerate(endpoints): | ||
| monitor_dict[endpoint] = monitors[idx] | ||
| if endpoint not in prefix_count: | ||
| prefix_count[endpoint] =0 |
Contributor
There was a problem hiding this comment.
space after = for consistency
show/vnet.py
Outdated
| for k in vnet_rt_keys: | ||
| val = appl_db.get_all(appl_db.APPL_DB, k) | ||
| endpoints = val.get('endpoint').split(',') | ||
| monitors = val.get('endpoint_monitor').split(',') |
Contributor
There was a problem hiding this comment.
Extra space after =. Please fix spacing in all places
prsunny
approved these changes
Sep 30, 2022
Contributor
|
@siqbal1986 this change cannot be cherry-picked to 202205 branch cleanly. Please create separate PR. |
EdenGri
pushed a commit
to EdenGri/sonic-utilities
that referenced
this pull request
Oct 12, 2022
mdanish-kh
pushed a commit
to hamnarauf/sonic-utilities
that referenced
this pull request
Oct 22, 2022
siqbal1986
added a commit
to siqbal1986/sonic-utilities
that referenced
this pull request
Nov 2, 2022
* [show] vnet endpoint [ip/ipv6] command
siqbal1986
added a commit
that referenced
this pull request
Nov 3, 2022
What I did Created a new CLI Command Show vnet endpoint with option to give IPv4 or IPv6 address parameter. How I did it This command goes over all vxlan tunnels and checks each endpoints monitor session to populate the endpoint state. while doing so it also tracks the prefixes pointing to an endpoint. How to verify it run show vnet endpoint or show vnet endpoint<ipv4 address/ipv6 address>. >>show vnet endpoint Endpoint Endpoint Monitor prefix count status --------------------- --------------------- -------------- -------- fddd:a100:a251::a10:1 fddd:a100:a251::a10:1 1 Unknown fddd:a101:a251::a10:1 fddd:a101:a251::a10:1 1 Down 100.251.7.1 100.251.7.1 3 Up >>show vnet endpoint fddd:a101:a251::a10:1 Endpoint Endpoint Monitor prefix status --------------------- ------------------------- ---------------------------- -------- fddd:a101:a251::a10:1 ['fddd:a101:a251::a10:1'] ['fddd:a156:a251::a6:1/128'] Down >>show vnet endpoint 100.251.7.1 Endpoint Endpoint Monitor prefix status ----------- ------------------ ------------------------------------------------------------ -------- 100.251.7.1 ['100.251.7.1'] ['160.162.191.1/32', '160.163.191.1/32', '160.164.191.1/32'] Up Previous command output (if the output of a command-line utility has changed) None New command output (if the output of a command-line utility has changed) NA
preetham-singh
pushed a commit
to preetham-singh/sonic-utilities
that referenced
this pull request
Nov 21, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What I did
Created a new CLI Command Show vnet endpoint with option to give IPv4 or IPv6 address parameter.
How I did it
This command goes over all vxlan tunnels and checks each endpoints monitor session to populate the endpoint state. while doing so it also tracks the prefixes pointing to an endpoint.
How to verify it
run show vnet endpoint or show vnet endpoint<ipv4 address/ipv6 address>.
Previous command output (if the output of a command-line utility has changed)
None
New command output (if the output of a command-line utility has changed)
NA