yahoo-eng-team team mailing list archive

[Zhenyu Zheng <zhengzhenyu@xxxxxxxxxx>]

Public bug reported:

There is a lock in:
https://github.com/openstack/nova/blob/def4b17934a3b2cf783d0177d6a9632916dfd10f/nova/context.py#L363

it is used to prevent from creating duplicated db and mq connections for
racy context content switching, the rece condition actually only occours
if and only if there is no infomation about this cell in the cache(first
time call).

As for the current implementation, we lock the whole call, which may
affect performance.

** Affects: nova
     Importance: Undecided
     Assignee: Zhenyu Zheng (zhengzhenyu)
         Status: New

** Changed in: nova
     Assignee: (unassigned) => Zhenyu Zheng (zhengzhenyu)

** Description changed:

  There is a lock in:
  https://github.com/openstack/nova/blob/def4b17934a3b2cf783d0177d6a9632916dfd10f/nova/context.py#L363
  
- it is used to prevent from creating duplicated db and mq connections for racy context content switching,
- the rece condition actually only occours if and only if there is no infomation about this cell in the
- cache(first time call).
+ it is used to prevent from creating duplicated db and mq connections for
+ racy context content switching, the rece condition actually only occours
+ if and only if there is no infomation about this cell in the cache(first
+ time call).
  
  As for the current implementation, we lock the whole call, which may
  affect performance.

-- 
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/1788564

Title:
  Increase set_target_cell performance by refactor where we lock

Status in OpenStack Compute (nova):
  New

Bug description:
  There is a lock in:
  https://github.com/openstack/nova/blob/def4b17934a3b2cf783d0177d6a9632916dfd10f/nova/context.py#L363

  it is used to prevent from creating duplicated db and mq connections
  for racy context content switching, the rece condition actually only
  occours if and only if there is no infomation about this cell in the
  cache(first time call).

  As for the current implementation, we lock the whole call, which may
  affect performance.

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