[pve-devel] [PATCH v3 container] fix #4192: revamp check for systemd version
Thomas Lamprecht
t.lamprecht at proxmox.com
Wed Nov 16 13:14:24 CET 2022
Am 16/11/2022 um 13:13 schrieb Thomas Lamprecht:
> actually I did not yet pushed it out and rethought this, while I don't think that
> the perm change is problematic w.r.t. backward compat in practice, it seems a bit
> odd to argue for disk only, maybe we need to also check for CDROM, as cloudinit
> *adds* a cdrom drive, so it may required that too.
>
> But, to decide that I wanted to check our privilege docs, only to notice that
> we do not mention the cloudinit one there at all, not great...
>
> https://pve.proxmox.com/pve-docs/chapter-pveum.html#_privileges
>
> Please check the dev history to see if Cloudinit is deemed enough to add the CDROM
> or if we should add that priv to the check too, then re-send this with updated
> commit message, the whitespace fixes and a docs patch.
>
argh, replied to the wrong patch, please ignore.
More information about the pve-devel
mailing list