<div dir="ltr">Peter -<br><br><div><div class="gmail_extra">We discussed better use of transactions in irc, but I don't think anyone has had a chance to look at it. This would be a very useful thing to have someone look at. I'm fine with holding off on the oslo.db sessions work until we're sure the code is correct w/r/t multi-processing so that tempest is going to pass consistently.<br>
</div><div class="gmail_extra"><br></div><div class="gmail_extra">- Brant<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 30, 2013 at 5:08 PM, Peter Feiner <span dir="ltr"><<a href="mailto:peter@gridcentric.ca" target="_blank">peter@gridcentric.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Brant,<br>
<br>
In addition to the race you've fixed in<br>
<a href="https://review.openstack.org/#/c/50767/" target="_blank">https://review.openstack.org/#/c/50767/</a>, it looks like there are quite<br>
a few more races in the SQL backend of keystone.assignment. I filed a<br>
bug to this effect: <a href="https://bugs.launchpad.net/keystone/+bug/1246489" target="_blank">https://bugs.launchpad.net/keystone/+bug/1246489</a>.<br>
The general problem is that transactions are used somewhat<br>
indiscriminately. The fix (i.e., using transactions judiciously) is<br>
straightforward and should be mostly independent of your ongoing<br>
oslo.db sessions port in <a href="https://review.openstack.org/#/c/49460/" target="_blank">https://review.openstack.org/#/c/49460/</a>. So,<br>
unless you already have something in the works, I'll get started on<br>
that tomorrow.<br>
<br>
I'm eager to fix these races so<br>
<a href="https://review.openstack.org/#/c/42967/" target="_blank">https://review.openstack.org/#/c/42967/</a> can reliably pass tempest :-)<br>
<span class="HOEnZb"><font color="#888888"><br>
Peter<br>
</font></span></blockquote></div><br></div></div></div>