Skip to content

Conversation

@sergepetrenko
Copy link
Collaborator

(This PR is a backport of #11940 to release/2.11 to a future 2.11.8 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

(cherry picked from commit a4d038d)

@sergepetrenko sergepetrenko self-assigned this Oct 16, 2025
@sergepetrenko sergepetrenko requested a review from a team as a code owner October 16, 2025 12:29
@sergepetrenko sergepetrenko changed the title replication: update raft instance id on anon replica register [backport 2.11] replication: update raft instance id on anon replica register Oct 16, 2025
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 tarantool#11938

NO_DOC=bugfix

(cherry picked from commit a4d038d)
@sergepetrenko sergepetrenko force-pushed the raft-id-update-on-anon-2.11 branch from ae216e4 to 69555e1 Compare October 16, 2025 12:39
@coveralls
Copy link

Coverage Status

coverage: 86.099% (+0.008%) from 86.091%
when pulling 69555e1 on sergepetrenko:raft-id-update-on-anon-2.11
into f456c12
on tarantool:release/2.11
.

@sergepetrenko sergepetrenko added the full-ci Enables all tests for a pull request label Oct 16, 2025
@sergepetrenko
Copy link
Collaborator Author

The fuzzer build fails because it requires C++17 support and also forces some more strict checks on fresh compilers. This will be solved separately in #11902

@sergepetrenko sergepetrenko merged commit b764456 into tarantool:release/2.11 Oct 16, 2025
124 of 126 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables all tests for a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants