Skip to content

[FIXED] NRG: Quorum reporting after leader stepdown#7522

Merged
neilalexander merged 1 commit intomainfrom
maurice/quorum-after-stepdown
Nov 6, 2025
Merged

[FIXED] NRG: Quorum reporting after leader stepdown#7522
neilalexander merged 1 commit intomainfrom
maurice/quorum-after-stepdown

Conversation

@MauriceVanVeen
Copy link
Copy Markdown
Member

Follow-up of #7402. When shutting down a server with LDM or having the leader step down, all peer timestamps would be cleared. This resulted in quorum being reported as lost for all Raft nodes that the server was leader for, a "NO quorum, stalled" message to be printed, and an advisory to be sent.

This PR fixes that by ensuring the leader remembers the timestamps after stepping down. Once the new leader comes online the other follower's timestamp can still be cleared.

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

@MauriceVanVeen MauriceVanVeen requested a review from a team as a code owner November 6, 2025 15:42
Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
@MauriceVanVeen MauriceVanVeen force-pushed the maurice/quorum-after-stepdown branch from a8cd00c to 8d0552e Compare November 6, 2025 16:10
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 a098c7e into main Nov 6, 2025
48 checks passed
@neilalexander neilalexander deleted the maurice/quorum-after-stepdown branch November 6, 2025 17:39
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.

2 participants