]> git-server-git.apps.pok.os.sepia.ceph.com Git - teuthology.git/commit
projects / teuthology.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 596e0a4) | patch
lock.ops.unlock_one_safe: Invert run-match logic
authorZack Cerza <zack@cerza.org>
Wed, 19 Mar 2025 18:35:11 +0000 (12:35 -0600)
committerDan Mick <dan.mick@redhat.com>
Fri, 12 Dec 2025 09:18:37 +0000 (01:18 -0800)
commitdfdcab7bca7a8d017665f254e088632d99e2f042
tree25cd46b7d5c704964be953f8fc80c9669e2e0f7atree | snapshot
parent596e0a42f088a62e6292d6586fccd535f86076d4commit | diff
lock.ops.unlock_one_safe: Invert run-match logic

When unlock_one_safe is called with run_name, the caller means to express
"unlock this node if it belongs to this run".
When it is called with run_name and job_id, it means "unlock this node if it
belongs to this job in this run".
We had inverted the logic, causing leaks on reimage failures.

Signed-off-by: Zack Cerza <zack@cerza.org>
(cherry picked from commit b9a0b03d807de6204e2f01e3622993856097cde7)
teuthology/lock/ops.py diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.