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

[Alexandre DERUMIER]

>>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?