sonic-buildimage icon indicating copy to clipboard operation
sonic-buildimage copied to clipboard

[Mellanox] Fix 2700a1 sensors.conf and update psu_sensors.json accordingly

Open yuazhe opened this issue 1 year ago • 3 comments

Why I did it

previously 2700a1 sensors.conf is missing bus section for psu, so add it and update the psu_sensors.json with the info.

fix psu_sensors_conf_updater to treat PSR psu as same as PSF one.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

use sensors command and verify the output

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

  • [ ] 201811
  • [ ] 201911
  • [ ] 202006
  • [ ] 202012
  • [ ] 202106
  • [ ] 202111
  • [ ] 202205
  • [ ] 202211
  • [x] 202305
  • [x] 202311
  • [x] 202405

Tested branch (Please provide the tested image version)

  • [x] 202405
  • [ ]

Description for the changelog

Link to config_db schema for YANG module changes

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

yuazhe avatar Jul 09 '24 07:07 yuazhe

/azpw run Azure.sonic-buildimage

yuazhe avatar Jul 11 '24 02:07 yuazhe

/AzurePipelines run Azure.sonic-buildimage

mssonicbld avatar Jul 11 '24 02:07 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jul 11 '24 02:07 azure-pipelines[bot]

/azpw run Azure.sonic-buildimage

yuazhe avatar Jul 17 '24 03:07 yuazhe

/AzurePipelines run Azure.sonic-buildimage

mssonicbld avatar Jul 17 '24 03:07 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jul 17 '24 03:07 azure-pipelines[bot]

/azpw run Azure.sonic-buildimage

yuazhe avatar Jul 29 '24 06:07 yuazhe

/AzurePipelines run Azure.sonic-buildimage

mssonicbld avatar Jul 29 '24 06:07 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jul 29 '24 06:07 azure-pipelines[bot]

@yuazhe please create a PR against 202311 and correlate it to this one for tracking

liat-grozovik avatar Jul 29 '24 06:07 liat-grozovik

@yuazhe please create a PR against 202311 and correlate it to this one for tracking

The conflict is caused by PSU sensors dynamic configuration https://github.com/sonic-net/sonic-buildimage/pull/18528 doesn't been cherry-pick to 202311 yet.

yuazhe avatar Jul 31 '24 02:07 yuazhe

Discussed offline. The change LGTM.

bingwang-ms avatar Aug 27 '24 03:08 bingwang-ms

Re-adding tags to trigger auto cherry-pick

bingwang-ms avatar Aug 27 '24 16:08 bingwang-ms

Cherry-pick PR to 202311: https://github.com/sonic-net/sonic-buildimage/pull/20034

mssonicbld avatar Aug 27 '24 16:08 mssonicbld

Cherry-pick PR to 202405: https://github.com/sonic-net/sonic-buildimage/pull/20035

mssonicbld avatar Aug 27 '24 16:08 mssonicbld