<div dir="ltr"><div dir="ltr">The first one of those was trivial (although I am confused as to why it didn't fail for the test run where the patch was proposed, I can't see an obvious race condition). I have proposed a fix at <a href="https://review.openstack.org/#/c/621346/">https://review.openstack.org/#/c/621346/</a> .<div><br></div><div>Michael</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Dec 1, 2018 at 8:30 AM Matt Riedemann <<a href="mailto:mriedemos@gmail.com">mriedemos@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just FYI there are a couple of regressions hitting nova unit and <br>
functional test jobs right now:<br>
<br>
<a href="https://bugs.launchpad.net/nova/+bug/1806123" rel="noreferrer" target="_blank">https://bugs.launchpad.net/nova/+bug/1806123</a><br>
<br>
^ Is likely due to mocking a global for the new I/O concurrency <br>
semaphore stuff in the libvirt driver. I'm not sure what we should do <br>
about that one. During the code review I suggested writing a fixture <br>
which would essentially maintain the context manager (so we have <br>
__enter__ and __exit__) but just make it a noop, but we still want to <br>
make sure it's called in places where it's used.<br>
<br>
<a href="https://bugs.launchpad.net/nova/+bug/1806126" rel="noreferrer" target="_blank">https://bugs.launchpad.net/nova/+bug/1806126</a><br>
<br>
^ Is a bit hairier since I'm seeing both weird, potentially global mock, <br>
failures and also timeouts, also potentially because of mocking globals. <br>
Since there is no functional code change tied to that one yet (it's <br>
still being reviewed, this was a recreate test change only), I have <br>
proposed a revert to try and stem the bleeding unless someone (mdbooth?) <br>
can root cause and fix it faster.<br>
<br>
Happy rechecking!<br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt<br>
<br>
</blockquote></div>