handlers/pgsql: make use of the new lib/vserver functionality

diff --git a/ChangeLog b/ChangeLog
index 691620c272a7dde18eaf73d8d0b8678920653b86..d4e222697396d65b10c078f3f110bd90f466c884 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -17,6 +17,8 @@ version 0.9.3 -- unreleased
         . now works when multiple vservers names are given (separated by space)
           in vsnames config variable
         . make use of new lib/vserver functionality
+        pgsql
+         . make use of new lib/vserver functionality
         rdiff
         . make use of new lib/vserver functionality
         . fixed globbing bug in include, exclude and vsinclude options

diff --git a/handlers/pgsql b/handlers/pgsql
index ccfe7593bf367f59b7490d6bcbab2aa5c90906e6..63062f7084175454940c4eff9be561f73d991a54 100644 (file)
--- a/handlers/pgsql
+++ b/handlers/pgsql
@@ -12,19 +12,20 @@ localhost=`hostname`
 
 # If vservers are configured, decide if the handler should
 # use them or if it should just operate on the host
-if [ "$vservers" == "yes" ]; then
-    if [ ! -z $vsname ]; then          
-       info "using vserver '$vsname'"
-       usevserver=1
-    else
-       info "no vserver name specified, actions will be performed on the host"
-    fi
+local usevserver=no
+if [ $vservers_are_available = yes ]; then
+   if ! vservers_exist "$vsname" ; then
+      fatal "At least one of the vservers listed in vsnames ($vsnames) does not exist."
+   else
+      info "using vserver '$vsname'"
+      usevserver=yes
+   fi
 fi
 
 # As needed, make sure that :
 #   * the specified vserver exists and is running
 #   * the specified vserver or host has the needed executables
-if [ $usevserver ]; then
+if [ $usevserver = yes ]; then
     info "examining vserver '$vsname'"
     # does it exist ?
     vroot="$VROOTDIR/$vsname"
@@ -58,10 +59,10 @@ fi
 [ -d $vroot$backupdir ] || fatal "Backup directory '$vroot$backupdir'"
 
 # give backup dir the good uid and permissions
-# (in respect to the vserver, if $usevserver)
+# (in respect to the vserver, if $usevserver = yes)
 pguid=`getent passwd postgres | awk -F: '{print $3}'`
 [ -n "$pguid" ] || \
-    fatal "No user called postgres`[ $usevserver != 1 ] || echo \" on vserver $vsname\"`."
+    fatal "No user called postgres`[ $usevserver = no ] || echo \" on vserver $vsname\"`."
 debug "chown $pguid $vroot$backupdir"
 chown $pguid $vroot$backupdir
 debug "chmod 700 $vroot$backupdir"
@@ -69,7 +70,7 @@ chmod 700 $vroot$backupdir
 
 # if $databases = all, use pg_dumpall
 if [ "$databases" == "all" ]; then
-    if [ $usevserver ]; then
+    if [ $usevserver = yes ]; then
        execstr="$VSERVER $vsname exec su - postgres -c $PGSQLDUMPALL > $backupdir/${vsname}.sql"
     else
        execstr="su - postgres -c $PGSQLDUMPALL > $backupdir/${localhost}-all.sql"
@@ -90,7 +91,7 @@ if [ "$databases" == "all" ]; then
 # else use pg_dump on each specified database
 else
     for db in $databases; do
-       if [ $usevserver ]
+       if [ $usevserver = yes ]
            then
            execstr="$VSERVER $vsname exec su - postgres -c $PGSQLDUMP $db > $backupdir/${db}.sql"
        else