[pve-devel] [PATCH pve-manager] fix #7011: ceph monitor: set ownership of monitor logs

[Maximiliano Sandoval]

Thomas Lamprecht <t.lamprecht at proxmox.com> writes:

> Am 12.12.25 um 14:05 schrieb Dominik Rusovac:
>> Ownership of ceph logs is now set to ceph:ceph after the creation of a
>> new monitor and before the new monitor starts. Hence, effective ceph
>> monitor logging on freshly set up ceph clusters no longer depends on the
>> first upgrade of ceph-common.
>
> Might it be a better fix to then change the postinst script of
> ceph-common, or whatever packages postinst script creates those
> directories, to chown them to ceph:ceph? That way it would also work
> if one installs ceph directly, circumventing pveceph. While that is
> not exactly something we promote, but it's not really hard, and
> packaging is often a good place to take care of such things like
> directory ownership

The directories are created with the right permissions and owner, the
issue here is that the monitor logs generated when we create the monitor
(the command above the call introduced by the patch) are created with
root as the owner.

-- 
Maximiliano