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

[v24.1.x] stm_manager: tighten interaction between bg application and snapshots #21254

Merged
merged 2 commits into from
Jul 5, 2024
Merged

[v24.1.x] stm_manager: tighten interaction between bg application and snapshots #21254

merged 2 commits into from
Jul 5, 2024

Conversation

vbotbuildovich
Copy link
Collaborator

Backport of PR #18576
Fixes: #21252,

bharathv added 2 commits July 5, 2024 01:23
@bharathv @vbotbuildovich
stm_manager: tighten interaction between bg application and snapshots
f1ac0e3 
wait for background applicators to finish before applying raft
snapshots, else there is a danger of next_to_apply offset moving
backwards with interleaved bg_apply and snapshot fibers.

(cherry picked from commit c8290e5)
@bharathv @vbotbuildovich
stm_manager/tests: test racing bg apply with snapshots
657dcdd 
(cherry picked from commit 9ed66c9)
@vbotbuildovich vbotbuildovich added this to the v24.1.x-next milestone Jul 5, 2024
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Jul 5, 2024
@vbotbuildovich vbotbuildovich requested a review from bharathv July 5, 2024 01:23
@bharathv bharathv merged commit cfb5755 into redpanda-data:v24.1.x Jul 5, 2024
18 checks passed
@BenPope BenPope modified the milestones: v24.1.x-next, v24.1.10 Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bharathv bharathv bharathv approved these changes

Assignees
No one assigned
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Milestone
v24.1.10
Development

Successfully merging this pull request may close these issues.
3 participants
@vbotbuildovich @bharathv @BenPope