| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Alexander Lakhin <exclusion(at)gmail(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: 007_pre_auth.pl fails if injection_points_wakeup hits unintended process |
| Date: | 2026-11-02 07:08:13 |
| Message-ID: | aug23e2491A3BCln@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, Nov 01, 2026 at 03:00:00PM +0200, Alexander Lakhin wrote:
> That is, the wakeup can hit autovacuum worker or logical replication
> launcher, not the intended client backend. This test failure can be easily
> reproduced with:
> @@ -753,2 +753,3 @@ InitPostgres(const char *in_dbname, Oid dboid,
> pgstat_bestart_initial();
> +if (AmRegularBackendProcess()) pg_usleep(1000000);
> INJECTION_POINT("init-pre-auth", NULL);
>
> and
> echo "autovacuum_naptime = 1s" >/tmp/temp.config
> TEMP_CONFIG=/tmp/temp.config PROVE_TESTS="t/007*" make -s check -C src/test/authentication
>
> Reproduced starting from e2080261c.
Thanks for the report. I am planning to tackle this one with the
following tweak to the test's conf:
autovacuum = off
max_logical_replication_workers = 0
Does that work for you?
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| preauth-test.patch | text/plain | 639 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bertrand Drouvot | 2026-11-02 07:18:00 | Re: [PATCH] pgstat: Add helper to drop entries from a stats kind's hash |
| Previous Message | Alexander Pyhalov | 2026-11-02 06:58:40 | Re: Asynchronous MergeAppend |