Bugfix kv-secrets debug and import tasks (#516)

Because the operation variable has no default within the rol its existence should be checked.
Azure · Dec 15, 2023 · 51afc15 · 51afc15
1 parent 6c7d903
commit 51afc15
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/deploy/ansible/roles-misc/0.2-kv-secrets/tasks/main.yaml b/deploy/ansible/roles-misc/0.2-kv-secrets/tasks/main.yaml
@@ -32,14 +32,17 @@
                                             - "use_msi_for_clusters:            {{ use_msi_for_clusters }}"
                                             - "platform:                        {{ platform | upper }}"
     verbosity:                              2
-  when:                                   operation == "fencing"
+  when:
+                                          - operation is defined
+                                          - operation == "fencing"
 
 # -------------------------------------+---------------------------------------8
 #
 # -------------------------------------+---------------------------------------8
 - name:                                   "0.2 Key Vault: - Import S User tasks"
   ansible.builtin.import_tasks:           "s_user.yaml"
   when:
+                                          - operation is defined
                                           - operation == "SoftwareAcquisition"
 
 
@@ -49,6 +52,7 @@
 - name:                                   "0.2 Key Vault: - Import Fencing secrets"
   ansible.builtin.import_tasks:           "fencing.yaml"
   when:
+                                          - operation is defined
                                           - operation == "fencing"
                                           - (database_high_availability and database_cluster_type == "AFA") or
                                             (scs_high_availability      and      scs_cluster_type == "AFA")         # AFA (Azure Fencing Agent)
@@ -62,6 +66,7 @@
   ansible.builtin.import_tasks:           "wincluster-witness.yaml"
   # TODO: update when clause more appropriately
   when:
+                                          - operation is defined
                                           - operation == "fencing"
                                           - (scs_high_availability or database_high_availability)
                                           - not use_msi_for_clusters