Skip to content

Commit 0dcc207

Browse files
committed
mgr/cephadm: Add namespace netmask parameters to nvmeof configuration
Fixes https://tracker.ceph.com/issues/68542 Signed-off-by: Gil Bregman <[email protected]>
1 parent dd4b357 commit 0dcc207

File tree

3 files changed

+17
-2
lines changed

3 files changed

+17
-2
lines changed

src/pybind/mgr/cephadm/services/nvmeof.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def prepare_create(self, daemon_spec: CephadmDaemonDeploySpec) -> CephadmDaemonD
5555
'addr': addr,
5656
'discovery_addr': discovery_addr,
5757
'port': spec.port,
58-
'spdk_log_level': 'WARNING',
58+
'spdk_log_level': '',
5959
'rpc_socket_dir': '/var/tmp/',
6060
'rpc_socket_name': 'spdk.sock',
6161
'transport_tcp_options': transport_tcp_options,

src/pybind/mgr/cephadm/templates/services/nvmeof/ceph-nvmeof.conf.j2

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@ allowed_consecutive_spdk_ping_failures = {{ spec.allowed_consecutive_spdk_ping_f
2020
spdk_ping_interval_in_seconds = {{ spec.spdk_ping_interval_in_seconds }}
2121
ping_spdk_under_lock = {{ spec.ping_spdk_under_lock }}
2222
enable_monitor_client = {{ spec.enable_monitor_client }}
23+
max_hosts_per_namespace = {{ spec.max_hosts_per_namespace }}
24+
max_namespaces_with_netmask = {{ spec.max_namespaces_with_netmask }}
2325

2426
[gateway-logs]
2527
log_level = {{ spec.log_level }}
@@ -53,7 +55,15 @@ rpc_socket_dir = {{ spec.rpc_socket_dir }}
5355
rpc_socket_name = {{ spec.rpc_socket_name }}
5456
timeout = {{ spec.spdk_timeout }}
5557
bdevs_per_cluster = {{ spec.bdevs_per_cluster }}
58+
{% if spec.spdk_log_level %}
5659
log_level = {{ spec.spdk_log_level }}
60+
{% endif %}
61+
{% if spec.spdk_protocol_log_level %}
62+
protocol_log_level = {{ spec.spdk_protocol_log_level }}
63+
{% endif %}
64+
{% if spec.spdk_log_file_dir %}
65+
log_file_dir = {{ spec.spdk_log_file_dir }}
66+
{% endif %}
5767
conn_retries = {{ spec.conn_retries }}
5868
transports = {{ spec.transports }}
5969
{% if transport_tcp_options %}
@@ -65,4 +75,7 @@ tgt_cmd_extra_args = {{ spec.tgt_cmd_extra_args }}
6575

6676
[monitor]
6777
timeout = {{ spec.monitor_timeout }}
78+
{% if spec.monitor_client_log_file_dir %}
79+
log_file_dir = {{ spec.monitor_client_log_file_dir }}
80+
{% endif %}
6881

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -409,6 +409,8 @@ def test_nvmeof_config(self, _get_name, _run_cephadm, cephadm_module: CephadmOrc
409409
spdk_ping_interval_in_seconds = 2.0
410410
ping_spdk_under_lock = False
411411
enable_monitor_client = True
412+
max_hosts_per_namespace = 1
413+
max_namespaces_with_netmask = 1000
412414
413415
[gateway-logs]
414416
log_level = INFO
@@ -442,7 +444,7 @@ def test_nvmeof_config(self, _get_name, _run_cephadm, cephadm_module: CephadmOrc
442444
rpc_socket_name = spdk.sock
443445
timeout = 60.0
444446
bdevs_per_cluster = 32
445-
log_level = WARNING
447+
protocol_log_level = WARNING
446448
conn_retries = 10
447449
transports = tcp
448450
transport_tcp_options = {{"in_capsule_data_size": 8192, "max_io_qpairs_per_ctrlr": 7}}

0 commit comments

Comments
 (0)