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

Start sequence #754

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Start sequence #754

wants to merge 5 commits into from

Conversation

burmanm
Copy link
Contributor

@burmanm burmanm commented Feb 10, 2025

What this PR does:

  • Prevent single bad behaving pod from blocking the start sequence in case of cluster stop/start or if new nodes are spawned.
  • Start from the highest index number first like StatefulSet controller does

This is branching from PR #753

Which issue(s) this PR fixes:
Fixes #750
Fixes #757

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CHANGELOG.md updated (not required for documentation PRs)
  • CLA Signed: DataStax CLA

…, not containerStatus (since it is not going to be started)
…est index first and then go lower. At the same time, if a pod has failed to start on the previous attempt, move it to the bottom of the starting sequence to prevent one failing node from blocking the start sequence of other pods (in a non-bootstrapped / stopped/start situation)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant