<div dir="ltr">I'm continuing to work through updating the imagebackend refactor patches. A bunch of them at the top of the queue already have +2 from jaypipes, who expressed an interest in getting some of these merged. I think the following could be ready to go (in order):<div><br></div><div><a href="https://review.openstack.org/#/c/337159/">https://review.openstack.org/#/c/337159/</a>  libvirt: Rewrite _test_finish_migration [1]</div><div><a href="https://review.openstack.org/#/c/338993/">https://review.openstack.org/#/c/338993/</a>  libvirt: Test disk creation in test_hard_reboot<br></div><div><a href="https://review.openstack.org/#/c/339114/">https://review.openstack.org/#/c/339114/</a>  libvirt: Cleanup test_create_configdrive<br></div><div><a href="https://review.openstack.org/#/c/333272/">https://review.openstack.org/#/c/333272/</a>  libvirt: Rename Backend snapshot and image [2]<br></div><div><a href="https://review.openstack.org/#/c/331115/">https://review.openstack.org/#/c/331115/</a>  libvirt: Never copy a swap disk during cold migration<br></div><div><a href="https://review.openstack.org/#/c/331118/">https://review.openstack.org/#/c/331118/</a>  libvirt: Don't re-resize disks in finish_migration() [3]</div><div><br></div><div>[1] This has -1 from melwitt, but I don't personally think this issue is worth a respin. However, she mentioned on IRC that there may be more, so that could be moot. Anyway, being the top of the queue this is obviously a blocker.</div><div><div><br></div><div>[2] This doesn't have +2 from jaypipes, although it's uncontroversial and uncomplicated. Hopefully just an oversight?</div><div><br></div><div>[3] This patch removes the function which melwitt identified in [1] lost some test coverage, which is why I'm hoping not to respin for that.</div><div><br></div><div>I'll leave it there for the moment, because I need to talk through the next patch with ftersin. It looks like it will require a release note at least, if not a change.</div><div><br></div><div>If anybody would like to slog through some of the above and add a second +2 I'd be very grateful. There's plenty more in the queue after those!</div><div><br></div><div>Thanks,</div><div><br></div><div>Matt</div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><span style="font-size:12.8px">Matthew Booth</span><br></div><div>Red Hat Engineering, Virtualisation Team</div><div><br></div><div>Phone: +442070094448 (UK)</div><div><br></div></div></div></div></div>
</div></div>