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

[Mellanox] [SIMX] Disable cpu_pack and comex_amb for SN5640 and SN5600 #21558

Merged

Conversation

DavidZagury
Copy link
Contributor

@DavidZagury DavidZagury commented Jan 29, 2025

Why I did it

Mellanox SN5600, SN5640 SIMX platform does not support cpu thermal sensors

Work item tracking
  • Microsoft ADO (number only):

How I did it

Update DEVICE_DATA configuration for the SIMX platform

How to verify it

Check no error exists in syslog

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)

@DavidZagury DavidZagury requested a review from lguohan as a code owner January 29, 2025 15:09
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DavidZagury DavidZagury changed the title Disable cpu_pack for SN5640 SIMX and SN5600 SIMX and disable comex_am… [Mellanox] Disable cpu_pack for SN5640 SIMX and SN5600 SIMX and disable comex_am… Jan 29, 2025
@liat-grozovik liat-grozovik changed the title [Mellanox] Disable cpu_pack for SN5640 SIMX and SN5600 SIMX and disable comex_am… [Mellanox] [SIMX] Disable cpu_pack and comex_amb for SN5640 and SN5600 Feb 2, 2025
@liat-grozovik liat-grozovik merged commit 8da3f09 into sonic-net:master Feb 2, 2025
12 checks passed
@liat-grozovik
Copy link
Collaborator

@DavidZagury check conflict and if needed create a separate PR and relate to this one for follow up

@DavidZagury
Copy link
Contributor Author

@liat-grozovik the conflicts is because this PR - #21478 was not cherry-picked to 202411, Noa has added a request to cherry-pick it, and then the conflict will be removed.

@kperumalbfn
Copy link
Contributor

@DavidZagury PR - #21478 has been approved for 202411, we can merge this after that.

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202412: Azure/sonic-buildimage-msft#591

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #21776

prabhataravind pushed a commit to prabhataravind/sonic-buildimage that referenced this pull request Mar 5, 2025
sonic-net#21558)

- Why I did it
Mellanox SN5600, SN5640 SIMX platform does not support cpu thermal sensors

- How I did it
Update DEVICE_DATA configuration for the SIMX platform

- How to verify it
Check no error exists in syslog
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.

5 participants