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.

Sign up for GitHub

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

Jump to bottom

certain assumptions of read_porttab_mappings is no longer valid #10

Closed
lguohan opened this issue Feb 27, 2017 · 1 comment
Closed

certain assumptions of read_porttab_mappings is no longer valid #10

lguohan opened this issue Feb 27, 2017 · 1 comment

Comments

@lguohan
Copy link
Contributor

lguohan commented Feb 27, 2017

For example, port name for s6000 is from Ethernet0 to Ethernet63, no longer step by 4.

physical_port = logical_port.split('Ethernet').pop()
physical_port = int(physical_port.split('s').pop(0))/4
@wadelnn wadelnn mentioned this issue Jun 6, 2017
Closed
@lguohan
Copy link
Contributor Author

lguohan commented Aug 16, 2018

introduced index column in port_config.ini to solve this problem.

@lguohan lguohan closed this as completed Aug 16, 2018
praveen-li pushed a commit to praveen-li/sonic-utilities that referenced this issue Feb 3, 2020
@zhenggen-xu
[main.py]: Verify from Asic DB that Port are deleted. (sonic-net#10)
8e97325 
* [main.py]: Verify from Asic DB that Port are deleted.

Wait Max for a min.

An internal function is written, which will move to config_mgmt.py
file once delete Port and add Port APIs are merged.

Signed-off-by: Praveen Chaudhary <pchaudhary@linkedin.com>

* [main.py]: Address Review Comments.

Signed-off-by: Praveen Chaudhary <pchaudhary@linkedin.com>
vdahiya12 pushed a commit to vdahiya12/sonic-utilities that referenced this issue Jul 23, 2021
@jleveque @lguohan
[ledd]: Update non-port-name flags to ignore in DB table (sonic-net#10)
d1df471
mihirpat1 pushed a commit to mihirpat1/sonic-utilities that referenced this issue Sep 15, 2023
@jleveque @lguohan
[sonic_eeprom] If reading from what appears to be a corrupt cache fil…
664d6cc 
…e, delete file and read directly from EEPROM (sonic-net#10)

* [sonic_eeprom] If reading from what appears to be a corrupt cache file, delete file and read directly from EEPROM

* Refactor to eliminate recursion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lguohan