Disable fsync throughout TAP test suites.

Most suites already did so via start_test_server(), but the pg_rewind,
pg_ctl and pg_controldata suites ran a postmaster or initdb with fsync
enabled.  This halves the pg_rewind suite's runtime on buildfarm member
tern.  It makes tern and that machine's other buildfarm members less
vulnerable to noise failures from postmaster startup overrunning the 60s
pg_ctl timeout.  Back-patch to 9.5, where pg_rewind was introduced.

diff --git a/src/bin/pg_controldata/t/001_pg_controldata.pl b/src/bin/pg_controldata/t/001_pg_controldata.pl
index e36fa2d45d99ccf404bdd9a896d61c4f0f62e3f8..e2b0d420a23c8d8575b610026733c179a5574ebb 100644 (file)
--- a/src/bin/pg_controldata/t/001_pg_controldata.pl
+++ b/src/bin/pg_controldata/t/001_pg_controldata.pl
@@ -11,6 +11,6 @@ program_options_handling_ok('pg_controldata');
 command_fails(['pg_controldata'], 'pg_controldata without arguments fails');
 command_fails([ 'pg_controldata', 'nonexistent' ],
        'pg_controldata with nonexistent directory fails');
-system_or_bail 'initdb', '-D', "$tempdir/data", '-A', 'trust';
+standard_initdb "$tempdir/data";
 command_like([ 'pg_controldata', "$tempdir/data" ],
        qr/checkpoint/, 'pg_controldata produces output');

diff --git a/src/bin/pg_ctl/t/001_start_stop.pl b/src/bin/pg_ctl/t/001_start_stop.pl
index dae47a80135a33f39856f57232ec66f30a084e65..177a16f7afe80e83133ed2bd5b91adb88fef860d 100644 (file)
--- a/src/bin/pg_ctl/t/001_start_stop.pl
+++ b/src/bin/pg_ctl/t/001_start_stop.pl
@@ -14,12 +14,14 @@ program_options_handling_ok('pg_ctl');
 command_exit_is([ 'pg_ctl', 'start', '-D', "$tempdir/nonexistent" ],
        1, 'pg_ctl start with nonexistent directory');
 
-command_ok([ 'pg_ctl', 'initdb', '-D', "$tempdir/data" ], 'pg_ctl initdb');
+command_ok([ 'pg_ctl', 'initdb', '-D', "$tempdir/data", '-o', '-N' ],
+       'pg_ctl initdb');
 command_ok(
        [ $ENV{PG_REGRESS}, '--config-auth',
                "$tempdir/data" ],
        'configure authentication');
 open CONF, ">>$tempdir/data/postgresql.conf";
+print CONF "fsync = off\n";
 if (! $windows_os)
 {
        print CONF "listen_addresses = ''\n";

diff --git a/src/test/perl/TestLib.pm b/src/test/perl/TestLib.pm
index 4927d45d59c87c1296b57aaed7b626d1879ceb3e..02533ebde538bae0a9b7938cb7358f378f9406d3 100644 (file)
--- a/src/test/perl/TestLib.pm
+++ b/src/test/perl/TestLib.pm
@@ -143,6 +143,7 @@ sub standard_initdb
 
        open CONF, ">>$pgdata/postgresql.conf";
        print CONF "\n# Added by TestLib.pm)\n";
+       print CONF "fsync = off\n";
        if ($windows_os)
        {
                print CONF "listen_addresses = '127.0.0.1'\n";
@@ -189,7 +190,7 @@ sub start_test_server
        standard_initdb "$tempdir/pgdata";
 
        $ret = system_log('pg_ctl', '-D', "$tempdir/pgdata", '-w', '-l',
-         "$log_path/postmaster.log", '-o', "--fsync=off --log-statement=all",
+         "$log_path/postmaster.log", '-o', "--log-statement=all",
          'start');
 
        if ($ret != 0)