Skip to content

Fix test_system_health issue#21216

Merged
judyjoseph merged 1 commit intosonic-net:masterfrom
JibinBao:fix_sys_health_issue
Nov 7, 2025
Merged

Fix test_system_health issue#21216
judyjoseph merged 1 commit intosonic-net:masterfrom
JibinBao:fix_sys_health_issue

Conversation

@JibinBao
Copy link
Copy Markdown
Contributor

@JibinBao JibinBao commented Nov 6, 2025

Description of PR

Fix issue of #21183.
Only when dut is mellanox, we can call get_platform_data to get platform info.

Summary:
Fixes # (issue)

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

Approach

What is the motivation for this PR?

Fix issue of #21183

How did you do it?

Only when dut is mellanox, we can call get_platform_data to get platform info.

How did you verify/test it?

Run it on non mellanox device

Any platform specific information?

Any

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Nov 6, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Nov 6, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@judyjoseph
Copy link
Copy Markdown
Contributor

judyjoseph commented Nov 6, 2025

@JibinBao there are some flake errors please fix

@JibinBao
Fix test_system_health issue
bf7bcc3 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
@JibinBao JibinBao force-pushed the fix_sys_health_issue branch from 58d3096 to bf7bcc3 Compare November 7, 2025 01:30
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Nov 7, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Nov 7, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

JibinBao commented Nov 7, 2025

@JibinBao there are some flake errors please fix

@judyjoseph Fixed. Could you please review it?

arista-nwolfe
arista-nwolfe approved these changes Nov 7, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@arista-nwolfe arista-nwolfe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this. LGTM.

@judyjoseph judyjoseph merged commit fa23e0b into sonic-net:master Nov 7, 2025
20 checks passed
@arista-nwolfe arista-nwolfe mentioned this pull request Nov 7, 2025
Closed
@JibinBao JibinBao mentioned this pull request Nov 11, 2025
Merged
11 tasks
dcaugher pushed a commit to dcaugher/sonic-mgmt that referenced this pull request Nov 12, 2025
@JibinBao @dcaugher
Fix test_system_health issue (sonic-net#21216)
e201c3c 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
dcaugher pushed a commit to dcaugher/sonic-mgmt that referenced this pull request Nov 12, 2025
@JibinBao @dcaugher
Fix test_system_health issue (sonic-net#21216)
66fb4bc 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
vikumarks pushed a commit to vikumarks/sonic-mgmt that referenced this pull request Dec 1, 2025
@JibinBao @vikumarks
Fix test_system_health issue (sonic-net#21216)
98d11f1 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: vikumarks <[email protected]>
albertovillarreal-keys pushed a commit to albertovillarreal-keys/sonic-mgmt that referenced this pull request Dec 2, 2025
@JibinBao @albertovillarreal-keys
Fix test_system_health issue (sonic-net#21216)
10ae545 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Alberto Villarreal <[email protected]>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
@JibinBao
Fix test_system_health issue (sonic-net#21216)
cbb4b60 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: opcoder0 <[email protected]>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Dec 9, 2025
@JibinBao @nissampa
Fix test_system_health issue (sonic-net#21216)
2a0d297 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Nishanth Sampath Kumar <[email protected]>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Dec 11, 2025
@JibinBao @selldinesh
Fix test_system_health issue (sonic-net#21216)
6b200e5 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: selldinesh <[email protected]>
echuawu pushed a commit to echuawu/sonic-mgmt that referenced this pull request Dec 12, 2025
@JibinBao @echuawu
Fix test_system_health issue (sonic-net#21216)
865e26c 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
saravanan-nexthop pushed a commit to saravanan-nexthop/sonic-mgmt that referenced this pull request Dec 15, 2025
@JibinBao @saravanan-nexthop
Fix test_system_health issue (sonic-net#21216)
63ddbb6 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Saravanan <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
@JibinBao @gshemesh2
Fix test_system_health issue (sonic-net#21216)
c15d587 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
@JibinBao @AharonMalkin
Fix test_system_health issue (sonic-net#21216)
1abcb8e 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
@JibinBao @gshemesh2
Fix test_system_health issue (sonic-net#21216)
0501703 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
@JibinBao @venu-nexthop
Fix test_system_health issue (sonic-net#21216)
94f84ea 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
yifan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 14, 2026
@JibinBao @yifan-nexthop
Fix test_system_health issue (sonic-net#21216)
f842010 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: YiFan Wang <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
@JibinBao @gshemesh2
Fix test_system_health issue (sonic-net#21216)
facc502 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Guy Shemesh <[email protected]>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
@JibinBao @lakshmi-nexthop
Fix test_system_health issue (sonic-net#21216)
6e425c2 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Lakshmi Yarramaneni <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
@JibinBao @ytzur1
Fix test_system_health issue (sonic-net#21216)
355e447 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
@JibinBao @ytzur1
Fix test_system_health issue (sonic-net#21216)
c8f7b9f 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Yael Tzur <[email protected]>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
@JibinBao @abhishek-nexthop
Fix test_system_health issue (sonic-net#21216)
4267539 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
@JibinBao @rraghav-cisco
Fix test_system_health issue (sonic-net#21216)
ad20a1e 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Raghavendran Ramanathan <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
@JibinBao
Fix test_system_health issue (sonic-net#21216)
d357cde 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Apr 1, 2026
@JibinBao @selldinesh
Fix test_system_health issue (sonic-net#21216)
42cbfa3 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: selldinesh <[email protected]>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Apr 20, 2026
@JibinBao @rraghav-cisco
Fix test_system_health issue (sonic-net#21216)
10adb7f 
1. Only when dut is mellanox, we can call get_platform_data to get platform info
2. Relevant issue: sonic-net#21183

Signed-off-by: Raghavendran Ramanathan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@judyjoseph judyjoseph judyjoseph approved these changes

@nhe-NV nhe-NV nhe-NV approved these changes

@arista-nwolfe arista-nwolfe arista-nwolfe approved these changes

Assignees

No one assigned

Labels

regression

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@JibinBao @mssonicbld @judyjoseph @nhe-NV @arista-nwolfe