Re: [pve-devel] [PATCH v5 pve-storage, pve-manager 00/11] Fix #4759: Configure Permissions for ceph-crash.service

[Friedrich Weber <f.weber@proxmox.com>]

On 02/04/2024 16:55, Max Carrara wrote:
> Fix #4759: Configure Permissions for ceph-crash.service - Version 5
> ===================================================================

Thanks for the v4! Consider this

Tested-by: Friedrich Weber <f.weber@proxmox.com>

Details:

- like Maximiliano, removed the version check against 0.0.0 for testing

- set up a fresh PVE 8.1 cluster (without Ceph yet) and installed the
patched pve-storage and pve-manager. Noticed that this creates
/etc/pve/ceph/ even though Ceph is not set up, but I suppose this is
intended (and the extra directory shouldn't hurt). Afterwards, set up a
Quincy cluster. Did not notice any issues. Keyring was created and Ceph
config was written correctly, ceph-crash was able to post crash reports.

- installed patched packages on top of my existing PVE 8.1 + Reef
cluster, did not notice any issues. As expected, ceph-crash was
restarted and started posting crash reports.

- did the same for an existing PVE 8.0 + Quincy cluster (as part of the
upgrade to PVE 8.1), did not notice any issues

- installed on top of a Ceph-less standalone node, did not notice any issues