Skip to content

[backport 2.11] test: stabilize replication rebootstrap and bootstrap_leader tests #10999

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

sergepetrenko
Copy link
Collaborator

No description provided.

The commit 89cd7f4 ("replication: do not account booting replicas
in sync quorum") allowed replicas to leave orphan mode without waiting
for everyone listed in box.cfg.replication.

As a result, bootstrap_leader test started exiting too early, because it
relied on one of the replicas staying in orphan mode until all the
others finish bootstrap. Fix it.

Also, remove "memtx" and "vinyl" configuration for the test. There is
nothing in the test related to engines.

While we're at it, remove the test from fragile list. It hasn't flaked
since long ago, excluding the last couple of weeks, which's fixed now.

Closes tarantool/tarantool-qa#63

NO_CHANGELOG=flaky test
NO_DOC=flaky test

(cherry picked from commit abf54f3)
The commit 89cd7f4 ("replication: do not account booting replicas
in sync quorum") allowed replicas to leave orphan mode without waiting
for everyone listed in box.cfg.replication.

As a result, bootstrap_leader test started exiting too early, because it
relied on one of the replicas staying in orphan mode until all the
others finish bootstrap. Fix it.

While we're at it, remove the test from fragile list. It hasn't flaked
since long ago, excluding the last couple of weeks, which's fixed now.

Closes tarantool/tarantool-qa#56

NO_CHANGELOG=flaky test
NO_DOC=flaky test

(cherry picked from commit 58d9312)
@coveralls
Copy link

Coverage Status

coverage: 85.984% (+0.006%) from 85.978%
when pulling 26a3b08 on sergepetrenko:bootstrap-leader-test-2.11
into abbe796
on tarantool:release/2.11
.

@sergepetrenko sergepetrenko added the full-ci Enables all tests for a pull request label Dec 26, 2024
@sergepetrenko sergepetrenko merged commit 69a0271 into tarantool:release/2.11 Dec 27, 2024
161 checks passed
@sergepetrenko sergepetrenko deleted the bootstrap-leader-test-2.11 branch March 17, 2025 07:21
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.

2 participants