diff options
| author | Michael Paquier <michael@paquier.xyz> | 2025-10-31 11:24:24 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2025-10-31 11:24:24 +0900 |
| commit | c9e38a569c5fe8006a08f63ba6b63d8021704c53 (patch) | |
| tree | cb996cdd2e8f6f55d124277169e902ec7cf0dedc | |
| parent | 3896e861b348f282813f165f60fb8ab3342114c6 (diff) | |
Add test tracking WAL receiver shutdown for primary_conninfo updates
The test introduced by this commit checks that a reload of
primary_conninfo leads to a WAL receiver restarted, by looking at the
request generated in the server logs. This is something for what there
was no coverage.
This has come up for a different patch, while discussing a regression
where a WAL receiver should not be stopped while waiting for a new
position to stream, like at the end of a timeline. In the case of the
other patch, we want to check that this log entry is not generated, but
if the error message is reworded the test would become silently broken.
The test of this commit ensures that we at least keep track the log
message format, for a supported scenario.
Extracted from a larger patch by the same author.
Author: Xuneng Zhou <xunengzhou@gmail.com>
Discussion: https://postgr.es/m/aQKlC1v2_MXGV6_9@paquier.xyz
| -rw-r--r-- | src/test/recovery/t/040_standby_failover_slots_sync.pl | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/src/test/recovery/t/040_standby_failover_slots_sync.pl b/src/test/recovery/t/040_standby_failover_slots_sync.pl index 3059bb8177b..1627e619b1b 100644 --- a/src/test/recovery/t/040_standby_failover_slots_sync.pl +++ b/src/test/recovery/t/040_standby_failover_slots_sync.pl @@ -344,8 +344,20 @@ like( ################################################## $standby1->append_conf('postgresql.conf', "primary_conninfo = '$connstr_1'"); + +# Capture the log position before reload to check for walreceiver +# termination. +$log_offset = -s $standby1->logfile; + $standby1->reload; +# Wait for the walreceiver to be stopped and restarted after a configuration +# reload. When primary_conninfo changes, the walreceiver should be +# terminated and a new one spawned. +$standby1->wait_for_log( + qr/FATAL: .* terminating walreceiver process due to administrator command/, + $log_offset); + ($result, $stdout, $stderr) = $standby1->psql('postgres', "SELECT pg_sync_replication_slots();"); like( |