Skip to content

[Mellanox] Align sensors labels for human readable output for MSN2700#2365

Merged
yxieca merged 1 commit intosonic-net:masterfrom
shlomibitton:shlomi_sensors_msn2700
Oct 22, 2020
Merged

[Mellanox] Align sensors labels for human readable output for MSN2700#2365
yxieca merged 1 commit intosonic-net:masterfrom
shlomibitton:shlomi_sensors_msn2700

Conversation

@shlomibitton
Copy link
Copy Markdown
Contributor

Signed-off-by: Shlomi Bitton [email protected]

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

Align sensors labels for human readable output for MSN2700.

How did you do it?

Modify 'sku-sensors-data.yml' file with correct labels from 'sonic-buildimage' repo.

How did you verify/test it?

Run sensors test.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@shlomibitton
Copy link
Copy Markdown
Contributor Author

Has a dependency in sonic-net/sonic-buildimage#5661

@yxieca yxieca merged commit 443217e into sonic-net:master Oct 22, 2020
@shlomibitton shlomibitton deleted the shlomi_sensors_msn2700 branch March 25, 2021 08:11
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Update sonic-utilities submodule pointer to include the following:
[202205][subinterface]Added additional checks in portchannel and subinterface commands (sonic-net#2371)
[202205] Use warm-boot infrastructure for fast-boot (sonic-net#2365)
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Update sonic-utilities submodule pointer to include the following:
* b739efc [subinterface]Added additional checks in portchannel and subinterface commands (sonic-net#2345) ([sonic-net#2371](sonic-net/sonic-utilities#2371))
* d01153a Use warm-boot infrastructure for fast-boot ([sonic-net#2365](sonic-net/sonic-utilities#2365))
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