Replace ansible.builtin.shell by ansible.builtin.command

redhat-cop · Jan 8, 2024 · 91f82fb · 91f82fb
1 parent 94c1457
commit 91f82fb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/roles/deploy_flask_app/meta/main.yaml b/roles/deploy_flask_app/meta/main.yaml
@@ -1,4 +1,4 @@
 ---
-# dependencies:
-#   - role: cloud.aws_ops.aws_setup_credentials
-# allow_duplicates: true
+dependencies:
+  - role: cloud.aws_ops.aws_setup_credentials
+allow_duplicates: true
diff --git a/roles/deploy_flask_app/tasks/start_containers.yaml b/roles/deploy_flask_app/tasks/start_containers.yaml
@@ -74,10 +74,8 @@
         deploy_flask_app_worker_hostname: "{{ '{{' }} inventory_hostname {{ '}}' }}"
 
     - name: Deploy application into workers
-      ansible.builtin.shell:
-        cmd: >-
-          ansible-playbook
-          -i {{ deploy_flask_app_workers_inventory_file }}
-          {{ deploy_flask_app_workers_playbook_file }}
-          -v
-      changed_when: false
+      ansible.builtin.command: >-
+        ansible-playbook
+        --inventory {{ deploy_flask_app_workers_inventory_file }}
+        {{ deploy_flask_app_workers_playbook_file }}
+        -v