Skip to content

Commit 3de18c8

Browse files
authored
Merge pull request ceph#59694 from rkachach/fix_issue_67975
mgr/cephadm: open ceph-exporter when firewalld is enabled Reviewed-by: Adam King <[email protected]>
2 parents 74139ac + fd93ecd commit 3de18c8

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -716,10 +716,10 @@ def test_ceph_exporter_config_security_enabled(self, _get_fqdn, _run_cephadm, ce
716716
"name": "ceph-exporter.test",
717717
"image": "",
718718
"deploy_arguments": [],
719-
"params": {},
719+
"params": {"tcp_ports": [9926]},
720720
"meta": {
721721
"service_name": "ceph-exporter",
722-
"ports": [],
722+
"ports": [9926],
723723
"ip": None,
724724
"deployed_by": [],
725725
"rank": None,

src/python-common/ceph/deployment/service_spec.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2839,6 +2839,9 @@ def __init__(self,
28392839
self.prio_limit = prio_limit
28402840
self.stats_period = stats_period
28412841

2842+
def get_port_start(self) -> List[int]:
2843+
return [self.port or 9926]
2844+
28422845
def validate(self) -> None:
28432846
super(CephExporterSpec, self).validate()
28442847

0 commit comments

Comments
 (0)