[pve-devel] [PATCH] vmstate snapshot : do no write machine to running config

Alexandre DERUMIER aderumier at odiso.com
Fri Aug 29 10:01:52 CEST 2014


>>For example, in snapshot_rollback(),$forcemachine is wrong because you do not copy machine 
>>config with snapshot_apply_config() .... 

I just send a patch,

we just need to take $forcemachine from snapshot machine value, and not current config.



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

De: "Alexandre DERUMIER" <aderumier at odiso.com> 
À: "Dietmar Maurer" <dietmar at proxmox.com> 
Cc: pve-devel at pve.proxmox.com 
Envoyé: Vendredi 29 Août 2014 09:25:46 
Objet: Re: [pve-devel] [PATCH] vmstate snapshot : do no write machine to running config 

>>For example, in snapshot_rollback(),$forcemachine is wrong because you do not copy machine 
>>config with snapshot_apply_config() .... 
>> 
>>or do I miss something? 

Ok, I'll check that ! 

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

De: "Dietmar Maurer" <dietmar at proxmox.com> 
À: "Alexandre Derumier" <aderumier at odiso.com>, pve-devel at pve.proxmox.com 
Envoyé: Vendredi 29 Août 2014 08:58:35 
Objet: RE: [pve-devel] [PATCH] vmstate snapshot : do no write machine to running config 


> Currently,if we don't have a "machine" option in running config, and we take a 
> vmstate snapshot 
> 
> the machine option is write in the snapshot (ok), but also in the running config 
> (bad) 

Yes, we should fix this when we create a snapshot. 

But your patch also affect rollback, where we have special code to restore machine config. 

For example, in snapshot_rollback(),$forcemachine is wrong because you do not copy machine 
config with snapshot_apply_config() .... 

or do I miss something? 
_______________________________________________ 
pve-devel mailing list 
pve-devel at pve.proxmox.com 
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel 



More information about the pve-devel mailing list