Skip to content
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

Show Vnet alias to name mapping #924

Merged
merged 1 commit into from
May 28, 2020
Merged

Conversation

prsunny
Copy link
Contributor

@prsunny prsunny commented May 27, 2020

- What I did
Mapping for alias to vnet name

- How I did it

- How to verify it
Run show command

- 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)

admin@str-sn-01:~$ show vnet alias 
Alias        Name
-----------  ------
vnet-guid-1  Vnet1
vnet-guid-2  Vnet2

admin@str-sn-01:~$ show vnet alias vnet-guid-2
Alias        Name
-----------  ------
vnet-guid-2  Vnet2

@prsunny
Copy link
Contributor Author

prsunny commented May 27, 2020

@dzhangalibaba, please review

@prsunny prsunny requested review from alguadam and lguohan and removed request for alguadam May 27, 2020 16:09
@prsunny prsunny merged commit b37135c into sonic-net:master May 28, 2020
@prsunny prsunny deleted the vnet_guid branch May 28, 2020 23:32
abdosi pushed a commit that referenced this pull request Jun 3, 2020
abdosi pushed a commit to abdosi/sonic-utilities that referenced this pull request Aug 4, 2020
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Make sure db_migrator is run after all config are loaded during (sonic-net#926)
Vnet alias mapping (sonic-net#924)
Changes to make lldp show command for multi-npu platforms. (sonic-net#914)
[Mellanox] Fix thermal control issue: use natural sort for fan
status and thermal status (sonic-net#836)
[Mellanox] add document for thermal control related cli (sonic-net#832)
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.

3 participants