Skip to content

[backport 3.3] replication: update raft instance id on anon replica register#11944

Merged
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11940
Oct 16, 2025
Merged

[backport 3.3] replication: update raft instance id on anon replica register#11944
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11940

Conversation

@TarantoolBot
Copy link
Collaborator

@TarantoolBot TarantoolBot commented Oct 16, 2025

(This PR is a backport of #11940 to release/3.3 to a future 3.3.4 release.)


The raft subsystem has its own copy of an instance id, raft::self. It's set during initial box.cfg() call, after local recovery or bootstrap.

It was forgotten to update raft::self after an anonymous replica register. This could lead to various assertion failures in raft code in debug build, and in the release build the raft state would be silently corrupted. Fix this.

Closes #11938

NO_DOC=bugfix

The raft subsystem has its own copy of an instance id, raft::self. It's
set during initial `box.cfg()` call, after local recovery or bootstrap.

It was forgotten to update raft::self after an anonymous replica
register. This could lead to various assertion failures in raft code in
debug build, and in the release build the raft state would be silently
corrupted. Fix this.

Closes #11938

NO_DOC=bugfix

(cherry picked from commit a4d038d)
@TarantoolBot TarantoolBot requested a review from a team as a code owner October 16, 2025 11:54
@TarantoolBot TarantoolBot changed the title [Backport release/3.3] replication: update raft instance id on anon replica register [backport 3.3] replication: update raft instance id on anon replica register Oct 16, 2025
@coveralls
Copy link

Coverage Status

coverage: 87.547% (-0.02%) from 87.563%
when pulling de41ada on backport/release/3.3/11940
into 4ef11b6
on release/3.3
.

@sergepetrenko sergepetrenko merged commit cf0c096 into release/3.3 Oct 16, 2025
25 checks passed
@sergepetrenko sergepetrenko deleted the backport/release/3.3/11940 branch October 16, 2025 14:32
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.

4 participants