tags 956083 + patch,pending
thanks

Hello Tina!

I have prepared a _very_ simple patch for this issue that lacks
elegance, but I think solves the problem: As it is invoked before the
data rotation, it will notify via stderr and exit before destroying
past data:

diff --git a/autopostgresqlbackup b/autopostgresqlbackup
index 19f2264..2b9eb14 100755
--- a/autopostgresqlbackup
+++ b/autopostgresqlbackup
@@ -367,6 +367,14 @@ dbdump () {
                 pg_dump --username=$USERNAME $PGHOST $OPT $db >> $2
             fi
         fi
+       # Was the dump successful? We can only guess based on the
+       # return status of the command.
+       if [ $? != 0 ]
+       then
+           echo "Errors reported by dump process! (return code: $?)" >&2
+           echo $@ >&2
+           exit 1
+       fi
     done
     return 0
 }


I will now proceed to build the package and upload to the 5-day
delayed queue.

FWIW, this is part of the LatinAmerica 2021.03 BSP ;-)

    https://wiki.debian.org/BSP/2021/03/LatinAmerica

Attachment: signature.asc
Description: PGP signature

Reply via email to