Merge lp:~jtv/launchpad/bug-445011 into lp:launchpad

[Jeroen T. Vermeulen]

[Jeroen T. Vermeulen (jtv)]

= Bug 445011 =

My previous branch was redundant with ongoing work. This one fixes
just the database setup script to work with postgres 8.4.

Jeroen

[Stuart Bishop (stub)]

Cool.

I haven't tested it - this script scares me ;)

review: Approve

=== modified file 'utilities/launchpad-database-setup'
--- utilities/launchpad-database-setup	2009-07-17 00:26:05 +0000
+++ utilities/launchpad-database-setup	2009-10-07 09:55:24 +0000
@@ -17,17 +17,22 @@
 # https://launchpad.canonical.com/DatabaseSetup which are intended for
 # initial Launchpad setup on an otherwise unconfigured postgresql instance
 
-if [ -e /etc/init.d/postgresql-8.3 ]; then
-  pgversion=8.3
-  echo "Using postgres 8.3"
-elif [ -e /etc/init.d/postgresql-8.2]; then
-  pgversion=8.2
-  echo "Using postgres 8.2"
-else
+for pgversion in 8.4 8.3 8.2
+do
+  if [ -e /etc/init.d/postgresql-$pgversion ]
+  then
+    break
+  fi
+done
+
+if [ -z "$pgversion" ]
+then
   echo "Unable to determine your postgres version."
   exit 1
 fi
 
+echo "Using postgres $pgversion"
+
 # Make sure that we have the correct version running on port 5432
 sudo grep -q "port.*5432" /etc/postgresql/$pgversion/main/postgresql.conf
 if [ $? -ne 0 ]; then
@@ -81,9 +86,17 @@
 log_statement='all'
 log_line_prefix='[%t] %q%u@%d '
 fsync = off
+
+EOF
+
+if [ "$pgversion" = 8.2 -o "$pgversion" = 8.3 ]
+then
+  sudo grep -q '^[[:space:]]*max_fsm_relations' /etc/postgresql/$pgversion/main/postgresql.conf || \
+  sudo tee -a /etc/postgresql/$pgversion/main/postgresql.conf <<'EOF'
 max_fsm_relations=2000
 
 EOF
+fi
 
 sudo /etc/init.d/postgresql-$pgversion start