Skip to content

Commit 2bc29b5

Browse files
committed
Enable automatic execution for HA SCS test cases and add ENSA version check in kill-message-server.yml
1 parent b18212d commit 2bc29b5

2 files changed

Lines changed: 31 additions & 8 deletions

File tree

src/roles/ha_scs/tasks/kill-message-server.yml

Lines changed: 30 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,26 @@
3333
test_execution_start: "{{ now(utc=true, fmt='%Y-%m-%d %H:%M:%S') }}"
3434
test_execution_hostname: "{{ hostvars[cluster_status_pre.ascs_node].ansible_hostname }}"
3535

36+
- name: "Test Execution: Check for ENSA version"
37+
ansible.builtin.shell: pgrep -f 'enq.sap{{ sap_sid | upper }}' | wc -l
38+
register: ensa2_check
39+
failed_when: false
40+
3641
- name: "Test Execution: Kill Message Server Process"
37-
ansible.builtin.shell: pgrep -f "ms.sap{{ sap_sid | upper }}" | xargs kill -9
42+
ansible.builtin.shell: |
43+
while true; do
44+
pgrep -f 'ms.sap{{ sap_sid | upper }}' | xargs kill -9 || break
45+
done
3846
register: kill_ms_result
39-
failed_when: false
47+
failed_when: "'kill: not enough arguments' in kill_ms_result.stderr"
48+
ignore_errors: true
4049

4150
- name: "Test Execution: Wait for Failover"
4251
ansible.builtin.pause:
43-
seconds: 60
52+
seconds: 30
4453

45-
- name: "Test Execution: Validate SCS cluster status"
54+
- name: "Test Execution: Validate SCS cluster status for ENSA1"
55+
when: ensa2_check.stdout == "0"
4656
get_cluster_status_scs:
4757
sap_sid: "{{ sap_sid | lower }}"
4858
ansible_os_family: "{{ ansible_os_family | upper }}"
@@ -54,6 +64,19 @@
5464
cluster_status_test_execution.ascs_node == cluster_status_pre.ers_node
5565
and cluster_status_test_execution.ers_node == cluster_status_pre.ascs_node
5666
67+
- name: "Test Execution: Validate SCS cluster status for ENSA2"
68+
when: ensa2_check.stdout != "0"
69+
get_cluster_status_scs:
70+
sap_sid: "{{ sap_sid | lower }}"
71+
ansible_os_family: "{{ ansible_os_family | upper }}"
72+
register: cluster_status_test_execution
73+
retries: 50
74+
delay: 10
75+
failed_when: false
76+
until: |
77+
cluster_status_test_execution.ascs_node != ""
78+
and cluster_status_test_execution.ers_node != ""
79+
5780
- name: "Test Execution: Cleanup resources"
5881
ansible.builtin.command: crm_resource --cleanup
5982
register: cleanup_failed_resource_test_execution
@@ -67,15 +90,15 @@
6790
ansible.builtin.set_fact:
6891
test_case_message_from_test_case: |
6992
Old ASCS: {{ cluster_status_pre.ascs_node }}
70-
New ASCS: {{ hostvars[cluster_status_pre.ascs_node].cluster_status_test_execution.ascs_node }}
93+
New ASCS: {{ cluster_status_test_execution.ascs_node }}
7194
Old ERS: {{ cluster_status_pre.ers_node }}
72-
New ERS: {{ hostvars[cluster_status_pre.ascs_node].cluster_status_test_execution.ers_node }}
95+
New ERS: {{ cluster_status_test_execution.ers_node }}
7396
test_case_details_from_test_case: {
7497
"Pre Validations: Validate SCS cluster status": "{{ cluster_status_pre }}",
7598
"Pre Validations: CleanUp any failed resource": "{{ cleanup_failed_resource_pre }}",
7699
"Test Execution: Kill Message Server Process": "{{ kill_ms_result }}",
77100
"Test Execution: Cleanup resources": "{{ cleanup_failed_resource_test_execution }}",
78-
"Post Validations Result": "{{ hostvars[cluster_status_pre.ers_node].cluster_status_test_execution }}",
101+
"Post Validations Result": "{{ cluster_status_test_execution }}",
79102
}
80103
# /*---------------------------------------------------------------------------
81104
# | Post Validations |

src/vars/input-api.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ test_groups:
177177
ASCS node by forcefully terminating it using the kill -9 signal. It verifies proper cluster
178178
reaction, automatic failover to the ERS node, and ensures service continuity after
179179
the process failure.
180-
enabled: false
180+
enabled: true
181181

182182
- name: "Kill Enqueue Server Process"
183183
task_name: kill-enqueue-server

0 commit comments

Comments
 (0)