Tweaks

playbooks/nfs_export.yml

@@ -29,9 +29,10 @@
   tasks:
     - name: Grab the template name
       ansible.builtin.set_stats:
+        aggregate: false
+        per_host: false
         data:
           nfs_exports_template_name: "{{ tower_job_template_name | default('NA') }}"
-        aggregate: true
 
     - name: Create IP list and add to NFS exports
       when: not rear_backup_skip | default(false) | bool

playbooks/rear_backup.yml

@@ -6,15 +6,27 @@
   strategy: free
 
   tasks:
-    - name: Import rear_vars role
-      ansible.builtin.import_role:
-        name: rhc.rear.rear_vars
-
     - name: Perform ReaR Backup
       when:
         - ansible_distribution_major_version >= '7'
         - not rear_backup_skip | default(false) | bool
       block:
+        - name: Import rear_vars role
+          ansible.builtin.import_role:
+            name: rhc.rear.rear_vars
+
+        - name: Fetch the template name from NFS exports
+          ansible.builtin.set_fact:
+            job_template_name: "{{ nfs_exports_template_name }}"
+
         - name: Include the backup role
           ansible.builtin.include_role:
             name: rhc.rear.rear_backup
+      rescue:
+        - name: Clean up on ReaR failure
+          ansible.builtin.include_role:
+            name: rhc.rear.rear_remove
+
+        - name: Fail and print message
+          ansible.builtin.fail:
+            msg: "ReaR backup failed. All changes have been rolled back. Please retry."

playbooks/rear_vars.yml

@@ -6,13 +6,13 @@
   strategy: free
 
   tasks:
-    - name: Include rear_vars
+    - name: Import rear_vars role
       ansible.builtin.import_role:
-        name: ../roles/rear_vars
+        name: rhc.rear.rear_vars
 
     - name: Debug rear_vars_nfs_share
       ansible.builtin.debug:
-        msg: "{{ rear_vars_nfs_base }}/{{ rear_backup_workflow }} ({{ rear_vars_nfs_share[rear_backup_workflow] }})"
+        msg: "{{ rear_vars_nfs_base }}/{{ rear_backup_workflow }} ({{ rear_vars_nfs_fsid[rear_backup_workflow] }})"
 
     - name: Debug rear_vars_grub_label
       ansible.builtin.debug:

roles/nfs_server_define/tasks/main.yml

@@ -1,9 +1,13 @@
 ---
+- name: Debug job_template_name
+  ansible.builtin.debug:
+    msg: "We'd pick Non-Prod/Prod NFS servers from here based on the selected job template name: {{ job_template_name }}."
+
 - name: Check connectivity to NFS servers
   ansible.builtin.wait_for:
     host: "{{ item }}"
     port: 2049
-    timeout: 30
+    timeout: 5 # default: 30
   loop: "{{ rear_vars_nfs_np }}"
   ignore_errors: true
   register: active_nfs_servers

roles/rear_backup/tasks/main.yml

@@ -40,10 +40,6 @@
     - name: Write backup success file
       ansible.builtin.include_tasks:
         file: create_success_file.yml
-
-    - name: ReaR backup success
-      ansible.builtin.debug:
-        msg: "ReaR backup has completed successfully."
   rescue:
     - name: ReaR backup failed
       ansible.builtin.fail:
@@ -63,3 +59,7 @@
     mode: "0600"
   when:
     - rear_config_stat['stat']['exists'] | bool
+
+- name: ReaR backup success
+  ansible.builtin.debug:
+    msg: "ReaR backup has completed successfully."

roles/rear_local_cfg/templates/local.conf.j2

@@ -6,7 +6,7 @@ OUTPUT=ISO
 OUTPUT_URL=nfs://{{ nfs_srv }}{{ rear_vars_nfs_base }}/{{ rear_backup_workflow }}
 BACKUP=NETFS
 BACKUP_URL=nfs://{{ nfs_srv }}{{ rear_vars_nfs_base }}/{{ rear_backup_workflow }}
-# SSH_ROOT_PASSWORD="bob"
+# SSH_ROOT_PASSWORD="foo"
 USE_STATIC_NETWORKING=yes
 CLONES_ALL_USERS_GROUPS=yes
 SSH_FILES=yes

roles/rear_vars/defaults/main.yml

@@ -8,7 +8,7 @@ rear_backup_workflow: standalone
 
 # Used by:
 #   - rear_restore
-rear_restore_tmpdir: /tmp/rear_restore
+rear_restore_tmpdir: /var/tmp/rear_restore
 
 # Used by:
 #   - rear_backup