honour TEMP_CONFIG in pg_upgrade tests

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: honour TEMP_CONFIG in pg_upgrade tests
Date: 2015-09-17 13:35:04
Message-ID: 55FAC188.30709@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I propose to have the pg_upgrade test honour TEMP_CONFIG as pg_regress
(or its Makefile) does, by applying the patch below. There might be
opther places this should also be done, but this would be a start. The
motivation is to see if it stops the errors we often see on the somewhat
slow axolotl buildfarm member when checking pg_upgrade - axolotl's
TEMP_CONFIG sets up the stats temp directory on a ramdisk, as well as
turning off fsync.

This would be backpatched to 9.2.

cheers

andrew

diff --git a/src/bin/pg_upgrade/test.sh b/src/bin/pg_upgrade/test.sh
index ec3a7ed..c0dcf8b 100644
--- a/src/bin/pg_upgrade/test.sh
+++ b/src/bin/pg_upgrade/test.sh
@@ -21,6 +21,10 @@ unset MAKELEVEL
# authentication configuration.
standard_initdb() {
"$1" -N
+ if [ -n "$TEMP_CONFIG" -a -f "$TEMP_CONFIG" ]
+ then
+ cat "$TEMP_CONFIG" >> "$PGDATA/postgresql.conf"
+ fi
../../test/regress/pg_regress --config-auth "$PGDATA"
}

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2015-09-17 13:37:04 Re: planstate_tree_walker
Previous Message Tom Lane 2015-09-17 13:33:54 Re: planstate_tree_walker