Add use_db_compression option for backup database dumps

Enable optional pg_dump compression (-Z 9) via use_db_compression
boolean flag. Restore auto-detects compressed (.db.gz) or
uncompressed (.db) backups for backward compatibility.

Authored By: Christian M. Adams <chadams@redhat.com>
Assisted By: Claude

Author: rooftopcellist

config/crd/bases/awx.ansible.com_awxbackups.yaml

@@ -88,6 +88,10 @@ spec:
               pg_dump_suffix:
                 description: Additional parameters for the pg_dump command
                 type: string
+              use_db_compression:
+                description: Enable compression for database dumps. When true, pg_dump uses built-in compression. Default is false.
+                type: boolean
+                default: false
               postgres_label_selector:
                 description: Label selector used to identify postgres pod for backing up data
                 type: string

config/manifests/bases/awx-operator.clusterserviceversion.yaml

@@ -175,6 +175,12 @@ spec:
         path: additional_labels
         x-descriptors:
         - urn:alm:descriptor:com.tectonic.ui:advanced
+      - description: Enable compression for database dumps using pg_dump built-in compression
+        displayName: Use DB Compression
+        path: use_db_compression
+        x-descriptors:
+        - urn:alm:descriptor:com.tectonic.ui:advanced
+        - urn:alm:descriptor:com.tectonic.ui:booleanSwitch
       - displayName: Node Selector for backup management pod
         path: db_management_pod_node_selector
         x-descriptors:

roles/backup/defaults/main.yml

@@ -42,6 +42,9 @@ backup_resource_requirements:
 # Allow additional parameters to be added to the pg_dump backup command
 pg_dump_suffix: ''
 
+# Enable compression for database dumps
+use_db_compression: false
+
 # Labels defined on the resource, which should be propagated to child resources
 additional_labels: []
 

roles/backup/tasks/postgres.yml

@@ -64,13 +64,17 @@
     command: >-
       mkdir -p {{ backup_dir }}
 
+- name: Set database filename
+  set_fact:
+    _db_filename: "{{ use_db_compression | bool | ternary('tower.db.gz', 'tower.db') }}"
+
 - name: Precreate file for database dump
   k8s_exec:
     namespace: "{{ backup_pvc_namespace }}"
     pod: "{{ ansible_operator_meta.name }}-db-management"
     container: "{{ ansible_operator_meta.name }}-db-management"
     command: >-
-      touch {{ backup_dir }}/tower.db
+      touch {{ backup_dir }}/{{ _db_filename }}
 
 - name: Set resolvable_db_host
   set_fact:
@@ -121,6 +125,7 @@
       -d {{ awx_postgres_database }}
       -p {{ awx_postgres_port }}
       -F custom
+      {{ use_db_compression | bool | ternary('-Z 9', '') }}
       {{ pg_dump_suffix }}
   no_log: "{{ no_log }}"
 
@@ -147,7 +152,7 @@
       trap 'end_keepalive \"$keepalive_file\" \"$keepalive_pid\"' EXIT SIGINT SIGTERM
       echo keepalive_pid: $keepalive_pid
       set -e -o pipefail
-      PGPASSWORD='{{ awx_postgres_pass }}' {{ pgdump }} > {{ backup_dir }}/tower.db
+      PGPASSWORD='{{ awx_postgres_pass }}' {{ pgdump }} > {{ backup_dir }}/{{ _db_filename }}
       set +e +o pipefail
       echo 'Successful'
       "

roles/restore/tasks/postgres.yml

@@ -80,6 +80,19 @@
       -p {{ awx_postgres_port }}
   no_log: "{{ no_log }}"
 
+- name: Find database file
+  kubernetes.core.k8s_exec:
+    namespace: "{{ backup_pvc_namespace }}"
+    pod: "{{ ansible_operator_meta.name }}-db-management"
+    container: "{{ ansible_operator_meta.name }}-db-management"
+    command: >-
+      bash -c "ls {{ backup_dir }}/tower.db.gz 2>/dev/null || ls {{ backup_dir }}/tower.db 2>/dev/null"
+  register: _db_file_result
+
+- name: Set database filename for restore
+  set_fact:
+    _db_filename: "{{ _db_file_result.stdout | trim | basename }}"
+
 - name: Set pg_restore command
   set_fact:
     pg_restore: >-
@@ -159,7 +172,7 @@
       echo keepalive_pid: $keepalive_pid
       set -e -o pipefail
       {{ pg_drop_create }}
-      cat {{ backup_dir }}/tower.db | PGPASSWORD='{{ awx_postgres_pass }}' {{ pg_restore }}
+      cat {{ backup_dir }}/{{ _db_filename }} | PGPASSWORD='{{ awx_postgres_pass }}' {{ pg_restore }}
       PG_RC=$?
       set +e +o pipefail
       exit $PG_RC