From 351326fe9eda6ae0da89a6fbb5fa1cc66434f430 Mon Sep 17 00:00:00 2001 From: John Mulligan Date: Sat, 21 Oct 2023 16:16:32 -0400 Subject: [PATCH] cephadm: add test assertion for haproxy options, mounts Part of a series of commits to increase coverage of deployment path features with regards to container engine options, env vars and mounts. This will serve future refactoring efforts. Signed-off-by: John Mulligan --- src/cephadm/tests/test_deploy.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/cephadm/tests/test_deploy.py b/src/cephadm/tests/test_deploy.py index c45e05166f6..94f133e8db9 100644 --- a/src/cephadm/tests/test_deploy.py +++ b/src/cephadm/tests/test_deploy.py @@ -179,6 +179,8 @@ def test_deploy_haproxy_container(cephadm_fs, monkeypatch): 'quay.io/lfeuwbo/haproxy:latest haproxy -f /var/lib/haproxy/haproxy.cfg' ) assert '--pids-limit' not in runfile_lines[-1] + assert '--user=root' in runfile_lines[-1] + assert f'-v {basedir}/haproxy:/var/lib/haproxy' in runfile_lines[-1] _firewalld().open_ports.assert_not_called() assert not (basedir / 'config').exists() assert not (basedir / 'keyring').exists() -- 2.39.5