From d2d91d9ec1d89bdd771f98c262fef5a657b9918a Mon Sep 17 00:00:00 2001
From: LinPeiWen <591171850@qq.com>
Date: Wed, 3 Mar 2021 11:21:13 +0000
Subject: [PATCH] Use Docker healthchecks for senlin services

This change enables the use of Docker healthchecks for senlin services.
Implements: blueprint container-health-check

Change-Id: I148a0174c46244241580265d0fedc0fb1b92602c
---
 ansible/roles/senlin/defaults/main.yml        | 56 +++++++++++++++++++
 ansible/roles/senlin/handlers/main.yml        |  4 ++
 .../roles/senlin/tasks/check-containers.yml   |  1 +
 ...lthchecks-for-senlin-c46a3b39a846c597.yaml |  6 ++
 4 files changed, 67 insertions(+)
 create mode 100644 releasenotes/notes/implement-docker-healthchecks-for-senlin-c46a3b39a846c597.yaml

diff --git a/ansible/roles/senlin/defaults/main.yml b/ansible/roles/senlin/defaults/main.yml
index d5dbcc14e0..7ef7dcb083 100644
--- a/ansible/roles/senlin/defaults/main.yml
+++ b/ansible/roles/senlin/defaults/main.yml
@@ -9,6 +9,7 @@ senlin_services:
     image: "{{ senlin_api_image_full }}"
     volumes: "{{ senlin_api_default_volumes + senlin_api_extra_volumes }}"
     dimensions: "{{ senlin_api_dimensions }}"
+    healthcheck: "{{ senlin_api_healthcheck }}"
     haproxy:
       senlin_api:
         enabled: "{{ enable_senlin }}"
@@ -29,6 +30,7 @@ senlin_services:
     image: "{{ senlin_conductor_image_full }}"
     volumes: "{{ senlin_conductor_default_volumes + senlin_conductor_extra_volumes }}"
     dimensions: "{{ senlin_conductor_dimensions }}"
+    healthcheck: "{{ senlin_conductor_healthcheck }}"
   senlin-engine:
     container_name: senlin_engine
     group: senlin-engine
@@ -36,6 +38,7 @@ senlin_services:
     image: "{{ senlin_engine_image_full }}"
     volumes: "{{ senlin_engine_default_volumes + senlin_engine_extra_volumes }}"
     dimensions: "{{ senlin_engine_dimensions }}"
+    healthcheck: "{{ senlin_engine_healthcheck }}"
   senlin-health-manager:
     container_name: senlin_health_manager
     group: senlin-health-manager
@@ -43,6 +46,7 @@ senlin_services:
     image: "{{ senlin_health_manager_image_full }}"
     volumes: "{{ senlin_health_manager_default_volumes + senlin_health_manager_extra_volumes }}"
     dimensions: "{{ senlin_health_manager_dimensions }}"
+    healthcheck: "{{ senlin_health_manager_healthcheck }}"
 
 ####################
 # Database
@@ -79,6 +83,58 @@ senlin_conductor_dimensions: "{{ default_container_dimensions }}"
 senlin_engine_dimensions: "{{ default_container_dimensions }}"
 senlin_health_manager_dimensions: "{{ default_container_dimensions }}"
 
