Skip to content

[show]: If interface has no alias, 'show interfaces alias' outputs interface name as alias#115

Merged
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:alias_fix
Oct 6, 2017
Merged

[show]: If interface has no alias, 'show interfaces alias' outputs interface name as alias#115
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:alias_fix

Conversation

@jleveque
Copy link
Contributor

@jleveque jleveque commented Oct 2, 2017

This should not be necessary, because if port_config.ini does not contain aliases, sonic-cfggen should create the 'alias' element with a value of the port name, but it doesn't hurt to add this check in case there is an inconsistency between minigraph and port_config.ini.

@jleveque jleveque self-assigned this Oct 2, 2017
@jleveque jleveque requested review from lguohan and taoyl-ms October 2, 2017 22:09
@jleveque jleveque merged commit abd5e8a into sonic-net:master Oct 6, 2017
@jleveque jleveque deleted the alias_fix branch October 6, 2017 21:36
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Remove the use of constant PORT_CONFIG_INI in 201911 in sonic_sfp
[sff8472.py] Make hex keys all lowercase (sonic-net#115)
mihirpat1 pushed a commit to mihirpat1/sonic-utilities that referenced this pull request Sep 15, 2023
Alpha chars in hex-based keys should be lowercase
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-utilities that referenced this pull request Mar 20, 2025
sonic-net#115)

```<br>* fef2da88 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-utilities into 202412 (2025-02-13) [Sonic Automation]
* 8ceba38 - (origin/202411) [QOS] Skip showing unnecessary warning message (sonic-net#3762) (2025-02-11) [mssonicbld]
* 04b60e2 - Fix call for spanning-tree commands in dump script (sonic-net#3752) (2025-02-07) [mssonicbld]
* bbe29ae - [show][interface] Add changes for show interface errors command (sonic-net#3749) (2025-02-04) [mssonicbld]<br>```
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