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. _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
