Bug 1121796 – potential race condition on dialect init in SQLAlchemy 0.8.4

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1121796 - potential race condition on dialect init in SQLAlchemy 0.8.4

Summary:	potential race condition on dialect init in SQLAlchemy 0.8.4

Status:	CLOSED ERRATA

Aliases:	None

Product:	Red Hat OpenStack
Classification:	Red Hat
Component:	python-sqlalchemy (Show other bugs)
Sub Component:
Version:	5.0 (RHEL 6)
Hardware:	Unspecified Unspecified

Priority	high Severity high
Target Milestone:	z4
Target Release:	5.0 (RHEL 7)
Assigned To:	Michael Bayer
QA Contact:	Ami Jeain
Docs Contact:

URL:
Whiteboard:
Keywords:	ZStream

Depends On:
Blocks:	1198773
	Show dependency tree / graph

Reported:	2014-07-21 17:57 EDT by Michael Bayer
Modified:	2015-06-04 18:04 EDT (History)
CC List:	9 users (show)

See Also:
Fixed In Version:	0.8.4-2.el7ost
Doc Type:	Bug Fix
Doc Text:	Previously, an improvement to the connection pool such that new connections could be made concurrently, made it so that the 'init on first connect' routine of a SQLAlchemy dialect would not have been completed if concurrent routines proceeded at the same time. As a result, when a SQLAlchemy engine was first used, operations which relied on the state acquired during initial startup could fail, as this information would not have been completed. To resolve this issue, with this update, 'mutexing' was added to the event system which handles the initial dialect startup phase, so that connection attempts are again serialized, but only when the engine first starts up.
Story Points:	---
Clone Of:
Clones:	1198773 (view as bug list)
Environment:
Last Closed:	2015-04-16 10:37:13 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHBA-2015:0825	normal	SHIPPED_LIVE	Red Hat Enterprise Linux OpenStack Platform Bug Fix and Enhancement Advisory	2015-04-16 14:28:14 EDT

Groups: None (edit)

Description Michael Bayer 2014-07-21 17:57:59 EDT

Description of problem:

A critical fix was issued in SQLAlchemy 0.8.5 regarding dialect initialization, first introduced as a regression in 0.8.4 where the process of determining database version and attributes upon first connect is no longer synchronized, leading to initialization failures for concurrent connections that skip the initialization step.  


How reproducible:

start any two database connections from a new engine simultaneously, the second connection may not function correctly until the first connection completes its queries.


Steps to Reproduce:

1. the test case at https://bitbucket.org/zzzeek/sqlalchemy/commits/4ac0558e11497553a4d0460111d8fbd667396a7f#Ltest/engine/test_pool.pyT498 illustrates the race condition.

SQLAlchemy bug: https://bitbucket.org/zzzeek/sqlalchemy/issue/2964.  Note there are two separate git revisions to implement the fix fully.

Comment 4 Ami Jeain 2015-03-30 08:29:17 EDT

verified:
ran the code from: https://bitbucket.org/zzzeek/sqlalchemy/commits/4ac0558e11497553a4d0460111d8fbd667396a7f#Ltest/engine/test_pool.py

Comment 6 errata-xmlrpc 2015-04-16 10:37:13 EDT

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-0825.html

Note You need to log in before you can comment on or make changes to this bug.