Skip to content

[v25.1.x] [CORE-14581] cluster: partition_balancer multiple fibers in do_tick() check#28476

Merged
WillemKauf merged 2 commits into
redpanda-data:v25.1.xfrom
vbotbuildovich:backport-pr-28460-v25.1.x-453
Nov 11, 2025
Merged

[v25.1.x] [CORE-14581] cluster: partition_balancer multiple fibers in do_tick() check#28476
WillemKauf merged 2 commits into
redpanda-data:v25.1.xfrom
vbotbuildovich:backport-pr-28460-v25.1.x-453

Conversation

@vbotbuildovich

@vbotbuildovich vbotbuildovich commented Nov 11, 2025

Copy link
Copy Markdown
Collaborator

Backport of PR #28460

@WillemKauf @vbotbuildovich
cluster: move _tick_in_progress assignment
98a4eae 
Make this air-tight.

(cherry picked from commit bad1d9b)
@WillemKauf @vbotbuildovich
cluster: don't allow multiple do_tick()s at once
8696de4 
Check the value of `_tick_in_progress` before attempting a tick.
Multiple concurrent fibers in this function at once is an issue.

(cherry picked from commit e7af181)
@vbotbuildovich vbotbuildovich added this to the v25.1.x-next milestone Nov 11, 2025
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Nov 11, 2025
@WillemKauf WillemKauf enabled auto-merge November 11, 2025 16:43
@vbotbuildovich

vbotbuildovich commented Nov 11, 2025

Copy link
Copy Markdown
Collaborator Author

CI test results

test results on build#76039
test_class test_method test_arguments test_kind job_url test_status passed reason test_history
gtest_raft_rpunit gtest_raft_rpunit unit https://buildkite.com/redpanda/redpanda/builds/76039#019a73cc-aaa3-482b-83ea-e0f5f4fdffd2 FLAKY 1/2

@WillemKauf WillemKauf merged commit 94e973b into redpanda-data:v25.1.x Nov 11, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@WillemKauf WillemKauf WillemKauf approved these changes

Assignees

No one assigned

Labels

area/redpanda kind/backport PRs targeting a stable branch

Projects

None yet

Milestone

v25.1.x-next

Development

Successfully merging this pull request may close these issues.

2 participants

@vbotbuildovich @WillemKauf