Skip to content

[Mellanox] Update the Spectrum-2 platform PSU sensor's label in the sensor conf file#7706

Merged
jleveque merged 1 commit intosonic-net:masterfrom
keboliu:psu_sensor_naming
May 25, 2021
Merged

[Mellanox] Update the Spectrum-2 platform PSU sensor's label in the sensor conf file#7706
jleveque merged 1 commit intosonic-net:masterfrom
keboliu:psu_sensor_naming

Conversation

@keboliu
Copy link
Copy Markdown
Collaborator

@keboliu keboliu commented May 25, 2021

Why I did it

The label for PSU related sensors on the Spectrum-2 platform is not aligned with the physical location of the PSU.

How I did it

Update the label in the sensor conf file for those relevant platforms

How to verify it

Shutdown the PSU and check the sensors status aligned.

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

… file

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@jleveque jleveque merged commit 953f913 into sonic-net:master May 25, 2021
yxieca pushed a commit that referenced this pull request May 26, 2021
…ensor conf file (#7706)

#### Why I did it
The label for PSU related sensors on the Spectrum-2 platform is not aligned with the physical location of the PSU. 

#### How I did it
Update the label in the sensor conf file for those relevant platforms

Signed-off-by: Kebo Liu <kebol@nvidia.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
…ensor conf file (sonic-net#7706)

#### Why I did it
The label for PSU related sensors on the Spectrum-2 platform is not aligned with the physical location of the PSU. 

#### How I did it
Update the label in the sensor conf file for those relevant platforms

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@keboliu keboliu deleted the psu_sensor_naming branch February 9, 2022 06:23
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