From e6c44ef2597cf936a85728d2796e1e378bb8f46f Mon Sep 17 00:00:00 2001 From: John Mulligan Date: Sat, 21 Oct 2023 16:32:15 -0400 Subject: [PATCH] cephadm: add assertions for monitoring 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 098bdfc3f1d..98a2b2a6d1f 100644 --- a/src/cephadm/tests/test_deploy.py +++ b/src/cephadm/tests/test_deploy.py @@ -302,6 +302,8 @@ def test_deploy_a_monitoring_container(cephadm_fs, monkeypatch): assert runfile_lines[-1].endswith( 'quay.io/titans/prometheus:latest --config.file=/etc/prometheus/prometheus.yml --storage.tsdb.path=/prometheus --web.listen-address=:9095 --storage.tsdb.retention.time=15d --storage.tsdb.retention.size=0 --web.external-url=http://10.10.10.10:9095' ) + assert '--user 8765' in runfile_lines[-1] + assert f'-v /var/lib/ceph/{fsid}/prometheus.fire/etc/prometheus:/etc/prometheus:Z' in runfile_lines[-1] _firewalld().open_ports.assert_not_called() assert not (basedir / 'config').exists() assert not (basedir / 'keyring').exists() -- 2.39.5