<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 9 January 2018 at 15:28, Matthew Booth <span dir="ltr"><<a href="mailto:mbooth@redhat.com" target="_blank">mbooth@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div>In summary, the patch series is here:<div><br></div><a href="https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/local-disk-serial-numbers" target="_blank">https://review.openstack.org/#<wbr>/q/status:open+project:opensta<wbr>ck/nova+branch:master+topic:<wbr>bp/local-disk-serial-numbers</a><br><br></div>The bottom 3 patches, which add BDM.uuid have landed. The next 3 currently have a single +2. Since I last posted I have found and fixed a problem in swap_volume, which added 2 more patches to the series. There are currently 13 outstanding patches in the series.<br><br></div>The following 6 patches are the 'crux' patches. The others in the series are related fixes/cleanups (mostly renaming things and fixing tests) which I've moved into separate patches to reduce noise.<br><br>Add DriverLocalImageBlockDevice:<br><a href="https://review.openstack.org/#/c/526347/6" target="_blank">https://review.openstack.org/#<wbr>/c/526347/6</a></div></blockquote><div><br></div>This now has a +2!<br><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Add local_root to block_device_info:<br><a href="https://review.openstack.org/#/c/529029/6" target="_blank">https://review.openstack.org/#<wbr>/c/529029/6</a><br><div><div><div><div class="gmail_extra"><br></div><div class="gmail_extra">Pass DriverBlockDevice to driver.attach_volume<br></div><div class="gmail_extra"><a href="https://review.openstack.org/#/c/528363/" target="_blank">https://review.openstack.org/#<wbr>/c/528363/</a><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Expose volume host type and path independent of libvirt config
</div><div class="gmail_extra"><a href="https://review.openstack.org/#/c/530786/" target="_blank">https://review.openstack.org/#<wbr>/c/530786/</a><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Don't generate fake disk_info in swap_volume</div><div class="gmail_extra"><a href="https://review.openstack.org/#/c/530787/" target="_blank">https://review.openstack.org/#<wbr>/c/530787/</a><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Local disk serial numbers for the libvirt driver</div><div class="gmail_extra"><a href="https://review.openstack.org/#/c/529380/" target="_blank">https://review.openstack.org/#<wbr>/c/529380/</a></div></div></div></div></div></blockquote><div><br></div><div>These remain the crux patches.</div><div><br></div><div>Some of the simpler cleanup patches have also attracted +2s. As I've had to rebase the series due to merge conflicts a couple of times, I've moved these to the bottom so they can potentially go straight in. Cleanup patches with a single +2 already:</div><div><br></div><div><a href="https://review.openstack.org/#/c/531179/">https://review.openstack.org/#/c/531179/</a></div><div><a href="https://review.openstack.org/#/c/526346/">https://review.openstack.org/#/c/526346/</a></div><div><a href="https://review.openstack.org/#/c/528362/">https://review.openstack.org/#/c/528362/</a></div><div><br></div><div>Thanks,<br></div><div><br></div><div>Matt<br></div>-- <br></div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><span style="font-size:12.8px">Matthew Booth</span><br></div><div>Red Hat OpenStack Engineer, Compute DFG</div><div><br></div><div>Phone: +442070094448 (UK)</div><div><br></div></div></div></div></div></div></div>
</div></div>