Revert "Avoid risky pipes. Increase retires to handle clusters with single frozen or cold datanode"

This reverts commit d83d38f.

Author: ivareri

roles/elasticsearch/tasks/elasticsearch-rolling-start.yml

@@ -36,21 +36,23 @@
     port: "{{ elasticstack_elasticsearch_http_port }}"
     delay: 30
 
-- name: Confirm the node joins the cluster
-  ansible.builtin.uri:
-    url: "{{ elasticsearch_http_protocol }}://{{ elasticsearch_api_host }}:{{ elasticstack_elasticsearch_http_port }}/_cat/nodes?h=name&format=json"
-    method: GET
-    user: elastic
-    password: "{{ elasticstack_password.stdout }}"
-    validate_certs: no
-  register: response
-  until: node_found | bool
+- name: Confirm the node joins the cluster # noqa: risky-shell-pipe
+  ansible.builtin.shell: >
+    if test -n "$(ps -p $$ | grep bash)"; then set -o pipefail; fi;
+    curl
+    -k
+    -u elastic:{{ elasticstack_password.stdout }}
+    -s
+    -m 2
+    '{{ elasticsearch_http_protocol }}://{{ elasticsearch_api_host }}:{{ elasticstack_elasticsearch_http_port }}/_cat/nodes?h=name'
+    | grep
+    -E
+    '^{{ elasticsearch_nodename }}$'
+  register: result
+  until: result.rc == 0
   retries: 200
   delay: 3
   changed_when: false
-  vars:
-    node_found: "{{ response.json | json_query(node_query) | length > 0 }}"
-    node_query: "[?name=='{{ elasticsearch_nodename }}']"
 
 # Don't continue the play unless cluster health is OK
 - name: Cluster health check