On 9/30/19 2:44 PM, Oguz Bektas wrote:
> we overwrite the load_current_config method from AbstractConfig, in
> order to handle cipassword.
> 

see my reply to 05/18 regarding squashing, as when thinking about a
sense full commit message for this the felling that this does not
deserves to be a stand alone patch just got much much stronger.

> Signed-off-by: Oguz Bektas <o.bek...@proxmox.com>
> ---
>  PVE/QemuConfig.pm | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/PVE/QemuConfig.pm b/PVE/QemuConfig.pm
> index edbf1a7..7f229b3 100644
> --- a/PVE/QemuConfig.pm
> +++ b/PVE/QemuConfig.pm
> @@ -397,6 +397,20 @@ sub __snapshot_foreach_volume {
>  
>      PVE::QemuServer::foreach_drive($conf, $func);
>  }
> +
> +sub load_current_config {
> +    my ($class, $vmid, $snapname, $current) = @_;
> +
> +    my $conf = $class->SUPER::load_current_config($vmid, $snapname, 
> $current);
> +
> +    if ($conf->{cipassword}) {
> +     $conf->{cipassword} = '**********';
> +    }
> +
> +    return $conf;
> +}
> +
> +
>  # END implemented abstract methods from PVE::AbstractConfig
>  
>  1;
> 


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to