Here is an updated version of the pg_upgrade test script I posted a while ago. I've cleaned it up so that it offers moderately user-friendly feedback, it supports check and installcheck mode, and should use all the things from the right directories in either case.
diff --git i/contrib/pg_upgrade/.gitignore w/contrib/pg_upgrade/.gitignore index 03ec737..79c8cdb 100644 --- i/contrib/pg_upgrade/.gitignore +++ w/contrib/pg_upgrade/.gitignore @@ -1 +1,5 @@ /pg_upgrade +# Generated by test suite +delete_old_cluster.sh +/log/ +/tmp_check/ diff --git i/contrib/pg_upgrade/Makefile w/contrib/pg_upgrade/Makefile index 8f3fd7c..dcd2c04 100644 --- i/contrib/pg_upgrade/Makefile +++ w/contrib/pg_upgrade/Makefile @@ -11,6 +11,14 @@ OBJS = check.o controldata.o dump.o exec.o file.o function.o info.o \ PG_CPPFLAGS = -DFRONTEND -DDLSUFFIX=\"$(DLSUFFIX)\" -I$(srcdir) -I$(libpq_srcdir) PG_LIBS = $(libpq_pgport) +check: test.sh + MAKE=$(MAKE) bindir=$(bindir) libdir=$(libdir) $(SHELL) $< --install + +installcheck: test.sh + MAKE=$(MAKE) bindir=$(bindir) libdir=$(libdir) $(SHELL) $< + +EXTRA_CLEAN = delete_old_cluster.sh log/ tmp_check/ + ifdef USE_PGXS PG_CONFIG = pg_config PGXS := $(shell $(PG_CONFIG) --pgxs) diff --git i/contrib/pg_upgrade/test.sh w/contrib/pg_upgrade/test.sh index e69de29..7660951 100644 --- i/contrib/pg_upgrade/test.sh +++ w/contrib/pg_upgrade/test.sh @@ -0,0 +1,95 @@ +#!/bin/sh + +# contrib/pg_upgrade/test.sh +# +# Test driver for pg_upgrade. Initializes a new database cluster, +# runs the regression tests (to put in some data), runs pg_dumpall, +# runs pg_upgrade, runs pg_dumpall again, compares the dumps. +# +# Portions Copyright (c) 1996-2011, PostgreSQL Global Development Group +# Portions Copyright (c) 1994, Regents of the University of California + +set -e + +: ${MAKE=make} +: ${PGPORT=50432} +export PGPORT + +temp_root=$PWD/tmp_check + +if [ "$1" = '--install' ]; then + temp_install=$temp_root/install + bindir=$temp_install/$bindir + libdir=$temp_install/$libdir + + "$MAKE" -s -C ../.. install DESTDIR="$temp_install" + "$MAKE" -s -C ../pg_upgrade_support install DESTDIR="$temp_install" + "$MAKE" -s -C . install DESTDIR="$temp_install" + + # platform-specific magic to find the shared libraries; see pg_regress.c + LD_LIBRARY_PATH=$libdir:$LD_LIBRARY_PATH + export LD_LIBRARY_PATH + DYLD_LIBRARY_PATH=$libdir:$DYLD_LIBRARY_PATH + export DYLD_LIBRARY_PATH + LIBPATH=$libdir:$LIBPATH + export LIBPATH + PATH=$libdir:$PATH + + # We need to make it use psql from our temporary installation, + # because otherwise the installcheck run below would try to + # use psql from the proper installation directory, which might + # be outdated or missing. + EXTRA_REGRESS_OPTS=--psqldir=$bindir + export EXTRA_REGRESS_OPTS +fi + +PATH=$bindir:$PATH +export PATH + +PGDATA=$temp_root/data +export PGDATA +rm -rf "$PGDATA" "$PGDATA".old + +logdir=$PWD/log +rm -rf "$logdir" +mkdir "$logdir" + +set -x + +initdb +pg_ctl start -l "$logdir/postmaster1.log" -w +if "$MAKE" -C ../.. installcheck; then + pg_dumpall >"$temp_root"/dump1.sql || pg_dumpall1_status=$? +else + make_installcheck_status=$? +fi +pg_ctl -m fast stop +if [ -n "$make_installcheck_status" ]; then + exit 1 +fi +if [ -n "$pg_dumpall1_status" ]; then + echo "pg_dumpall of pre-upgrade database cluster failed" + exit 1 +fi + +mv "${PGDATA}" "${PGDATA}.old" + +initdb + +pg_upgrade -d "${PGDATA}.old" -D "${PGDATA}" -b "$bindir" -B "$bindir" + +pg_ctl start -l "$logdir/postmaster2.log" -w +pg_dumpall >"$temp_root"/dump2.sql || pg_dumpall2_status=$? +pg_ctl -m fast stop +if [ -n "$pg_dumpall2_status" ]; then + echo "pg_dumpall of post-upgrade database cluster failed" + exit 1 +fi + +if diff -q "$temp_root"/dump1.sql "$temp_root"/dump2.sql; then + echo PASSED + exit 0 +else + echo "dumps were not identical" + exit 1 +fi diff --git i/src/makefiles/pgxs.mk w/src/makefiles/pgxs.mk index 84a296a..7dc8742 100644 --- i/src/makefiles/pgxs.mk +++ w/src/makefiles/pgxs.mk @@ -207,7 +207,7 @@ ifdef OBJS rm -f $(OBJS) endif ifdef EXTRA_CLEAN - rm -f $(EXTRA_CLEAN) + rm -rf $(EXTRA_CLEAN) endif ifdef REGRESS # things created by various check targets diff --git i/src/test/regress/GNUmakefile w/src/test/regress/GNUmakefile index 90aea6c..ec29391 100644 --- i/src/test/regress/GNUmakefile +++ w/src/test/regress/GNUmakefile @@ -132,7 +132,7 @@ tablespace-setup: ## Run tests ## -REGRESS_OPTS = --dlpath=. +REGRESS_OPTS = --dlpath=. $(EXTRA_REGRESS_OPTS) check: all tablespace-setup $(pg_regress_check) $(REGRESS_OPTS) --schedule=$(srcdir)/parallel_schedule $(MAXCONNOPT) $(TEMP_CONF) $(EXTRA_TESTS)
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers