Skip to content

Conversation

@fmount
Copy link
Contributor

@fmount fmount commented Nov 25, 2025

Currently Zuul kuttl jobs fail due to the fact we're not de-allocating PVCs after each kuttl test execution. This patch aims to improve the cleanup process by removing them as part of the cleanup step.

Jira: https://issues.redhat.com/browse/OSPRH-22400

Currently Zuul fails due to the fact we're not deallocating PVCs after
each kuttl test execution. This patch aims to improve the cleanup
process by removing them as part of the clenup step.

Signed-off-by: Francesco Pantano <[email protected]>
@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bdb65f0897d64acda571b06bed584fab

openstack-k8s-operators-content-provider TIMED_OUT in 30m 56s
⚠️ glance-operator-kuttl SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider (non-voting)
⚠️ glance-operator-tempest SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider (non-voting)

@fmount
Copy link
Contributor Author

fmount commented Nov 25, 2025

recheck

@fmount
Copy link
Contributor Author

fmount commented Nov 25, 2025

/test glance-operator-build-kuttl

@fmount
Copy link
Contributor Author

fmount commented Nov 25, 2025

recheck

@fmount
Copy link
Contributor Author

fmount commented Nov 25, 2025

/test glance-operator-build-kuttl

@fmount fmount force-pushed the kuttl_zuul_fix branch 4 times, most recently from ce03f73 to 9be3f13 Compare November 25, 2025 22:30
Glance tempest job doesn't need a full ceph deployment. This patch adds
the cifmw variables to skip rgw, dashboard and cephfs deployment.

Signed-off-by: Francesco Pantano <[email protected]>
@fmount
Copy link
Contributor Author

fmount commented Nov 26, 2025

recheck

@fmount fmount requested a review from tosky November 26, 2025 09:46
@fmount
Copy link
Contributor Author

fmount commented Nov 26, 2025

/test glance-operator-build-kuttl

@fmount
Copy link
Contributor Author

fmount commented Nov 26, 2025

recheck

@fmount fmount force-pushed the kuttl_zuul_fix branch 3 times, most recently from 598f4c4 to 883cd6d Compare November 26, 2025 17:34
@fmount
Copy link
Contributor Author

fmount commented Nov 26, 2025

/test glance-operator-build-kuttl

@fmount
Copy link
Contributor Author

fmount commented Nov 27, 2025

recheck

1 similar comment
@fmount
Copy link
Contributor Author

fmount commented Nov 27, 2025

recheck

Copy link
Contributor

@tosky tosky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the record, as discussed with @fmount, it is not feasible to add a shared code to delete the resources because the tests are not really independent (same test suite)

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 27, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount, tosky

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit e107825 into openstack-k8s-operators:main Nov 27, 2025
9 checks passed
@tosky
Copy link
Contributor

tosky commented Nov 28, 2025

/cherry-pick 18.0-fr4

@openshift-cherrypick-robot

@tosky: #838 failed to apply on top of branch "18.0-fr4":

Applying: Delete allocated PVCs after each kuttl test execution
Applying: Do not deploy additional ceph services
Using index info to reconstruct a base tree...
M	zuul.d/jobs.yaml
Falling back to patching base and 3-way merge...
Auto-merging zuul.d/jobs.yaml
CONFLICT (content): Merge conflict in zuul.d/jobs.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0002 Do not deploy additional ceph services

In response to this:

/cherry-pick 18.0-fr4

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@fmount
Copy link
Contributor Author

fmount commented Nov 28, 2025

I think it's ok to not cherry pick this change and target the CI bump to FR5 along with Ceph Tentacle

@tosky
Copy link
Contributor

tosky commented Nov 28, 2025

I think it's ok to not cherry pick this change and target the CI bump to FR5 along with Ceph Tentacle

I thought it made sense to at least have the jobs working in 18.0-fr4, otherwise they just burn CPU time without being useful at all.
I wouldn't make them voting on 18.0-fr4, but it may be useful to have them around. I guess we need to backport something else too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants