<div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">Hi,Cr¨ªstian</span></font><br><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">I've read your patch, which has solved the snapshots restore problems more clearly. I agree with you that we've already fixed this issue in separate patches, and I need to make some changes for the formatting issues and test case.</span></font><br><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">My patch will be responsible for the url redirect problem, which modifies base.py and function 'revert' in vmsnapshots.py.</span></font><br><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">Besides, I found there are two points need to pay attention to:</span></font><br><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">1. snap.listAllChildren() in _backup_snapshots.py need flag in older libvirt versions, as I used "0.10.2", it raised "listAllChildren() takes exactly 2 arguments (1 given)"&nbsp; </span></font><br><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">2. UI transfer param 'name' no matter you update vm name or not (of course while you edit a vm), so do we only need to bakup and restore the snapshots while vm name is really updated ? </span></font><br><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">Best regards,</span></font><br><font class="" size="3"><span style="font-family: Times New Roman; font-size: 18px;">Zongmei Gou</span></font></div></div></div><br><br><span title="neteasefooter"><span id="netease_mail_footer"></span></span>