Skip to content

Commit 824dc47

Browse files
lotfihamidAlex-Welsh
authored andcommitted
Fix trove_api healthcheck with TLS backend
When trove_enable_tls_backend is enabled, the API healthcheck must use https scheme instead of plain http. This change updates the healthcheck command accordingly. Change-Id: I5782a5da30434cf5c5ae0fcb60b3c6f178351fb7 Closes-Bug: #2136977 Signed-off-by: hamidlotfi <[email protected]>
1 parent dcd0b5b commit 824dc47

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ansible/roles/trove/defaults/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ trove_api_enable_healthchecks: "{{ enable_container_healthchecks }}"
101101
trove_api_healthcheck_interval: "{{ default_container_healthcheck_interval }}"
102102
trove_api_healthcheck_retries: "{{ default_container_healthcheck_retries }}"
103103
trove_api_healthcheck_start_period: "{{ default_container_healthcheck_start_period }}"
104-
trove_api_healthcheck_test: ["CMD-SHELL", "healthcheck_curl http://{{ api_interface_address | put_address_in_context('url') }}:{{ trove_api_port }}"]
104+
trove_api_healthcheck_test: ["CMD-SHELL", "healthcheck_curl {{ 'https' if trove_enable_tls_backend | bool else 'http' }}://{{ api_interface_address | put_address_in_context('url') }}:{{ trove_api_port }}"]
105105
trove_api_healthcheck_timeout: "{{ default_container_healthcheck_timeout }}"
106106
trove_api_healthcheck:
107107
interval: "{{ trove_api_healthcheck_interval }}"

0 commit comments

Comments
 (0)