← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1154811] Re: sqlalchemy impl calls back to DB API

 

** Changed in: nova
       Status: Fix Committed => Fix Released

** Changed in: nova
    Milestone: None => grizzly-rc1

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1154811

Title:
  sqlalchemy impl calls back to DB API

Status in OpenStack Compute (Nova):
  Fix Released

Bug description:
  In 3 places, sqlalchemy has calls back to the DB API:

  db.instance_get_by_uuid(context, instance_uuid)
  db.aggregate_get(context, aggregate_id)
  db.ec2_instance_create(context, instance_ref['uuid'])

  While this an eventlet bug, when tpool is enabled..  The call path
  ends up being:

  tpool.execute {  sqlalchemy_method { tpool.execute{ sqlalchemy_method
  } } }

  eventlet tries to deal with recursive tpool.execute()s but it seems to
  cause lockups.

  Either way, it's not necessary to call back to DB API from the
  sqlalchemy backend, and I think that should be avoided.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1154811/+subscriptions