fix: flag for ibm power

Author: sean-freeman

all/ansible_playbooks_for_sap/create_ansible_extravars.tf

@@ -38,5 +38,8 @@ sap_vm_provision_ssh_host_private_key_file_path: "${abspath(path.root)}/tmp/host
 ${ strcontains(var.module_var_ansible_sap_scenario_selection, "hana") ? "# Ansible Playbooks for SAP - SAP HANA variables when multiple hosts (HA or Scale-Out)" : "" }
 ${ strcontains(var.module_var_ansible_sap_scenario_selection, "hana") ? "sap_hana_install_update_firewall: true" : "" }
 
+${ var.module_var_ibmpower_flag ? "# Ansible Playbooks for SAP - IBM Power" : "" }
+${ var.module_var_ibmpower_flag ? "sap_storage_setup_multipath_enable_and_detect: true" : "" }
+
 EOF
 }

all/ansible_playbooks_for_sap/module_variables.tf

@@ -46,3 +46,7 @@ variable "module_var_ansible_sap_system_nwas_java_scs_instance_nr" {}
 variable "module_var_ansible_sap_system_nwas_java_ci_instance_nr" {}
 variable "module_var_ansible_sap_maintenance_planner_transaction_name" {}
 variable "module_var_ansible_sap_software_download_directory" {}
+
+variable "module_var_ibmpower_flag" {
+  default = false
+}

all/ansible_playbooks_for_sap/note_dev.md

@@ -0,0 +1,43 @@
+# CPU Architecture detection
+
+Added flag for IBM Power. With the homogenised approach leveraging 'existing hosts' created by Terraform and execution of the Ansible Playbooks for SAP, there is no Terraform Variable that indicates the hosts are IBM Power.
+
+
+# Alternative data conversion
+
+Data conversion from Terraform Map of host specifications, to a Python (Ansible) Dictionary can be achieved instead using Jinja:
+
+```yaml
+    # For end user ease of use, the host specifications dictionary uses disk_count to indicate how many disks will be provisioned
+    # However the sap_storage_setup Ansible Role can not detect disk_count, and requires the key to be renamed lvm_lv_stripes
+    - name: Convert sap_vm_provision_*_host_specifications_dictionary.storage_definition to sap_storage_setup.sap_storage_setup_definition
+      ansible.builtin.set_fact:
+        sap_storage_setup_definition: "{{ sap_storage_setup_definition | default([]) + [converted_element] }}"
+      vars:
+        converted_element: |
+          {% set current_element = (convert_item | dict2items) %}
+          {% set new_element = [] %}
+          {% for entry in current_element %}
+            {%- if "disk_count" in entry.key %}
+              {%- set conv = new_element.extend([
+                {
+                  'key': 'lvm_lv_stripes',
+                  'value': entry.value,
+                }
+              ]) %}
+            {%- elif not "disk_type" in entry.key %}
+              {%- set add_entry = new_element.extend([
+                {
+                  'key': entry.key,
+                  'value': entry.value,
+                }
+              ]) %}
+            {%- endif -%}
+          {% endfor %}
+          {{ new_element | items2dict }}
+      loop: "{{ terraform_host_specification_storage_definition | list }}"
+      # loop: "{{ terraform_host_specification[inventory_hostname]['storage_definition'] | list }}"
+      loop_control:
+        loop_var: convert_item
+        label: "{{ convert_item.name }}"
+```