Re: 007_pre_auth.pl fails if injection_points_wakeup hits unintended process

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

In response to

Responses

Browse pgsql-hackers by date

  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