Hello Daniel,
18.09.2026 01:24, Daniel Gustafsson wrote:
Pushed with tiny bits of tweaking and with a backpatch down to 18. Thanks!
./daniel
The test addition coined with bca67e5a3 is not very robust,
unfortunately. Buildfarm animal serinus found a way to break it [1]
as below:
# Failed test 'oauth_validator_libraries restored'
# at /home/bf/bf-build/serinus/HEAD/pgsql/src/test/modules/oauth_validator/t/001_server.pl line 152.
# got: ' '
# expected: 'validator'
# Looks like you failed 1 test of 171.
I've managed to reproduce this locally with a sleep:
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -569,6 +569,7 @@ ProcessConfigFileInternal(GucContext context, bool applySettings, int elevel)
item->name, item->value)));
}
item->applied = true;
+pg_usleep(100000);
}
else if (scres == 0)
{
It makes oauth_validator/001_server fail deterministically for me.
Could you take a look, please?
[1]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=serinus&dt=2026-09-29%2013%3A57%3A41 Best regards,
Alexander