openstack
/
oslo.concurrency
Code Issues Proposed changes

Default lockutils to using a tempdir 

* Previously lockutils attempted to use __file__/../ as the lock path.
  This causes packages to fail if the user running the code doesn't have
  +w access to that dir. Remove the default so it will use mkdtemp
  instead
* Fixes bug 1107950

Change-Id: Ie0cd6fac5403a53a768eb4a0ce7da602b3fef755
This commit is contained in:
Jason Kölker 2013-01-28 17:32:02 -06:00
parent d93e3baa02
commit 3ca8b8b8e4
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
openstack/common/lockutils.py
View File

@ -39,9 +39,8 @@ util_opts = [
cfg.BoolOpt('disable_process_locking', default=False,
help='Whether to disable inter-process locks'),
cfg.StrOpt('lock_path',
default=os.path.abspath(os.path.join(os.path.dirname(__file__),
'../')),
help='Directory to use for lock files')
help=('Directory to use for lock files. Default to a '
'temp directory'))
]