[pve-devel] [PATCH 2/6] vmconfig_hotplug_pending : add update_net

[Dietmar Maurer]

> >>THis looks also problematic. What if someone sets hotplug=0? This also
> >>use wrong values if the VM is migrated?
> 
> I manage this inside deviceplug|unplug.
> (I remove the pending device at the end)
> 
> 
> sub vm_deviceplug {
> ....
>    return 1 if !$conf->{hotplug};
> ...
>    #delete pending device after hotplug
>     if($conf->{pending}->{$deviceid}){
>         $conf->{$deviceid} = $optvalue;
>         delete $conf->{pending}->{$deviceid};
>         PVE::QemuServer::update_config_nolock($vmid, $conf, 1);
>     }
> }

What is someone wants:

hotplug: 0

Then you want to keep changes in [PENDING], without applying them.