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

[v23.1.x] Topic-aware leadership balancing #9901

Conversation

vbotbuildovich
Copy link
Collaborator

Backport of PR #7997

Adds a couple of abstract classes that will be used to generalize
constraints the leader balancer will be trying to maximize while
balancing leadership. These clases are similar to the partition
balancer's constraints.

(cherry picked from commit 4a4a56a)
Defines a soft_constraint for determining error for how evenly
distributed leadership is across shard in the cluster. Also
moves the logic for a recommended reassignment for improving this
error into a soft_constraint as well.

(cherry picked from commit 24281e5)
This change allows one balance tick to schedule leader transfers
up to that max in flight limit. This allows for the calculate
state of the cluster to be reused for multiple transfer selections
rather than being regenerated every selection.

(cherry picked from commit 497d261)
@vbotbuildovich vbotbuildovich added this to the v23.1.x-next milestone Apr 6, 2023
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Apr 6, 2023
@piyushredpanda
Copy link
Contributor

@ballard26 : The failure seems to be just #9774, we are trying to cut v23.1.7 by this Wednesday; just a default balancer change left here?

@piyushredpanda piyushredpanda modified the milestones: v23.1.x-next, v23.1.7 Apr 10, 2023
@piyushredpanda piyushredpanda marked this pull request as ready for review April 10, 2023 22:43
Copy link
Member

@dotnwat dotnwat left a comment

Choose a reason for hiding this comment

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

lgtm

@piyushredpanda
Copy link
Contributor

Failure seems to be #7418 now. Force-merging.

@piyushredpanda piyushredpanda merged commit ef8e659 into redpanda-data:v23.1.x Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants