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

tests: mitigate MaintenanceTest failures #5408

Closed
wants to merge 1 commit into from
Closed

tests: mitigate MaintenanceTest failures #5408

wants to merge 1 commit into from

Conversation

jcsp
Copy link
Contributor

@jcsp jcsp commented Jul 8, 2022 •
edited
Loading

Cover letter

The real fix will be to make the leader balancer aware
of maintenance mode, but the test has become much more
unstable since recent leader balancer changes to do
more movements concurrently, so its worth mitigating
that.

The workaround is to set a short mute timeout so that
muting nodes has no real effect, and a short idle timeout
so that post-maintenance leader movements happen promptly.

Related: https://github.com/redpanda-data/redpanda/issues/4772

Release notes

  • none

@jcsp
tests: mitigate MaintenanceTest failures by disabling leader_balancer
a84feda 
The real fix will be to make the leader balancer aware
of maintenance mode, but the test has become much more
unstable since recent leader balancer changes to do
more movements concurrently, so for the moment just
run the test with the leader balancer disabled.

Related: redpanda-data#4772
@jcsp jcsp closed this Jul 8, 2022
@jcsp jcsp changed the title tests: mitigate MaintenanceTest failures by disabling leader_balancer tests: mitigate MaintenanceTest failures Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dotnwat dotnwat Awaiting requested review from dotnwat

@NyaliaLui NyaliaLui Awaiting requested review from NyaliaLui

Assignees
No one assigned
Labels
area/tests kind/tech-debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jcsp