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

[Edgecore][AS9726-32D] Enhance sonic_platform feature #14237

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

laura-chang
Copy link

Why I did it

Add code to pass pytest.

How I did it

Add needed code to sonic_platform and driver.

How to verify it

Use sonic-mgmt to verify that.

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

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

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)

@laura-chang laura-chang requested a review from lguohan as a code owner March 14, 2023 09:53
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 14, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@laura-chang laura-chang changed the title as9726-32d api2.0 sonic_platform related [Edgecore][AS9726-32D] Enhance sonic_platform feature Mar 14, 2023
laura-chang and others added 7 commits March 21, 2023 16:59
Detail:
    Because port: [1, 3, 5, 7, 9, 11, 13, 15, 17, 19, 21, 23, 25, 27, 29, 31] support ZR/ZR+ for high power transmit.
    So need to add check 400G xcvr on these port's temperature for new thermal plan.

Signed-off-by: michael_shih <michael_shih@edge-core.com>
roger530-ho and others added 2 commits November 20, 2023 07:25
1. Sync the log buffer to the disk before powering off the DUT.
2. Change the decision of FAN direction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants