backport of the fix for bug #1947370 make lock_path a requird config option when prvisouls it was optional
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Triaged
|
Low
|
sean mooney | ||
os-brick |
Invalid
|
Undecided
|
Unassigned |
Bug Description
https:/
as part of fixing bug 1947370 (https:/
https:/
made the external lock_path config option required with no default provided
this was then backported breaking nova unit tests on stabel branches and potentially
any deployment that upgrade to a new version of os-brick without this defined.
i don't belive that such a backport is in line with stable policy and if it was to be backported
a sane default like /tmp/os_brick_lock would be required to not break existing installs.
this i currently breaking downstream unit test for redhat osp 17 and its also breaking the upstream stable wallayb unittest for nova.
it is unclear if this has directly broken any real world deployment but it has the potential too.
as noted in this revert patch https:/
it is trival to reproduce this
git clone https:/
cd nova-test
git checkout --track origin/
tox -e py3
^ this shoudl fail with the lock_path excption
cd ..
git clone https:/
cd os-brick-revert
git fetch https:/
cd ../nova-test
.tox/py3/
tox -e py3
that will no longer have the lock_path error
.tox/py38/
while I'm not sure the revert is the correct way to proceed we will need to blacklist the broken os-brick release in the requirement repo and come up with a backportable fix for all affected branches.
i have confirmed that the test that are failing are not setting REQUIRES_LOCKING to true so the fixture
that create a tmp dir for the file system lock is not enabled.
this was not previous required because os-brick did not use file system locks in these code paths but now that it does nova should just set that in the test setup.
ill push a patch to correct that shortly triageing this as low since it only affect untitest, real deployments would have this configured for novas file system locks.
we can likely close this os-brick bug and works round this form the nova side by backporting my patch to every release where the os-brick patch was backported.