Skip to content

Comments

Aligned the sensors file for SN5640 platform#21715

Merged
wangxin merged 1 commit intosonic-net:masterfrom
AharonMalkin:sensors
Jan 15, 2026
Merged

Aligned the sensors file for SN5640 platform#21715
wangxin merged 1 commit intosonic-net:masterfrom
AharonMalkin:sensors

Conversation

@AharonMalkin
Copy link
Contributor

@AharonMalkin AharonMalkin commented Dec 16, 2025

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

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

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AharonMalkin
Copy link
Contributor Author

Hey @r12f, cherry pick for 202412: Azure/sonic-mgmt.msft#934

@AharonMalkin
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AharonMalkin
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin merged commit 35f7c69 into sonic-net:master Jan 15, 2026
23 checks passed
PriyanshTratiya pushed a commit to PriyanshTratiya/sonic-mgmt that referenced this pull request Jan 21, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
AndoniSanguesa pushed a commit to AndoniSanguesa/sonic-mgmt that referenced this pull request Jan 21, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Andoni Sanguesa <andoniesanguesa@gmail.com>
AndoniSanguesa pushed a commit to AndoniSanguesa/sonic-mgmt that referenced this pull request Jan 21, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Andoni Sanguesa <andoniesanguesa@gmail.com>
saravanan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 22, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Saravanan Sellappa <saravanan@nexthop.ai>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Feb 12, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: nnelluri-cisco <nnelluri@cisco.com>
AharonMalkin added a commit to AharonMalkin/sonic-mgmt that referenced this pull request Feb 12, 2026
AharonMalkin added a commit to AharonMalkin/sonic-mgmt that referenced this pull request Feb 12, 2026
This reverts commit 35f7c69.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 18, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
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

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.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
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.

4 participants