Skip to content

[Mellanox] backport psu sensors dynamic configuration to 202305#19560

Closed
yuazhe wants to merge 4 commits intosonic-net:202305from
yuazhe:202305_backport_psu_sensors_dynamic_configuration
Closed

[Mellanox] backport psu sensors dynamic configuration to 202305#19560
yuazhe wants to merge 4 commits intosonic-net:202305from
yuazhe:202305_backport_psu_sensors_dynamic_configuration

Conversation

@yuazhe
Copy link
Contributor

@yuazhe yuazhe commented Jul 12, 2024

Why I did it

This pr is used to backport #18528.
However, it doesn't contain commit #18722, because that one had been labeled with request for 202305.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

yuazhe added 4 commits July 12, 2024 10:32
Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
add a script to dynamically update the original sensors.conf psu section
to match the currently used psu model.

add a json file to store related platform and psu info.

simx revision is irrelevant, so there is no change on it.

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
once the get_model api detect a psu model changing, it would firstly
regenerate sensors.conf and then restart sensord to active the new
configuration.

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
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.

Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
@yuazhe yuazhe requested a review from lguohan as a code owner July 12, 2024 07:38
@yuazhe yuazhe marked this pull request as draft July 17, 2024 03:21
@yuazhe yuazhe closed this Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant