9ab2625ce2
371d7f120 added a change that configures the tcp (plaintext) port alongside the tls port, this causes the INFO command for tcp_port to return that instead of the tls port when running in tls, and that broke the sentinel tests that query it. the fix is to add a method that gets the right port from CONFIG instead of relying on the tcp_port info field.
40 lines
1.2 KiB
Tcl
40 lines
1.2 KiB
Tcl
# Test Sentinel configuration consistency after partitions heal.
|
|
|
|
source "../tests/includes/init-tests.tcl"
|
|
|
|
test "We can failover with Sentinel 1 crashed" {
|
|
set old_port [RPort $master_id]
|
|
set addr [S 0 SENTINEL GET-MASTER-ADDR-BY-NAME mymaster]
|
|
assert {[lindex $addr 1] == $old_port}
|
|
|
|
# Crash Sentinel 1
|
|
kill_instance sentinel 1
|
|
|
|
kill_instance redis $master_id
|
|
foreach_sentinel_id id {
|
|
if {$id != 1} {
|
|
wait_for_condition 1000 50 {
|
|
[lindex [S $id SENTINEL GET-MASTER-ADDR-BY-NAME mymaster] 1] != $old_port
|
|
} else {
|
|
fail "Sentinel $id did not receive failover info"
|
|
}
|
|
}
|
|
}
|
|
restart_instance redis $master_id
|
|
set addr [S 0 SENTINEL GET-MASTER-ADDR-BY-NAME mymaster]
|
|
set master_id [get_instance_id_by_port redis [lindex $addr 1]]
|
|
}
|
|
|
|
test "After Sentinel 1 is restarted, its config gets updated" {
|
|
restart_instance sentinel 1
|
|
wait_for_condition 1000 50 {
|
|
[lindex [S 1 SENTINEL GET-MASTER-ADDR-BY-NAME mymaster] 1] != $old_port
|
|
} else {
|
|
fail "Restarted Sentinel did not receive failover info"
|
|
}
|
|
}
|
|
|
|
test "New master [join $addr {:}] role matches" {
|
|
assert {[RI $master_id role] eq {master}}
|
|
}
|