+senlin_api_enable_healthchecks: "{{ enable_container_healthchecks }}"
+senlin_api_healthcheck_interval: "{{ default_container_healthcheck_interval }}"
+senlin_api_healthcheck_retries: "{{ default_container_healthcheck_retries }}"
+senlin_api_healthcheck_start_period: "{{ default_container_healthcheck_start_period }}"
+senlin_api_healthcheck_test: ["CMD-SHELL", "healthcheck_curl http://{{ api_interface_address |  put_address_in_context('url') }}:{{ senlin_api_listen_port }}"]
+senlin_api_healthcheck_timeout: "{{ default_container_healthcheck_timeout }}"
+senlin_api_healthcheck:
+  interval: "{{ senlin_api_healthcheck_interval }}"
+  retries: "{{ senlin_api_healthcheck_retries }}"
+  start_period: "{{ senlin_api_healthcheck_start_period }}"
+  test: "{% if senlin_api_enable_healthchecks | bool %}{{ senlin_api_healthcheck_test }}{% else %}NONE{% endif %}"
+  timeout: "{{ senlin_api_healthcheck_timeout }}"
+
+senlin_conductor_enable_healthchecks: "{{ enable_container_healthchecks }}"
+senlin_conductor_healthcheck_interval: "{{ default_container_healthcheck_interval }}"
+senlin_conductor_healthcheck_retries: "{{ default_container_healthcheck_retries }}"
+senlin_conductor_healthcheck_start_period: "{{ default_container_healthcheck_start_period }}"
+senlin_conductor_healthcheck_test: ["CMD-SHELL", "healthcheck_port senlin-conductor {{ om_rpc_port }}"]
+senlin_conductor_healthcheck_timeout: "{{ default_container_healthcheck_timeout }}"
+senlin_conductor_healthcheck:
+  interval: "{{ senlin_conductor_healthcheck_interval }}"
+  retries: "{{ senlin_conductor_healthcheck_retries }}"
+  start_period: "{{ senlin_conductor_healthcheck_start_period }}"
+  test: "{% if senlin_conductor_enable_healthchecks | bool %}{{ senlin_conductor_healthcheck_test }}{% else %}NONE{% endif %}"
+  timeout: "{{ senlin_conductor_healthcheck_timeout }}"
+
+senlin_engine_enable_healthchecks: "{{ enable_container_healthchecks }}"
+senlin_engine_healthcheck_interval: "{{ default_container_healthcheck_interval }}"
+senlin_engine_healthcheck_retries: "{{ default_container_healthcheck_retries }}"
+senlin_engine_healthcheck_start_period: "{{ default_container_healthcheck_start_period }}"
+senlin_engine_healthcheck_test: ["CMD-SHELL", "healthcheck_port senlin-engine {{ om_rpc_port }}"]
+senlin_engine_healthcheck_timeout: "{{ default_container_healthcheck_timeout }}"
+senlin_engine_healthcheck:
+  interval: "{{ senlin_engine_healthcheck_interval }}"
+  retries: "{{ senlin_engine_healthcheck_retries }}"
+  start_period: "{{ senlin_engine_healthcheck_start_period }}"
+  test: "{% if senlin_engine_enable_healthchecks | bool %}{{ senlin_engine_healthcheck_test }}{% else %}NONE{% endif %}"
+  timeout: "{{ senlin_engine_healthcheck_timeout }}"
+
+senlin_health_manager_enable_healthchecks: "{{ enable_container_healthchecks }}"
+senlin_health_manager_healthcheck_interval: "{{ default_container_healthcheck_interval }}"
+senlin_health_manager_healthcheck_retries: "{{ default_container_healthcheck_retries }}"
+senlin_health_manager_healthcheck_start_period: "{{ default_container_healthcheck_start_period }}"
+senlin_health_manager_healthcheck_test: ["CMD-SHELL", "healthcheck_port senlin-health-manager {{ om_rpc_port }}"]
+senlin_health_manager_healthcheck_timeout: "{{ default_container_healthcheck_timeout }}"
+senlin_health_manager_healthcheck:
+  interval: "{{ senlin_health_manager_healthcheck_interval }}"
+  retries: "{{ senlin_health_manager_healthcheck_retries }}"
+  start_period: "{{ senlin_health_manager_healthcheck_start_period }}"
+  test: "{% if senlin_health_manager_enable_healthchecks | bool %}{{ senlin_health_manager_healthcheck_test }}{% else %}NONE{% endif %}"
+  timeout: "{{ senlin_health_manager_healthcheck_timeout }}"
+
 senlin_api_default_volumes:
   - "{{ node_config_directory }}/senlin-api/:{{ container_config_directory }}/:ro"
   - "/etc/localtime:/etc/localtime:ro"
diff --git a/ansible/roles/senlin/handlers/main.yml b/ansible/roles/senlin/handlers/main.yml
index 51fbe4604e..be8ba86113 100644
--- a/ansible/roles/senlin/handlers/main.yml
+++ b/ansible/roles/senlin/handlers/main.yml
@@ -11,6 +11,7 @@
     image: "{{ service.image }}"
     volumes: "{{ service.volumes|reject('equalto', '')|list }}"
     dimensions: "{{ service.dimensions }}"
+    healthcheck: "{{ service.healthcheck | default(omit) }}"
   when:
     - kolla_action != "config"
 
@@ -26,6 +27,7 @@
     image: "{{ service.image }}"
     volumes: "{{ service.volumes|reject('equalto', '')|list }}"
     dimensions: "{{ service.dimensions }}"
+    healthcheck: "{{ service.healthcheck | default(omit) }}"
   when:
     - kolla_action != "config"
 
@@ -41,6 +43,7 @@
     image: "{{ service.image }}"
     volumes: "{{ service.volumes|reject('equalto', '')|list }}"
     dimensions: "{{ service.dimensions }}"
+    healthcheck: "{{ service.healthcheck | default(omit) }}"
   when:
     - kolla_action != "config"
 
@@ -56,5 +59,6 @@
     image: "{{ service.image }}"
     volumes: "{{ service.volumes|reject('equalto', '')|list }}"
     dimensions: "{{ service.dimensions }}"
+    healthcheck: "{{ service.healthcheck | default(omit) }}"
   when:
     - kolla_action != "config"
diff --git a/ansible/roles/senlin/tasks/check-containers.yml b/ansible/roles/senlin/tasks/check-containers.yml
index fa59e48de5..7596400ede 100644
--- a/ansible/roles/senlin/tasks/check-containers.yml
+++ b/ansible/roles/senlin/tasks/check-containers.yml
@@ -8,6 +8,7 @@
     image: "{{ item.value.image }}"
     volumes: "{{ item.value.volumes|reject('equalto', '')|list }}"
     dimensions: "{{ item.value.dimensions }}"
+    healthcheck: "{{ item.value.healthcheck | default(omit) }}"
   when:
     - inventory_hostname in groups[item.value.group]
     - item.value.enabled | bool
diff --git a/releasenotes/notes/implement-docker-healthchecks-for-senlin-c46a3b39a846c597.yaml b/releasenotes/notes/implement-docker-healthchecks-for-senlin-c46a3b39a846c597.yaml
new file mode 100644
index 0000000000..cb7bcb5280
--- /dev/null
+++ b/releasenotes/notes/implement-docker-healthchecks-for-senlin-c46a3b39a846c597.yaml
@@ -0,0 +1,6 @@
+---
+features:
+  - |
+    Implements container healthchecks for senlin services.
+    See `blueprint
+    <https://blueprints.launchpad.net/kolla-ansible/+spec/container-health-check>`__