Skip to content

resolve replication test timing sensitivity - 2nd attempt#9988

Merged
oranagra merged 1 commit intoredis:unstablefrom
oranagra:investigate_ci_error
Dec 22, 2021
Merged

resolve replication test timing sensitivity - 2nd attempt#9988
oranagra merged 1 commit intoredis:unstablefrom
oranagra:investigate_ci_error

Conversation

@oranagra
Copy link
Member

issue started failing after #9878 was merged (made an exiting test more sensitive)
looks like #9982 didn't help, tested this one and it seems to work better.

this commit does two things:

  1. reduce the extra delay i added earlier and instead add more keys, the effect no duration of replication is the same, but the intervals in which the server is responsive to the tcl client is higher.
  2. improve the test infra to print context when assert_error fails.

@oranagra oranagra merged commit b756739 into redis:unstable Dec 22, 2021
@oranagra oranagra deleted the investigate_ci_error branch December 22, 2021 21:37
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