diff options
author | Oran Agra <oran@redislabs.com> | 2022-11-30 22:03:23 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-11-30 22:03:23 +0200 |
commit | b0250b4508c9ed527fb0c885f615ae97975f3b2d (patch) | |
tree | e7adc99e86f2a635c0b1d59b29a16284b8144e0d | |
parent | 72e90695ec8091e483c50a5f0003372f9bebc06d (diff) | |
download | redis-b0250b4508c9ed527fb0c885f615ae97975f3b2d.tar.gz |
Try to fix a race in psync2 test (#11553)
This test sets the master ping interval to 1 hour, in order to avoid
pings in the replicatoin stream incrementing the replication offset,
however, it didn't increase the repl-timeout so on slow machines
where the test took more than 60 seconds, the replicas would drop
and reconnect.
```
*** [err]: PSYNC2: Partial resync after restart using RDB aux fields in tests/integration/psync2.tcl
Replica didn't partial sync
```
The test would detect 4 additional partial syncs where it expects
only one.
-rw-r--r-- | tests/integration/psync2.tcl | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/integration/psync2.tcl b/tests/integration/psync2.tcl index 32158e048..a258f1b83 100644 --- a/tests/integration/psync2.tcl +++ b/tests/integration/psync2.tcl @@ -304,6 +304,10 @@ start_server {} { # In absence of pings, are the instances really able to have # the exact same offset? $R($master_id) config set repl-ping-replica-period 3600 + for {set j 0} {$j < 5} {incr j} { + if {$j == $master_id} continue + $R($j) config set repl-timeout 10000 + } wait_for_condition 500 100 { [status $R($master_id) master_repl_offset] == [status $R(0) master_repl_offset] && [status $R($master_id) master_repl_offset] == [status $R(1) master_repl_offset] && |