Skip to content

[FIXED] Consumer not found after meta recovery#7523

Merged
neilalexander merged 1 commit intomainfrom
maurice/meta-snap-consumer-404
Nov 7, 2025
Merged

[FIXED] Consumer not found after meta recovery#7523
neilalexander merged 1 commit intomainfrom
maurice/meta-snap-consumer-404

Conversation

@MauriceVanVeen
Copy link
Copy Markdown
Member

Follow-up of #7510. If a consumer existed in the meta snapshot but was deleted as part of the recovery, it would "re-appear" in the health check and result in a consumer not found error.

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 November 6, 2025 19:10
Copy link
Copy Markdown
Member

@wallyqs wallyqs left a comment

Choose a reason for hiding this comment

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

LGTM

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 9472a92 into main Nov 7, 2025
129 of 136 checks passed
@neilalexander neilalexander deleted the maurice/meta-snap-consumer-404 branch November 7, 2025 08:32
neilalexander added a commit that referenced this pull request Nov 7, 2025
Includes the following:

- #7518
- #7519
- #7520
- #7521
- #7522
- #7523
- #7517
- #7525
- #7526
- #7527
- #7524

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

- #7518
- #7519
- #7520
- #7521
- #7522
- #7523
- #7517
- #7525
- #7526
- #7527
- #7524

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.

3 participants