[pve-devel] [PATCH manager] fix #4808: ceph: use setting names with underscores

Maximiliano Sandoval m.sandoval at proxmox.com
Tue Sep 5 10:53:31 CEST 2023


As suggested in [1], it is recommended to use `_` in all cases when
dealing with config files.

[1] https://docs.ceph.com/en/reef/rados/configuration/ceph-conf/#option-names

Signed-off-by: Maximiliano Sandoval <m.sandoval at proxmox.com>
---
 PVE/API2/Ceph/MDS.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/PVE/API2/Ceph/MDS.pm b/PVE/API2/Ceph/MDS.pm
index 1cb0b74f..6fc0ae45 100644
--- a/PVE/API2/Ceph/MDS.pm
+++ b/PVE/API2/Ceph/MDS.pm
@@ -153,10 +153,10 @@ __PACKAGE__->register_method ({
 	    }
 
 	    $cfg->{$section}->{host} = $nodename;
-	    $cfg->{$section}->{"mds standby for name"} = 'pve';
+	    $cfg->{$section}->{'mds_standby_for_name'} = 'pve';
 
 	    if ($param->{hotstandby}) {
-		$cfg->{$section}->{"mds standby replay"} = 'true';
+		$cfg->{$section}->{'mds_standby_replay'} = 'true';
 	    }
 
 	    cfs_write_file('ceph.conf', $cfg);
-- 
2.39.2






More information about the pve-devel mailing list