Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

More zuul jobs, and generate JUnitXML reports for KUTTL #479

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

More zuul jobs, and generate JUnitXML reports for KUTTL #479

wants to merge 2 commits into from
+59 −5

Conversation

tosky
Copy link

@tosky tosky commented Dec 24, 2024 •
edited
Loading

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 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
Author

tosky commented Dec 24, 2024

recheck

@tosky tosky force-pushed the more-zuul-and-junitxml branch from 15faa7a to 19e321f Compare January 7, 2025 23:47
@tosky tosky marked this pull request as ready for review January 7, 2025 23:47
@tosky
Copy link
Author

tosky commented Jan 7, 2025

recheck

@openshift-ci openshift-ci bot requested review from Akrog and viroel January 7, 2025 23:47
@tosky
KUTTL: generate JUnitXML reports
1d2ba9b 
- 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 <ltoscano@redhat.com>
@tosky tosky force-pushed the more-zuul-and-junitxml branch from 19e321f to 1db1e4e Compare January 7, 2025 23:53
@tosky
Copy link
Author

tosky commented Jan 8, 2025

recheck

@fmount fmount self-requested a review January 8, 2025 09:50
@fserucas
Copy link

fserucas commented Jan 8, 2025

recheck

@tosky tosky force-pushed the more-zuul-and-junitxml branch 2 times, most recently from ad06ce4 to f43985e Compare January 10, 2025 16:07
fmount
fmount approved these changes Jan 12, 2025
View reviewed changes
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

@openshift-ci openshift-ci bot added the lgtm label Jan 12, 2025
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 12, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fmount, tosky
Once this PR has been reviewed and has the lgtm label, please assign stuggi for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@tosky tosky marked this pull request as draft January 13, 2025 09:01
@tosky
Copy link
Author

tosky commented Jan 13, 2025

Moving back to draft while I figure it out why the tempest job does not run the expected tests

@tosky tosky force-pushed the more-zuul-and-junitxml branch from f43985e to 418a622 Compare January 13, 2025 11:26
@openshift-ci openshift-ci bot removed the lgtm label Jan 13, 2025
@tosky
Add KUTTL and tempest zuul jobs
bd8c52f 
They are meant to replace the prow ones in the long term.

Signed-off-by: Luigi Toscano <ltoscano@redhat.com>
@tosky tosky force-pushed the more-zuul-and-junitxml branch from 418a622 to bd8c52f Compare January 13, 2025 14:50
@tosky tosky marked this pull request as ready for review January 13, 2025 17:32
@openshift-ci openshift-ci bot requested a review from abays January 13, 2025 17:32
@tosky
Copy link
Author

tosky commented Jan 13, 2025

Now some volume tempest tests are correctly executed in zuul too.

@tosky tosky requested a review from fmount January 13, 2025 17:35
@fmount
Copy link
Contributor

fmount commented Jan 14, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jan 14, 2025
@fmount fmount requested a review from ASBishop January 14, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Akrog Akrog Awaiting requested review from Akrog

@viroel viroel Awaiting requested review from viroel

@abays abays Awaiting requested review from abays

@fmount fmount Awaiting requested review from fmount

@ASBishop ASBishop Awaiting requested review from ASBishop

Assignees

@fmount fmount

Labels
lgtm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tosky @fserucas @fmount