Skip to content

Commit

Permalink
Import KV secrets in SCS HA Installation Playbook (#518)
Browse files Browse the repository at this point in the history
"The task includes an option with an undefined variable. The error was: 'ansible.vars.hostvars.HostVarsVars object' has no attribute 'sap_fencing_spn_client_id'.
  • Loading branch information
hjstam authored Dec 19, 2023
1 parent 51afc15 commit 3f54e88
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions deploy/ansible/playbook_05_00_00_sap_scs_install.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,15 +70,6 @@
- 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:
Expand Down Expand Up @@ -443,6 +434,15 @@
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 %}"
Expand Down

0 comments on commit 3f54e88

Please sign in to comment.