[pve-devel] [PATCH] Implement pending change

[Alexandre DERUMIER]

>>And if we need to keep a reference, we do that in the default section/config. 
>>IMHO there is no need to maintain multiple 'unusedX' lists. 

I'm thinking about this case, vm online hotplug off


[CONF]
virtio0 : local:100/vm-100-disk-1.qcow2
unused0 : ...

-delete virtio0

[CONF]
virtio0: local:100/vm-100-disk-1.qcow2
unused0 : ...
unused1 : local:100/vm-100-disk-1.qcow2  


-delete unused0
-> I simply delete the vol and remove unused0


but

-delete unused1
So, what happen here ?




----- Mail original ----- 

De: "Dietmar Maurer" <dietmar at proxmox.com> 
À: "Alexandre DERUMIER" <aderumier at odiso.com> 
Cc: pve-devel at pve.proxmox.com 
Envoyé: Lundi 27 Octobre 2014 13:25:43 
Objet: RE: [pve-devel] [PATCH] Implement pending change 

> > >>And why do you want to maintain a list of unused disks inside [PENDING]? 
> > 
> > because we can delete and add a new disk on same id. (or swap it) If I 
> > don't maintain a list of unused disk and use delete: 

And if we need to keep a reference, we do that in the default section/config. 
IMHO there is no need to maintain multiple 'unusedX' lists.