From 022dd4b4680888ddb6e3745b39fc9c4706f3aac1 Mon Sep 17 00:00:00 2001 From: John Mulligan Date: Sat, 21 Oct 2023 16:35:32 -0400 Subject: [PATCH] cephadm: add test assertion for snmp env file option 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 | 1 + 1 file changed, 1 insertion(+) diff --git a/src/cephadm/tests/test_deploy.py b/src/cephadm/tests/test_deploy.py index d7712beb85f..098bdfc3f1d 100644 --- a/src/cephadm/tests/test_deploy.py +++ b/src/cephadm/tests/test_deploy.py @@ -97,6 +97,7 @@ def test_deploy_snmp_container(cephadm_fs, monkeypatch): 'quay.io/aaabbb/snmp:latest --web.listen-address=:9464 --snmp.destination=192.168.100.10:8899 --snmp.version=V2c --log.level=info --snmp.trap-description-template=/etc/snmp_notifier/description-template.tpl' ) assert '--pids-limit' not in runfile_lines[-1] + assert f'--env-file=/var/lib/ceph/{fsid}/snmp-gateway.sunmop/snmp-gateway.conf' in runfile_lines[-1] _firewalld().open_ports.assert_not_called() basedir = pathlib.Path(f'/var/lib/ceph/{fsid}/snmp-gateway.sunmop') assert basedir.is_dir() -- 2.39.5