[pve-devel] [PATCH manager] ui: cloudinit: disallow unprivileged users to regenerate images

Daniel Kral d.kral at proxmox.com
Thu Nov 7 14:28:29 CET 2024


On 11/4/24 13:02, Fiona Ebner wrote:
> Since those users can already use the cloudinit_update endpoint (just
> not via UI) and thus effectively don't need "VM.Config.CDROM", I think
> it's better to change the UI to use that endpoint as well.
> 
> The endpoint was added here [0] and the switch in the UI was already
> proposed [1] as part of that but it seems never applied.
> 
> [0]:
> https://lore.proxmox.com/pve-devel/20220622115206.3295425-1-aderumier@odiso.com/
> 
> [1]:
> https://lore.proxmox.com/pve-devel/20220622141335.3616129-2-aderumier@odiso.com/

Thanks for pointing that out! I also think that it would be a better 
solution. I've already applied the UI patches locally, but since the 
cloudinit-related API endpoint's have been changed significantly since 
then, I will update those parts of the patch and send it anew as a v2 
with an added Originally-by trailer.




More information about the pve-devel mailing list