fix path

Obmondo · Mar 28, 2024 · 5eadc41 · 5eadc41
1 parent 33dbe86
commit 5eadc41
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/logical-backup/dump.sh b/logical-backup/dump.sh
@@ -124,12 +124,11 @@ function aws_upload {
 
 function aws_upload_dir {
   declare -r EXPECTED_SIZE="$1"
-  declare -r DB_NAME="$2"
-
+
   # mimic bucket setup from Spilo
   # to keep logical backups at the same path as WAL
   # NB: $LOGICAL_BACKUP_S3_BUCKET_SCOPE_SUFFIX already contains the leading "/" when set by the Postgres Operator
-  PATH_TO_BACKUP="s3://${LOGICAL_BACKUP_S3_BUCKET}/${POSTGRES_OPERATOR}/${LOGICAL_BACKUP_S3_BUCKET_SCOPE_SUFFIX}/${PGUSER}/logical_backups/${DB_NAME}/$(date +%s)"
+  PATH_TO_BACKUP="s3://${LOGICAL_BACKUP_S3_BUCKET}/${POSTGRES_OPERATOR}/${LOGICAL_BACKUP_S3_BUCKET_SCOPE_SUFFIX}/${PGUSER}/logical_backups/$(date +%s)"
 
   args=()
 
@@ -138,7 +137,7 @@ function aws_upload_dir {
   [[ ! -z "${LOGICAL_BACKUP_S3_REGION}" ]] && args+=("--region=${LOGICAL_BACKUP_S3_REGION}")
 
   echo "Uploading dump dir to s3"
-  aws s3 cp "$1" "$PATH_TO_BACKUP" "${args[@]//\'/}"
+  compress "$1" | aws s3 cp - "$PATH_TO_BACKUP" "${args[@]//\'/}"
 }
 function gcs_upload {
   PATH_TO_BACKUP=gs://${LOGICAL_BACKUP_S3_BUCKET}"/"${POSTGRES_OPERATOR}"/"${LOGICAL_BACKUP_S3_BUCKET_SCOPE_SUFFIX}"/logical_backups/"$(date +%s).sql.gz