Then term should be assigned to vote response.

Then term should be assigned to vote response.

It is assigned, but the term is still 0 when the node initializes, unless I misunderstood what you were referring to.
I can attach the full backtrace if that helps, but the fatal is being called in this scenario pretty consistently.

I see. When the raft group is first initialized, it should not have term 0 (INVALID_TERM).

@BusyJay makes sense, thank you!
I will initialize all our nodes with term=1 in their Raft state.

it should not have term 0

Perhaps we check it on initial state loaded? I encountered this issue days before also.

There are two cases: 1. constructing the raft node with existing data, so the node is already initialized and should have a valid term; 2. constructing the raft node without any data, so the node is not initialized and can have term 0.

Though adding a check that term must not be 0 if configuration is not empty should be OK.

Though adding a check that term must not be 0 if configuration is not empty should be OK.

I can work on a PR for that.
We were constructing a raft node without any data, but using pre-vote and priority so we ended up rejecting even before any data was sent, therefore getting this error during rejection message send.
But anyway, this check will help ensure this never happens.

Please send it when you are free, thanks!

Reopened

Created a PR - #513

Came here to open a similar issue and I think it fits here.

I would propose to remove stuff like INVALID_ID, INVALID_INDEX and INVALID_TERM and replace them with Option<NonZeroU64>. This way some bugs are not possible or at least quite obvious. What do you think?