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

[Thomas Lamprecht]

Am 05/09/2023 um 10:53 schrieb Maximiliano Sandoval:
> 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

The first thing I asked myself was "how is the priority though if both exist?"

Then I had to check the code and saw that this is only affecting
MDS creation, where we enforce that the config section doesn't exist,
so it's OK (but still relevant info, so I added it to the commit
message).

Anyhow, it would be still good to know how ceph handles the situation
where both exist, e.g.:

  mds standby for name = foo
  mds_standby_for_name = bar
  # what about mixing them?
  mds_standy for name

Also, did you checked for other such occurrences in our API?

Manually tracing all call sites where cfs_write_file('ceph.conf', ..) is
called should allow you to find most potential call sites using
the undesired style.

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

applied, thanks!