Skip to content

[FIXED] Filestore unlock when message erase fails#7431

Merged
neilalexander merged 1 commit intomainfrom
maurice/fs-no-unlock
Oct 14, 2025
Merged

[FIXED] Filestore unlock when message erase fails#7431
neilalexander merged 1 commit intomainfrom
maurice/fs-no-unlock

Conversation

@MauriceVanVeen
Copy link
Copy Markdown
Member

Follow-up of #5248, mb.eraseMsg and mb.slotInfo can fail, so should make sure to unlock if it errors.

Signed-off-by: Maurice van Veen github@mauricevanveen.com

Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
@MauriceVanVeen MauriceVanVeen requested a review from a team as a code owner October 14, 2025 10:55
Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

LGTM

@neilalexander neilalexander merged commit d9c72fd into main Oct 14, 2025
69 of 70 checks passed
@neilalexander neilalexander deleted the maurice/fs-no-unlock branch October 14, 2025 11:11
neilalexander added a commit that referenced this pull request Oct 14, 2025
Includes the following:

- #7427 
- #7428
- #7429 
- #7431

Signed-off-by: Neil Twigg <neil@nats.io>
neilalexander added a commit that referenced this pull request Oct 28, 2025
Includes the following:

- #7380
- #7384
- #7385
- #7388
- #7395
- #7400
- #7399
- #7401
- #7402
- #7423
- #7424
- #7411
- #7428
- #7429
- #7431
- #7435
- #7433
- #7443
- #7455
- #7465
- #7466
- #7460
- #7484
- #7479

Signed-off-by: Neil Twigg <neil@nats.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants