Parametrize async and poll to tune them as per need (#264)

* Updated tasks and default to parametrize async and poll to respect long running backup for AAP2.4+ with private automation hub * Added changelog fragment file --------- Co-authored-by: David Danielsson <[email protected]>
redhat-cop · Jan 10, 2025 · 84df647 · 84df647
1 parent f5482b5
commit 84df647
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 2 deletions.
diff --git a/changelogs/fragments/aap_backup_task.yaml b/changelogs/fragments/aap_backup_task.yaml
@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - aap_backup - Updated tasks and default to parametrize async and poll to respect long running backup for AAP2.4+ with private automation hub
diff --git a/roles/aap_backup/README.md b/roles/aap_backup/README.md
@@ -13,6 +13,8 @@ Available variables are listed below, along with default values defined (see def
 ```yaml
 aap_setup_prep_setup_dir:  # Must be set, though if the aap_setup_prepare role has been run prior, a fact will be set.
 aap_backup_dest: "/root"
+aap_backup_async: 10000
+aap_backup_poll: 20
 ```
 
 ## Example Playbook

diff --git a/roles/aap_backup/defaults/main.yml b/roles/aap_backup/defaults/main.yml
@@ -5,4 +5,6 @@
 
 # Default backup vars, these are default from installer
 aap_backup_dest: /var/tmp
+aap_backup_async: 10000
+aap_backup_poll: 20
 ...
diff --git a/roles/aap_backup/tasks/backup.yml b/roles/aap_backup/tasks/backup.yml
@@ -12,7 +12,7 @@
   ansible.builtin.command: ./setup.sh -e 'backup_dest={{ aap_backup_dest | quote }}' -b
   args:
     chdir: "{{ aap_setup_prep_setup_dir }}"
-  async: 10000
-  poll: 20
+  async: "{{ aap_backup_async }}"
+  poll: "{{ aap_backup_poll }}"
   changed_when: false # these will always run and will always report "changed" otherwise
 ...