From 693c5c8b2354688663f4f38a97b6ccb3494a8af4 Mon Sep 17 00:00:00 2001
From: Pierre Riteau <pierre@stackhpc.com>
Date: Tue, 12 Dec 2023 18:52:57 +0100
Subject: [PATCH] Fix Docker health check for sahara_engine

The wrong process name was being used.

Closes-Bug: #2046268
Change-Id: I5a5d4f227205e811732331ee6e020ccea67b6fab
---
 ansible/roles/sahara/defaults/main.yml                       | 2 +-
 .../fix-sahara-engine-healthcheck-cfc635de9e4f56bc.yaml      | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)
 create mode 100644 releasenotes/notes/fix-sahara-engine-healthcheck-cfc635de9e4f56bc.yaml

diff --git a/ansible/roles/sahara/defaults/main.yml b/ansible/roles/sahara/defaults/main.yml
index c22a6a16b6..5c74b42295 100644
--- a/ansible/roles/sahara/defaults/main.yml
+++ b/ansible/roles/sahara/defaults/main.yml
@@ -93,7 +93,7 @@ sahara_engine_enable_healthchecks: "{{ enable_container_healthchecks }}"
 sahara_engine_healthcheck_interval: "{{ default_container_healthcheck_interval }}"
 sahara_engine_healthcheck_retries: "{{ default_container_healthcheck_retries }}"
 sahara_engine_healthcheck_start_period: "{{ default_container_healthcheck_start_period }}"
-sahara_engine_healthcheck_test: ["CMD-SHELL", "healthcheck_port sahara_engine {{ om_rpc_port }}"]
+sahara_engine_healthcheck_test: ["CMD-SHELL", "healthcheck_port sahara-engine {{ om_rpc_port }}"]
 sahara_engine_healthcheck_timeout: "{{ default_container_healthcheck_timeout }}"
 sahara_engine_healthcheck:
   interval: "{{ sahara_engine_healthcheck_interval }}"
diff --git a/releasenotes/notes/fix-sahara-engine-healthcheck-cfc635de9e4f56bc.yaml b/releasenotes/notes/fix-sahara-engine-healthcheck-cfc635de9e4f56bc.yaml
new file mode 100644
index 0000000000..5e130ad24e
--- /dev/null
+++ b/releasenotes/notes/fix-sahara-engine-healthcheck-cfc635de9e4f56bc.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Fixes Docker health check for the ``sahara_engine`` container.
+    `LP#2046268 <https://bugs.launchpad.net/kolla-ansible/+bug/2046268>`__
-- 
GitLab