Okay Brant, sounds good. I'll start working on the SQL code today. Please bear in mind that I'm going to be on vacation next week, so a patch won't be ready until some time after November 11.
On Thu, Oct 31, 2013 at 9:46 AM, Brant Knudson <b...@acm.org> wrote: > Peter - > > 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. > > - Brant > > On Wed, Oct 30, 2013 at 5:08 PM, Peter Feiner <pe...@gridcentric.ca> wrote: >> >> Hi Brant, >> >> In addition to the race you've fixed in >> https://review.openstack.org/#/c/50767/, it looks like there are quite >> a few more races in the SQL backend of keystone.assignment. I filed a >> bug to this effect: https://bugs.launchpad.net/keystone/+bug/1246489. >> The general problem is that transactions are used somewhat >> indiscriminately. The fix (i.e., using transactions judiciously) is >> straightforward and should be mostly independent of your ongoing >> oslo.db sessions port in https://review.openstack.org/#/c/49460/. So, >> unless you already have something in the works, I'll get started on >> that tomorrow. >> >> I'm eager to fix these races so >> https://review.openstack.org/#/c/42967/ can reliably pass tempest :-) >> >> Peter > > > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev@lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
