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] Fixes sensors labels for human readable output for MSN3420 #5664

Merged
merged 1 commit into from
Oct 19, 2020
Merged

[Mellanox] Fixes sensors labels for human readable output for MSN3420 #5664

merged 1 commit into from
Oct 19, 2020

Conversation

shlomibitton
Copy link
Contributor

@shlomibitton shlomibitton commented Oct 19, 2020

Signed-off-by: Shlomi Bitton shlomibi@nvidia.com

- Why I did it
Fixes sensors labels for human readable output for MSN3420

- How I did it
Configured the correct labels on 'sensors.conf' file

- How to verify it
run 'sensors' command

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
@jleveque
Copy link
Contributor

Retest mellanox please

@jleveque jleveque merged commit a5242a6 into sonic-net:master Oct 19, 2020
@shlomibitton
Copy link
Contributor Author

Test alignment on sonic-mgmt: sonic-net/sonic-mgmt#2367

abdosi pushed a commit that referenced this pull request Nov 3, 2020
…#5664)

Fixes sensors labels for human readable output for MSN3420

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
…sonic-net#5664)

Fixes sensors labels for human readable output for MSN3420

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
@shlomibitton shlomibitton deleted the shlomi_sensors_msn3420 branch March 24, 2021 20: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