[device/celestica] Fix incorrect port index in 128x100G config file. #111
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
After the merge, this should be cherry-pick to other branches of Silverstone, too. |
pphuchar
force-pushed
the
sil-fix-100G-port-config
branch
from
49b1a6b
to
ca95ab7
Compare
tiantianlv
approved these changes
Sep 16, 2019
pphuchar
force-pushed
the
sil-fix-100G-port-config
branch
from
ca95ab7
to
9a261a4
Compare
|
@tiantianlv |
mudsut4ke
pushed a commit
that referenced
this pull request
Jan 25, 2021
* src/sonic-platform-common 7255d3a...111dcf7 (2): > [EEPROM] Add new function part_number_str to TlvInfoDecoder (#121) > Fix fp_port_index error to get correct index type (#111)
mudsut4ke
pushed a commit
that referenced
this pull request
Jan 25, 2021
…nic-net#5924) This should also fix the vsimage check build. Commits include: * src/sonic-platform-common a659219...e4883e5 (7): > [PsuBase] Reference inherited member constant (#142) > Thermalctld APIs for recording min and max temp (#131) > Common power consumption and supply APIs for modular chassis (#136) > [sonic-platform-base] Introduce APIs for modular chassis support (#124) > Remove shebangs from non-executable Python files (#140) > [sonic_eeprom] Make compatible with Python 2 and 3 (#127) > Add unit test infrastructure (#139) * src/sonic-platform-daemons 600d043...be21d91 (9): > [psud] Import of mock libraries determined by environment variable (#117) > [chassisd][thermalctld] Set testing env var at top of file, not in setup_function() (#114) > [chassisd][thermalctld] Import os module, now needed for env var checking (#113) > [chassisd][thermalctld] Import mock libraries based on environment variable (#112) > [xcvrd] Fix the console-script xcvrd startup (#111) > [Thermalctld] Update thermal info to CHASSIS_STATE_DB (#101) > PSUd changes to compute power-budget for Modular chassis (#104) > Introduce chassisd to monitor status of cards on chassis (#97) > [xcvrd] support for integrating Credo Y cable Ports initialization and status updates with xcvrd. (#105)
mudsut4ke
pushed a commit
that referenced
this pull request
Jan 28, 2022
2cd6236 [voq][chassis] Fix for issue in chassis redis server connection check (#110) 9da577d [port_util] Fix issue in function get_interface_oid_map (#114) 740a44c [port_util] Allow system without ports in config db run without errors (#109) d07682e Merge pull request #111 from xumia/azp-coverage 482cac2 [Ethernet-IB][index] Modifed the port_util to support the VoQ Inband Port (#113) 69eac92 Support azp diff coverage Signed-off-by: liora <liora@nvidia.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
- What I did
Fix the incorrect port index mapping in port_config.ini file.
This bug cause sfputil to return invalid output.
- How I did it
Update the index column of 128x100G config file follows front panel port index.
- How to verify it
Run the following command and verify all port can be reset.
Please see the output in the seperates test log(Shred drive)
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)