Am 16.05.22 um 18:07 schrieb Alexandre Derumier: > Signed-off-by: Alexandre Derumier <aderum...@odiso.com> > --- > PVE/API2/Qemu.pm | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm > index 1d5b66f..57a75fc 100644 > --- a/PVE/API2/Qemu.pm > +++ b/PVE/API2/Qemu.pm > @@ -1227,6 +1227,7 @@ __PACKAGE__->register_method({ > $conf = PVE::QemuConfig->load_current_config($param->{vmid}, > $param->{current});
It'd be a bit nicer to have load_current_config() not return the cloudinit section. And because the implementation in AbstractConfig.pm shouldn't need to know about the cloudinit section, I think we should override load_current_config() in QemuConfig.pm (calling the parent method and removing the cloudinit section there). > } > $conf->{cipassword} = '**********' if $conf->{cipassword}; > + delete $conf->{cloudinit}; > return $conf; > > }}); _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel