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] add PSU input voltage and current #11510

Merged
merged 3 commits into from
Aug 10, 2022

Conversation

orfar1994
Copy link
Contributor

@orfar1994 orfar1994 commented Jul 21, 2022

Depends on sonic-net/sonic-platform-common#295 and sonic-net/sonic-platform-daemons#276

Why I did it

Add PSU input voltage and input current to mlnx platform api.

How I did it

Implement 2 function of getting the psu voltage and psu current input:
Get the values from "power/psu{}_curr_in" , "power/psu{}_volt_in"

How to verify it

Manual test.
Run sonic-mgmt regression

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

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

Description for the changelog

Link to config_db schema for YANG module changes

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

2081428662

Signed-off-by: orfar1994 <orfar1994@gmail.com>
Signed-off-by: orfar1994 <orfar1994@gmail.com>
Signed-off-by: orfar1994 <orfar1994@gmail.com>
@orfar1994 orfar1994 requested a review from lguohan as a code owner July 21, 2022 12:13
@ghost
Copy link

ghost commented Jul 21, 2022

CLA assistant check
All CLA requirements met.

@orfar1994
Copy link
Contributor Author

/easycla

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 8, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@orfar1994
Copy link
Contributor Author

/easycla

@liat-grozovik liat-grozovik merged commit aec1248 into sonic-net:master Aug 10, 2022
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.

4 participants