Re: [Tarantool-patches] [PATCH 1/7] replication: always send raft state to subscribers

[Vladislav Shpilevoy via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

Thanks for working on this!

On 10.06.2021 15:32, Serge Petrenko via Tarantool-patches wrote:
> Tarantool used to send out raft state on subscribe only when raft was
> enabled. This was a safeguard against partially-upgraded clusters, where
> some nodes had no clue about Raft messages and couldn't handle them
> properly.
> 
> Actually, Raft state should be sent out always. For example, promote
> bumps Raft term, even when raft is disabled, and it's important that
> everyone in cluster has the same term, for the sake of promote at least.
> 
> So, send out Raft state to every subscriber with version >= 2.6.0
> (that's when Raft was introduced).
> 
> Closes #5438
> ---
>  src/box/box.cc                               | 11 +--
>  test/replication/gh-5438-raft-state.result   | 73 ++++++++++++++++++++
>  test/replication/gh-5438-raft-state.test.lua | 30 ++++++++

The test still works when I revert box.cc changes.