Revert "Import KV secrets in SCS HA Installation Playbook (#518)" (#519)

This reverts commit 3f54e88.
Azure · Dec 19, 2023 · 98d0a61 · 98d0a61
1 parent 3f54e88
commit 98d0a61
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/deploy/ansible/playbook_05_00_00_sap_scs_install.yaml b/deploy/ansible/playbook_05_00_00_sap_scs_install.yaml
@@ -70,6 +70,15 @@
                                             - scs_high_availability or database_high_availability
       tags:
                                             - 0.2-wincluster-witness
+# MKD - These two tasks (Above & Below) seem to achieve the same thing?
+    - name:                                 "SCS Installation Playbook: - Read/Create key vault secrets"
+      ansible.builtin.include_role:
+        name:                               roles-misc/0.2-kv-secrets
+        public:                             true
+      vars:
+        operation:                          fencing
+      tags:
+                                            - kv-secrets
 
     - name:                            "SCS Installation Playbook: - Read storage account details"
       ansible.builtin.include_role:
@@ -434,15 +443,6 @@
           tags:
                                                 - always
 
-        - name:                                 "SCS HA Installation Playbook: - Read/Create key vault secrets"
-          ansible.builtin.include_role:
-            name:                               roles-misc/0.2-kv-secrets
-            public:                             true
-          vars:
-            operation:                          fencing
-          tags:
-                                                - kv-secrets
-
         - name:                                 "SCS HA Installation Playbook: - Initialize facts... Fencing"
           ansible.builtin.set_fact:
             fencing_spn_client_id:              "{% if not use_msi_for_clusters %}{{ hostvars.localhost.sap_fencing_spn_client_id }}{% endif %}"