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

Dell S6000: Fix reboot failure issue #6656

Merged
merged 1 commit into from
Feb 5, 2021

Conversation

aravindmani-1
Copy link
Contributor

@aravindmani-1 aravindmani-1 commented Feb 3, 2021

- Why I did it
Reboot command in Dell S6000 failed to reboot the switch.
- How I did it
Added retry mechanism and CPU reset.
- How to verify it
Execute reboot command in Dell S6000.
- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog
S6000_reboot_UT.txt

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

@jleveque
Copy link
Contributor

jleveque commented Feb 4, 2021

@aravindmani-1: You flagged that this patch should be backported to the 201811 branch. Therefore, I assume it should be backported to the 201911, 202006 and 202012 branches as well, no?

@aravindmani-1
Copy link
Contributor Author

Hi @jleveque , Changed the cherry pick request to other branches.
Will raise a separate PR for 201811 branch.

@jleveque jleveque merged commit 85a6314 into sonic-net:master Feb 5, 2021
@aravindmani-1 aravindmani-1 deleted the Fix_S6000_rebootissue branch February 5, 2021 03:50
daall pushed a commit that referenced this pull request Feb 5, 2021
Reboot command in Dell S6000 failed to reboot the switch. Added retry mechanism and CPU reset.
@abdosi
Copy link
Contributor

abdosi commented Feb 19, 2021

@aravindmani-1 Please create PR for 201911.

@abdosi
Copy link
Contributor

abdosi commented Feb 26, 2021

Please create PR.

@rlhui

@aravindmani-1
Copy link
Contributor Author

@abdosi We will create a PR. #6835 will override this PR's changes.
So, we will raise the PR accordingly.

@abdosi
Copy link
Contributor

abdosi commented May 31, 2021

do we need this in 201911 ? #6835 had label removed for 201911

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