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

[Mellanox] [202205] Add sodimm sensor #13841

Merged
merged 2 commits into from
Mar 1, 2023

Conversation

Junchao-Mellanox
Copy link
Collaborator

@Junchao-Mellanox Junchao-Mellanox commented Feb 17, 2023

Backport sodimm sensor from PR #12671
Depends on #13372 to be cherry picked to 202205.

Why I did it

Add sodimm sensor

How I did it

Add sodimm sensor

How to verify it

Manual test

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)

@liat-grozovik
Copy link
Collaborator

@Junchao-Mellanox please refer to the PR from master who adds it as well.

@Junchao-Mellanox
Copy link
Collaborator Author

@Junchao-Mellanox please refer to the PR from master who adds it as well.

Done

@liat-grozovik
Copy link
Collaborator

liat-grozovik commented Mar 1, 2023

@Junchao-Mellanox the relevant PR was merged to 202205. Please check if this can be now move from draft.

@Junchao-Mellanox Junchao-Mellanox marked this pull request as ready for review March 1, 2023 09:34
@liat-grozovik liat-grozovik merged commit 740ae96 into sonic-net:202205 Mar 1, 2023
@Junchao-Mellanox Junchao-Mellanox deleted the support-sod branch June 12, 2023 04:38
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.

3 participants