[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