-
Notifications
You must be signed in to change notification settings - Fork 678
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
[debug dump util] Changes for EVPN and VxLAN dump module #1892
Conversation
Signed-off-by: Sudharsan Dhamal Gopalarathnam <[email protected]>
@SuvarnaMeenakshi @srj102 Please review |
This pull request introduces 4 alerts when merging 16575c4 into 9017d99 - view on LGTM.com new alerts:
|
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@prsunny - Can you please review ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comment: https://github.com/Azure/sonic-utilities/blob/3641d350d19a2317bbe6a507da117e1ee24cc092/dump/plugins/evpn.py#L70 - line is too long , can be modified for readability.
619cfbf
Done |
Signed-off-by: Sudharsan Dhamal Gopalarathnam [email protected]
What I did
Added dump utility for EVPN and VxLAN related modules
How I did it
Added changes implementing dump for vxlan and evpn tables
How to verify it
Added UT to verify the changes.
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)