Skip to content

[Docs] Complete VxLAN commands documentation #3817

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

link19430
Copy link

@link19430 link19430 commented Mar 27, 2025

Why I did it

The VxLAN command documentation was incomplete, missing descriptions for several show and config commands, which could affect user understanding and operation.

How I did it

Added detailed descriptions for show vxlan and 'config vxlan' commands.

How to verify it

Review the documentation to ensure all VxLAN-related commands are properly described and clearly explained.

The VxLAN command documentation was incomplete, missing descriptions for several `show` and `config` commands, which could affect user understanding and operation.

Added detailed descriptions for `show vxlan` and 'config vxlan' commands.
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

The VxLAN command documentation contained incorrect parameter names:
- `show vxlan remote_vni` should be `show vxlan remotevni`
- `show vxlan remote_mac` should be `show vxlan remotemac`

These inconsistencies could cause confusion for users following the documentation.
- Updated the command names in the documentation to match the correct CLI syntax.
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@link19430
Copy link
Author

@maipbui Please help review, thanks

link19430

This comment was marked as duplicate.

@link19430
Copy link
Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@link19430
Copy link
Author

Hi @maipbui
Is the 'Semgrep' need to be approved to verify ?
Thanks.

@link19430
Copy link
Author

Hi @maipbui
Could you help merge it ? thanks.

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.

3 participants