Skip to content

Conversation

@tosky
Copy link
Contributor

@tosky tosky commented Dec 24, 2024

Add KUTTL and tempest zuul jobs (meant to replace the prow ones in the long term)

KUTTL: generate JUnitXML reports

  • switch the type to XML (the accepted value is 'xml' lowercase, not uppercase);
  • tune the name a bit to highlight it is a report;
  • add the new reportGranularity parameter which is supported by kuttl 0.20.0 and will restore the pre-1.17 JUnitXML format (granularity by test case, instead of by step);
  • remove the URL of kuttl, so that it is not tied to a specific release.

Depends-On: openstack-k8s-operators/ci-framework#2628

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 24, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@tosky
Copy link
Contributor Author

tosky commented Jan 7, 2025

recheck

@tosky tosky marked this pull request as ready for review January 7, 2025 23:47
@openshift-ci openshift-ci bot requested review from abays and frenzyfriday January 7, 2025 23:48
@fmount fmount self-requested a review January 8, 2025 09:50
@tosky
Copy link
Contributor Author

tosky commented Feb 24, 2025

recheck

@cschwede
Copy link
Collaborator

/test all

@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/36199b234d564601ada878d358ea524b

openstack-k8s-operators-content-provider FAILURE in 14m 02s
⚠️ swift-operator-kuttl SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider (non-voting)
⚠️ swift-operator-tempest SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider (non-voting)

@tosky
Copy link
Contributor Author

tosky commented Jun 27, 2025

I would appreciate some help to understand why the swift-operator kuttl jobs require a few additional variables and why those are not passed around in the code I'm using to trigger the those test

- switch the type to XML (the accepted value is 'xml' lowercase,
  not uppercase);
- tune the name a bit to highlight it is a report;
- add the new reportGranularity parameter which is supported
  by kuttl 0.20.0 and will restore the pre-1.17 JUnitXML format
  (granularity by test case, instead of by step);
- remove the URL of kuttl, so that it is not tied to a specific
  release.

Signed-off-by: Luigi Toscano <[email protected]>
They are meant to replace the prow ones in the long term.

Signed-off-by: Luigi Toscano <[email protected]>
@tosky tosky force-pushed the more-zuul-and-junitxml branch from 86d7cb4 to 7676a13 Compare November 28, 2025 17:44
@tosky
Copy link
Contributor Author

tosky commented Dec 1, 2025

I don't know what happened, but it looks like both tempest and kuttl zuul tests work now!

@tosky tosky requested a review from cschwede December 1, 2025 11:43
@tosky
Copy link
Contributor Author

tosky commented Dec 1, 2025

recheck

1 similar comment
@tosky
Copy link
Contributor Author

tosky commented Dec 1, 2025

recheck

Copy link
Contributor

@fmount fmount left a comment

Choose a reason for hiding this comment

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

/lgtm

@fmount
Copy link
Contributor

fmount commented Dec 1, 2025

@cschwede to approve ^

Copy link
Collaborator

@cschwede cschwede left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 2, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cschwede, 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-ci openshift-ci bot added the approved label Dec 2, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 51f7715 into openstack-k8s-operators:main Dec 2, 2025
6 checks passed
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 2, 2025

@tosky: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/swift-operator-build-deploy-kuttl 7676a13 link unknown /test swift-operator-build-deploy-kuttl

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

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