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

Fix Belgite platform issues #13389

Merged
merged 1 commit into from
Jan 18, 2023
Merged

Fix Belgite platform issues #13389

merged 1 commit into from
Jan 18, 2023

Conversation

jemifdo
Copy link
Contributor

@jemifdo jemifdo commented Jan 17, 2023

Why I did it

Harden the Belgite platform and fix reported issues

How I did it

Fixed the issues in PSU, FAN, Temperature, LED and Port modules

How to verify it

Verified by loading the image in Belgite platform and confirm that click platform command works
Verified that pmon and its tasks up and running

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

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

Description for the changelog

As part of platform hardening this commit fixes several platform issues in various components like PSU, FAN, Temperature, LED.

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

image

As part of platform hardening this commit fixes several platform issues
in various components like PSU, FAN, Temperature, LED.
@jemifdo jemifdo requested a review from lguohan as a code owner January 17, 2023 09:43
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 17, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: jemifdo / name: Jemston Fernando (9995ba8)

@lguohan lguohan merged commit 892f265 into sonic-net:master Jan 18, 2023
@jemifdo
Copy link
Contributor Author

jemifdo commented Jan 18, 2023

@lguohan I need these changes backported for release 202211 and 202205, can you please do the needful.

@jemifdo
Copy link
Contributor Author

jemifdo commented Jan 23, 2023

@yxieca @lguohan A gentle reminder. Kindly add labels for backporting to release 202211 and 202205, or do I need to manually port it by creating new PRs?

@yxieca
Copy link
Contributor

yxieca commented Feb 21, 2023

@jemifdo What is the reason you want to back port this change to 202205 branch? Is it a committed feature/support for 202205 branch?

@jemifdo
Copy link
Contributor Author

jemifdo commented Feb 21, 2023 via email

@mssonicbld
Copy link
Collaborator

@jemifdo PR conflicts with 202205 branch

@yxieca
Copy link
Contributor

yxieca commented Feb 22, 2023

@jemifdo please create separate PR for 202205 branch.

@mssonicbld
Copy link
Collaborator

@jemifdo PR conflicts with 202211 branch

@yxieca
Copy link
Contributor

yxieca commented Feb 28, 2023

@jemifdo please create separate PR for 202205 branch.

@jemifdo
Copy link
Contributor Author

jemifdo commented Mar 1, 2023

Raised the following PRs for manual backporting the changes
#14036
#14035

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants