tempest: re-enable image dependency tests#2895
Merged
Merged
Conversation
Reverts commit 621a279 (PR #2885), which excluded both ImageDependencyTests as a workaround for the HTTP 409 failure caused by the Glance/Ceph RBD clone v1 incompatibility. The root cause is fixed by: - osism/defaults#284 — adds ceph_require_min_compat_client default - osism/container-image-ceph-ansible#690 — applies it via ceph-pools Merge this commit after both of the above are in. Related: #2884 AI-assisted: Claude Code Signed-off-by: Roger Luethi <luethi@osism.tech>
ideaship
force-pushed
the
rl_revert_image_dep_exclusion
branch
from
73de952 to
ca14645
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ImageDependencyTestsentries fromenvironments/openstack/files/tempest/exclude.lstthat were added as a workaround in Exclude failing image dependency tempest tests #2885Background
The tests were excluded because Glance returned HTTP 409 when deleting an image with active Nova ephemeral RBD clones. Root cause: the default Ceph compat level (luminous) uses RBD clone v1, where parent snapshots must remain protected as long as clones exist.
glance_store'sunprotect_snap()call raisesImageBusybefore thetrash_move()fallback is reached.The fix is to raise the Ceph OSD map
require-min-compat-clienttomimic, which enables RBD clone v2 cluster-wide. Under clone v2 the protection lock is not used,unprotect_snap()succeeds, and deletion proceeds viatrash_move().Prerequisites — merge after both of these
ceph_require_min_compat_client: mimicdefault variableceph-poolslifecycle stepTest plan
ImageDependencyTestspassingCloses #2884
🤖 Generated with Claude Code