Merge pull request #40 from mpagot/cluster_var_val

aws_region is mandatory for AWS cluster

roles/cluster/README.md

@@ -143,6 +143,7 @@ There are no specific variable requirements for GCP.
 
 ### Required Variables for AWS
 
+* aws_region - string - deployment region
 * aws_stonith_tag - string - When creating the AWS, some special tagging is
   required. The tag name created for the cluster must be passed to ansible
   here. See this

roles/cluster/tasks/pre-checks.yml

@@ -258,6 +258,7 @@
     - 'aws_route_table_id'
     - 'aws_access_key_id'
     - 'aws_secret_access_key'
+    - 'aws_region'
   when: platform_aws
 
 # If dual_corosync_rings is true, ensure that eth1 exists

roles/cluster/tasks/tasks_corosync_configuration.yml

@@ -104,7 +104,7 @@
       # to the others
       when: int_var_node_is_primary
 
-    - name: Copy cluster secret
+    - name: Copy cluster secret to other hosts
       ansible.posix.synchronize:
         src: /etc/corosync/authkey
         dest: /etc/corosync/authkey

roles/cluster/tasks/tasks_hana_cluster.yml

@@ -66,7 +66,7 @@
     crm cluster start
   when: int_var_node_is_primary
 
-- name: Ensure the cluster is started and is stable
+- name: Ensure the cluster is started and is stable for first node
   ansible.builtin.command: crm status
   changed_when: false
   register: int_reg_crm_status
@@ -99,7 +99,7 @@
   ansible.builtin.set_fact:
     int_fact_cluster_regex: "{{ ansible_play_hosts | length }} nodes configured"
 
-- name: Ensure the cluster is started and is stable
+- name: Ensure the cluster is started and is stable for second node
   ansible.builtin.command: crm status
   changed_when: false
   register: int_reg_crm_status