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: Z9332f fix reboot cause issue #8640

Merged
merged 1 commit into from
Sep 9, 2021

Conversation

aravindmani-1
Copy link
Contributor

@aravindmani-1 aravindmani-1 commented Aug 31, 2021

Why I did it

Power cycle test case fails for Z9332f in sonic-mgmt framework(Fixes #8605).

How I did it

Modified the platform API to return expected strings.

How to verify it

  1. Power cycle the device and verify the reboot reason.
  2. Run sonic-mgmt test_reboot script.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

UT logs:
UT logs.txt

A picture of a cute animal (not mandatory but encouraged)

@aravindmani-1 aravindmani-1 requested a review from lguohan as a code owner August 31, 2021 11:14
@aravindmani-1
Copy link
Contributor Author

@sujinmkang , @tjchadaga can you please review this PR?.

@gechiang
Copy link
Collaborator

gechiang commented Sep 7, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sujinmkang sujinmkang merged commit f53213e into sonic-net:master Sep 9, 2021
@aravindmani-1 aravindmani-1 deleted the z9332f_reboot_cause_fix branch September 9, 2021 04:45
qiluo-msft pushed a commit that referenced this pull request Sep 13, 2021
Why I did it
Power cycle test case fails for Z9332f in sonic-mgmt framework(#8605).

How I did it
Modified the platform API to return expected strings.

How to verify it
Power cycle the device and verify the reboot reason.
Run sonic-mgmt test_reboot script.
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.

[9332][platform_tests] test_power_off_reboot fails since reboot-cause is not recognized after power-cycle
5 participants