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

[CEL-SEA2]: fix sonic platform module import issue for sea2 #11447

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

Conversation

qnos
Copy link
Contributor

@qnos qnos commented Jul 14, 2022

Why I did it

Fix platform chassis/component error when importing these platform modules.

How I did it

Fix the platform module import and dependency issues.

How to verify it

Verify the platform in DUT with common show platform commands or import sonic_platform in python3.

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

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

Why I did it
Fix platform chassis/component error when importing these platform modules.

How I did it
Fix the platform module import and dependency issues.
@qnos qnos requested a review from lguohan as a code owner July 14, 2022 03:04
Why I did it
Fix platform chassis/component error when importing these platform modules.

How I did it
Fix the platform module import and dependency issues.
@qnos
Copy link
Contributor Author

qnos commented Jul 18, 2022

/azp run Azure.sonic-buildimage

@qnos
Copy link
Contributor Author

qnos commented Jul 19, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 11447 in repo Azure/sonic-buildimage

@qnos
Copy link
Contributor Author

qnos commented Aug 1, 2022

@lguohan Could you help to review it or assign to sonic-platform to review it? There was one belgite platform compiling issue fixed in this PR, I hope it could be merged into master ASAP. Thanks.

Fix issues in following modules:
1. fan
2. firmware
3. pcieinfo
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 4, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

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.

1 participant