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

[Platform] Apply community PR #34

Merged
merged 6 commits into from
Mar 29, 2024

Conversation

ChrisKe1643
Copy link

Why I did it

Fix mgmt pytest fail issue.

How I did it

Apply community PR :

sonic-net/sonic-buildimage#11593
sonic-net/sonic-buildimage#11716
sonic-net/sonic-buildimage#14361
sonic-net/sonic-buildimage#11805
sonic-net/sonic-buildimage#13973
sonic-net/sonic-buildimage#15859

How to verify it

Check show platform command display information

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

sonic-net/sonic-buildimage#11593
Signed-off-by: Brandon Chuang brandon_chuang@edge-core.com

Why I did it
The customer report of the PCIe Bus Errors upon the SDK initialization.
The root cause is BRCM MAC chipset DNX/XGS series does not support the ASPM function.

How I did it
Based on the internal info and discussion, update "pcie_aspm=off" into
ONIE_PLATFORM_EXTRA_CMDLINE_LINUX of installer.conf to resolve it.

How to verify it
…4630-54PE, AS7326-56X, AS7716-32X, AS7816-64X

sonic-net/sonic-buildimage#11716
Signed-off-by: Roger Ho roger530_ho@edge-core.com

Why I did it
Many SSD still depend on vendor utility to display vendor information, which will show below if there is no related vendor utility in the system.

admin@sonic:~$ show platform ssdhealth --vendor
Device Model : TS32XBTMM1600
Health       : N/A
Temperature  : 100C
N/A

How I did it
The ssdutil check ssd_util.py plugin exists in the system or not. If yes, use that plugin. So we create ssd_util.py to resolve this issue.
If it has no vendor tool to read vendor SSD information, then use generic SSD information to show more detail for users to see.

How to verify it
Refer to the original PR
…feature

sonic-net/sonic-buildimage#14361

Why I did it
Base on Adding PDDF common framework enhancements. #12863 to modify pddf-device.json and platform API.
Add code to pass pytest.

How I did it
Add needed code to sonic_platform and driver.

How to verify it
sonic-cli cmd:
show platform {fan, temperature, firmware status, syseeprom, psustatus}
sfputil cmd:
show {presence, eeprom, lpmode}
Verify function by sonic-mgmt tools:
Test file path: /tests/platform_tests/api/
 Test items: test_chassis.py, test_component.py, test_fan_drawer.py, test_psu.py, test_thermal.py, test_sfp.py
sonic-net/sonic-buildimage#11805
Support dynamic port breakout on AS9716-32D,AS7712-32X,AS7326-56X,AS5835-54X,AS4630-54PE, AS7726-32X, AS5835-54T, AS4630-54NPE, AS9726-32D, AS4630-54TE and AS9736-64D

Why I did it
Support dynamic port breakout on AS9716-32D,AS7712-32X,AS7326-56X,AS5835-54X,AS4630-54PE, AS7726-32X,
AS5835-54T, AS4630-54NPE, AS9726-32D, AS4630-54TE and AS9736-64D

How I did it
Add platform.json and hwsku.json

How to verify it
Refer to the original PR
sonic-net/sonic-buildimage#13973

Why I did it
Update as4630_54pe LED custom_led.bin.

How I did it
Replace custom_led.bin file.

How to verify it
Check LED status.
005_as4630_54pe_community_new-rule.log
…SDK on as4630_54

sonic-net/sonic-buildimage#15859

Why I did it
Adjust stable size parameter of SDK to solve SDK initialization issue.
Work item tracking
Microsoft ADO (number only): No

How I did it
Modify SDK configuration file of as4630_54pe.

How to verify it
Checking on SONiC.master.316689-c970ee0f4
@bryan1978 bryan1978 merged commit 76a729b into edge-core:202311.0 Mar 29, 2024
1 check passed
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.

None yet

2 participants