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

DellEMC : Z9432F determine reboot cause changes #20984

Merged

Conversation

arunlk-dell
Copy link
Contributor

Why I did it

The reboot cause is not properly determined after each reboot. So the reboot history is also not maintained.

How I did it

The failure in determining the reboot cause is due to pcisysfs.py script failure in reading the registers.
The pcisysfs.py script used was using the old python2 format which were failing.
Modified the install scripts to use the latest pcisysfs.py for register read and write.

How to verify it

Unit_Testlog_Z9432F.txt

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)

@arunlk-dell arunlk-dell requested a review from lguohan as a code owner December 2, 2024 05:38
@lguohan lguohan merged commit beafc46 into sonic-net:master Dec 16, 2024
12 checks passed
VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
Why I did it
The reboot cause is not properly determined after each reboot. So the reboot history is also not maintained.

How I did it
The failure in determining the reboot cause is due to pcisysfs.py script failure in reading the registers.
The pcisysfs.py script used was using the old python2 format which were failing.
Modified the install scripts to use the latest pcisysfs.py for register read and write.
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