On 25/07/2023 11:54, Thomas Lamprecht wrote: > How about a get_vm_user_cloudinit_options helper located directly > below the format definition, filtering out those keys that do not > make sense, or are off limits, and use that? The helper sounds good! AFAICT, $cloudinitoptions contains exactly all keys of $confdesc_cloudinit anyway -- so currently the helper could just return a hash that maps all keys of $confdesc_cloudinit to 1. If it's located close to the definition of $confdesc_cloudinit, I'd hope that it makes adding new options in the future less error-prone (even for adding a hypothetical option that should actually be modifiable by root only) I'll look into it.
_______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
