yahoo-eng-team team mailing list archive

[Ben Nemec <openstack@xxxxxxxxxxxx>]

I believe everyone is on oslo.concurrency now, so this should no longer
be an issue anywhere.

** Changed in: cinder
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1065531

Title:
  lockutils - remove lock dir creation and cleanup

Status in Cinder:
  Fix Released
Status in OpenStack Neutron (virtual network service):
  Fix Released
Status in OpenStack Compute (Nova):
  Fix Released
Status in The Oslo library incubator:
  Fix Released

Bug description:
  See https://review.openstack.org/14139

  This:

                      if not local_lock_path:
                          cleanup_dir = True
                          local_lock_path = tempfile.mkdtemp()

                      if not os.path.exists(local_lock_path):
                          cleanup_dir = True
                          ensure_tree(local_lock_path)
                      ...
                      finally:
                          # NOTE(vish): This removes the tempdir if we needed
                          #             to create one. This is used to cleanup
                          #             the locks left behind by unit tests.
                          if cleanup_dir:
                              shutil.rmtree(local_lock_path)

  
  Why are we deleting the lock dir here? Does that even work? i.e. what if someone concurrently tries to take the lock, re-creates the dir and lock a new file?

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