Skip to content

test: fix flaky wait for a replica to follow - #268

Open
Totktonada wants to merge 1 commit into
tarantool:masterfrom
Totktonada:fix-flaky-replica-tests
Open

Totktonada wants to merge 1 commit into
tarantool:masterfrom
Totktonada:fix-flaky-replica-tests

Conversation

@Totktonada

Copy link
Copy Markdown
Contributor

The replica status check may get the attempt to index field 'downstream' (a nil value) error. The downstream field is added when the replica subscribes, a moment after registration in _cluster.

The race in the check hits 200-master-replica.t, 240-ro-on-disconnect.t and 230-orphan-not-stalling-init.t. A replica left running by a failed test may also fail the following tests.

The race hits tarantool's integration workflow in CI from time to time.

Drafted by an LLM, conducted and corrected by the commit author.

The replica status check may get the `attempt to index field
'downstream' (a nil value)` error. The `downstream` field is added when
the replica subscribes, a moment after registration in `_cluster`.

The race in the check hits `200-master-replica.t`,
`240-ro-on-disconnect.t` and `230-orphan-not-stalling-init.t`. A replica
left running by a failed test may also fail the following tests.

The race hits tarantool's integration workflow in CI from time to time.

Drafted by an LLM, conducted and corrected by the commit author.

@Col-Waltz Col-Waltz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch!

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.

2 participants