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

[202012] [Mellanox] Add sensor conf file for new version of MSN3700/3700C/4600C platforms #11358

Merged
merged 4 commits into from
Jul 14, 2022

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Jul 6, 2022

Why I did it

MSN3700/3700C/4600C have been re-spined, the new HW version of platforms has different sensors, so need to apply the correct sensor.conf for them.

How I did it

  1. Add new sensor.conf files for the new re-spined platforms.
  2. Enhance the logic of "get_sensors_conf_path" for the related platforms in order to load the correct sensor.conf for each version of platforms.

How to verify it

run sensors test on different versions of platforms

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@liat-grozovik liat-grozovik merged commit aa4379d into sonic-net:202012 Jul 14, 2022
keboliu added a commit to keboliu/sonic-buildimage that referenced this pull request Jul 18, 2022
…C platforms (sonic-net#11358)

- Why I did it
MSN3700/3700C/4600C have been re-spined, the new HW version of platforms has different sensors, so need to apply the correct sensor.conf for them.

- How I did it
Add new sensor.conf files for the new re-spined platforms.
Enhance the logic of "get_sensors_conf_path" for the related platforms in order to load the correct sensor.conf for each version of platforms.

- How to verify it
run sensors test on different versions of platforms
Signed-off-by: Kebo Liu <[email protected]>
@keboliu keboliu deleted the 3700_4600c_sensor_conf branch August 29, 2022 08:19
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