Skip to content

Commit c389f82

Browse files
authored
Merge pull request ceph#53840 from rhcs-dashboard/add-health-check-haproxy
mgr/cephadm is not defining haproxy tcp healthchecks for Ganesha Reviewed-by: Adam King <[email protected]>
2 parents 7114dc6 + 6a6a9dd commit c389f82

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/pybind/mgr/cephadm/templates/services/ingress/haproxy.cfg.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,6 @@ backend backend
8585
default-server {{ default_server_opts|join(" ") }}
8686
{% endif %}
8787
{% for server in servers %}
88-
server {{ server.name }} {{ server.ip }}:{{ server.port }}
88+
server {{ server.name }} {{ server.ip }}:{{ server.port }} check
8989
{% endfor %}
9090
{% endif %}

src/pybind/mgr/cephadm/tests/test_services.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1650,7 +1650,7 @@ def fake_get_addr(hostname: str) -> str:
16501650
)
16511651
if enable_haproxy_protocol:
16521652
haproxy_txt += ' default-server send-proxy-v2\n'
1653-
haproxy_txt += ' server nfs.foo.0 192.168.122.111:12049\n'
1653+
haproxy_txt += ' server nfs.foo.0 192.168.122.111:12049 check\n'
16541654
haproxy_expected_conf = {
16551655
'files': {'haproxy.cfg': haproxy_txt}
16561656
}
@@ -2428,7 +2428,7 @@ def fake_keys():
24282428
' balance source\n'
24292429
' hash-type consistent\n'
24302430
' default-server send-proxy-v2\n'
2431-
' server nfs.foo.0 192.168.122.111:12049\n'
2431+
' server nfs.foo.0 192.168.122.111:12049 check\n'
24322432
)
24332433
haproxy_expected_conf = {
24342434
'files': {'haproxy.cfg': haproxy_txt}

0 commit comments

Comments
 (0)