commit 85d0a8b2e09e37639ad29bb3bbcbbad55719dd8e
Author: Bob.Haddleton <email address hidden>
Date: Sat Feb 9 21:24:54 2019 -0600
RedisLock release() should not check if the lock has been acquired
The RedisDriver coordinator _stop() method removes the lock from the
_acquired_locks set before calling the release() method on the lock,
so the release() method should not check to see if the lock is in the
set, since it never will be.
This was preventing the coordinator's _stop() method from releasing
the locks, so they had to timeout before another group member could
take over as leader.
Change-Id: Ie23d88f1a05983d7465f18a329cabc23d789a628
Closes-Bug: 1815330
(cherry picked from commit 842c7404b2d01c5dd718c14bb55f0cd47471f0e1)
Reviewed: https:/ /review. opendev. org/c/openstack /tooz/+ /848653 /opendev. org/openstack/ tooz/commit/ 85d0a8b2e09e376 39ad29bb3bbcbba d55719dd8e
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/train
commit 85d0a8b2e09e376 39ad29bb3bbcbba d55719dd8e
Author: Bob.Haddleton <email address hidden>
Date: Sat Feb 9 21:24:54 2019 -0600
RedisLock release() should not check if the lock has been acquired
The RedisDriver coordinator _stop() method removes the lock from the
_acquired_locks set before calling the release() method on the lock,
so the release() method should not check to see if the lock is in the
set, since it never will be.
This was preventing the coordinator's _stop() method from releasing
the locks, so they had to timeout before another group member could
take over as leader.
Change-Id: Ie23d88f1a05983 d7465f18a329cab c23d789a628 dd718c14bb55f0c d47471f0e1)
Closes-Bug: 1815330
(cherry picked from commit 842c7404b2d01c5