[openstack-dev] Keystone Concurrency & Races in SQL Assignment Backend

Peter Feiner peter at gridcentric.ca
Thu Oct 31 14:24:33 UTC 2013


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 <blk at 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 <peter at 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 at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



More information about the OpenStack-dev mailing list