Skip to content

Commit 11de53f

Browse files
committed
mgr/cephadm: revert SPDK RPC fields in nvmeof configuration
Fixes https://tracker.ceph.com/issues/67844 Signed-off-by: Gil Bregman <[email protected]>
1 parent cb28d39 commit 11de53f

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,9 @@ def prepare_create(self, daemon_spec: CephadmDaemonDeploySpec) -> CephadmDaemonD
5252
'name': name,
5353
'addr': host_ip,
5454
'port': spec.port,
55-
'spdk_protocol_log_level': 'WARNING',
56-
'rpc_socket': '/var/tmp/spdk.sock',
55+
'spdk_log_level': 'WARNING',
56+
'rpc_socket_dir': '/var/tmp/',
57+
'rpc_socket_name': 'spdk.sock',
5758
'transport_tcp_options': transport_tcp_options,
5859
'rados_id': rados_id
5960
}

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ omap_file_lock_retry_sleep_interval = {{ spec.omap_file_lock_retry_sleep_interva
1818
omap_file_update_reloads = {{ spec.omap_file_update_reloads }}
1919
allowed_consecutive_spdk_ping_failures = {{ spec.allowed_consecutive_spdk_ping_failures }}
2020
spdk_ping_interval_in_seconds = {{ spec.spdk_ping_interval_in_seconds }}
21+
ping_spdk_under_lock = {{ spec.ping_spdk_under_lock }}
2122
enable_monitor_client = {{ spec.enable_monitor_client }}
2223

2324
[gateway-logs]
@@ -48,11 +49,11 @@ root_ca_cert = /root.ca.cert
4849

4950
[spdk]
5051
tgt_path = {{ spec.tgt_path }}
51-
rpc_socket = {{ spec.rpc_socket }}
52+
rpc_socket_dir = {{ spec.rpc_socket_dir }}
53+
rpc_socket_name = {{ spec.rpc_socket_name }}
5254
timeout = {{ spec.spdk_timeout }}
5355
bdevs_per_cluster = {{ spec.bdevs_per_cluster }}
54-
log_level={{ spec.spdk_log_level }}
55-
protocol_log_level = {{ spec.spdk_protocol_log_level }}
56+
log_level = {{ spec.spdk_log_level }}
5657
conn_retries = {{ spec.conn_retries }}
5758
transports = {{ spec.transports }}
5859
{% if transport_tcp_options %}

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -407,6 +407,7 @@ def test_nvmeof_config(self, _get_name, _run_cephadm, cephadm_module: CephadmOrc
407407
omap_file_update_reloads = 10
408408
allowed_consecutive_spdk_ping_failures = 1
409409
spdk_ping_interval_in_seconds = 2.0
410+
ping_spdk_under_lock = False
410411
enable_monitor_client = True
411412
412413
[gateway-logs]
@@ -437,11 +438,11 @@ def test_nvmeof_config(self, _get_name, _run_cephadm, cephadm_module: CephadmOrc
437438
438439
[spdk]
439440
tgt_path = /usr/local/bin/nvmf_tgt
440-
rpc_socket = /var/tmp/spdk.sock
441+
rpc_socket_dir = /var/tmp/
442+
rpc_socket_name = spdk.sock
441443
timeout = 60.0
442444
bdevs_per_cluster = 32
443-
log_level=
444-
protocol_log_level = WARNING
445+
log_level = WARNING
445446
conn_retries = 10
446447
transports = tcp
447448
transport_tcp_options = {{"in_capsule_data_size": 8192, "max_io_qpairs_per_ctrlr": 7}}

0 commit comments

Comments
 (0)