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.

Sign up for GitHub

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

Jump to bottom

fix(operator): remove unnecessary pod from autofailover list #530

Merged
merged 1 commit into from
Oct 21, 2024
Merged

fix(operator): remove unnecessary pod from autofailover list #530

merged 1 commit into from
Oct 21, 2024

Conversation

kevinliu24
Copy link
Contributor

What type of PR is this?

  • bug
  • feature
  • enhancement

What problem(s) does this PR solve?

Issue(s) number:

#529

Description:

How do you solve it?

Special notes for your reviewer, ex. impact of this fix, design document, etc:

@kevinliu24 kevinliu24 self-assigned this Oct 19, 2024
MegaByte875
MegaByte875 approved these changes Oct 21, 2024
View reviewed changes
Copy link
Contributor

@MegaByte875 MegaByte875 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MegaByte875 MegaByte875 merged commit 77f0684 into release-1.8 Oct 21, 2024
1 check passed
@kevinliu24 kevinliu24 linked an issue Oct 21, 2024 that may be closed by this pull request
Autofailover Stuck in Unable to Find Pod Status After graphd scale down #529
Closed
@kevinliu24 kevinliu24 deleted the autofailover-fix branch October 21, 2024 17:00
@wey-gu wey-gu mentioned this pull request Oct 26, 2024
Open
kevinliu24 added a commit that referenced this pull request Nov 13, 2024
@kevinliu24
fix(operator): remove unnecessary pod from autofailover list (#530)
bd6d685
MegaByte875 pushed a commit that referenced this pull request Nov 13, 2024
@kevinliu24
[operator] Cherry pick auto-failover and leader balance commits (#534)
96cf048 
* fix(operator): remove unnecessary pod from autofailover list (#530)

* [operator] run leader balance multiple times until balanced. (#532)

[operator] run leader balance mutiple times until balanced.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MegaByte875 MegaByte875 MegaByte875 approved these changes

Assignees

@kevinliu24 kevinliu24

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Autofailover Stuck in Unable to Find Pod Status After graphd scale down
2 participants
@kevinliu24 @MegaByte875