Skip to content

Comments

[Manual cherry pick] Aligned the sensors file for SN5640 platform#934

Open
AharonMalkin wants to merge 1 commit intoAzure:202412from
AharonMalkin:sensors
Open

[Manual cherry pick] Aligned the sensors file for SN5640 platform#934
AharonMalkin wants to merge 1 commit intoAzure:202412from
AharonMalkin:sensors

Conversation

@AharonMalkin
Copy link

Description of PR

Summary:
This PR aligns the sensors yaml file to the updated sensors on sn5640 nvidia platform.
Sensor file updates exist here: Azure/sonic-buildimage-msft#1889

Manual Cherry pick of sonic-net/sonic-mgmt#21715

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

To align the expected sensors for sn5640.

How did you do it?

Modified the file according to Azure/sonic-buildimage-msft#1889

How did you verify/test it?

Ran the sensors test, 100% pass rate on sn5640

Any platform specific information?

related to sn5640 NVIDIA system only.

Supported testbed topology if it's a new test case?

Documentation

@r12f
Copy link
Contributor

r12f commented Dec 27, 2025

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f
Copy link
Contributor

r12f commented Jan 24, 2026

hi Aharon, pretest is failing and do you mind to help getting it fixed?

@AharonMalkin
Copy link
Author

/azpw run

@AharonMalkin
Copy link
Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 934 in repo Azure/sonic-mgmt.msft

@AharonMalkin
Copy link
Author

Hey @r12f, can you pls rerun as I don't have permission to do so, and the build found, and I cannot view the failure.

@r12f
Copy link
Contributor

r12f commented Feb 1, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AharonMalkin
Copy link
Author

@r12f can review?

@r12f
Copy link
Contributor

r12f commented Feb 16, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants