Skip to content

[action] [PR:18393] Have Nokia-IXR7250E SKUs check for hearbeat lost with sup when Sup kernel panics#298

Merged
mssonicbld merged 1 commit intoAzure:202405from
mssonicbld:cherry/msft-202405/18393
May 19, 2025
Merged

[action] [PR:18393] Have Nokia-IXR7250E SKUs check for hearbeat lost with sup when Sup kernel panics#298
mssonicbld merged 1 commit intoAzure:202405from
mssonicbld:cherry/msft-202405/18393

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
Fixes #18392

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

Changed in 7250e code makes kernel panic on sup cards leave reboot-cause on LCs as Heartbeat with supervisor card lost instead of cold

sonic-net/sonic-buildimage#22034

How did you do it?

Add Nokia-IXR7250E to SKUs checking for Heartbeat lost cause rather than cold cause in test_kdump.py

How did you verify/test it?

TBC

Any platform specific information?

7250E

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

N/A

Documentation

N/A

…rnel panics

<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->

Summary:
Fixes #18392

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

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

### Back port request
- [ ] 202012
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [x] 202405
- [ ] 202411

### Approach
#### What is the motivation for this PR?
Changed in 7250e code makes kernel panic on sup cards leave reboot-cause on LCs as `Heartbeat with supervisor card lost` instead of `cold`

sonic-net/sonic-buildimage#22034
#### How did you do it?
Add `Nokia-IXR7250E` to SKUs checking for `Heartbeat lost` cause rather than `cold` cause in `test_kdump.py`
#### How did you verify/test it?
TBC
#### Any platform specific information?
7250E
#### Supported testbed topology if it's a new test case?
N/A
### Documentation
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->
N/A
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-mgmt#18393

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@mssonicbld mssonicbld merged commit 10e45ba into Azure:202405 May 19, 2025
4 checks passed
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.

1 participant