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.

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

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

Merged

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

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 tosky force-pushed the more-zuul-and-junitxml branch from 190ff23 to b640431 Compare December 24, 2024 16:32
@tosky
Copy link
Contributor Author

tosky commented Jan 2, 2025

recheck

1 similar comment
@tosky
Copy link
Contributor Author

tosky commented Jan 2, 2025

recheck

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/221c1d8cef254360b7773aa19f68653c

openstack-k8s-operators-content-provider FAILURE in 4m 31s
⚠️ 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)

@tosky
Copy link
Contributor Author

tosky commented Jan 3, 2025

recheck

3 similar comments
@tosky
Copy link
Contributor Author

tosky commented Jan 7, 2025

recheck

@tosky
Copy link
Contributor Author

tosky commented Jan 7, 2025

recheck

@tosky
Copy link
Contributor Author

tosky commented Jan 7, 2025

recheck

tosky added 2 commits January 8, 2025 00:40
- 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>
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 b640431 to 34557ae Compare January 7, 2025 23:40
@tosky tosky marked this pull request as ready for review January 7, 2025 23:41
@openshift-ci openshift-ci bot requested review from konan-abhi and stuggi January 7, 2025 23:42
@tosky tosky requested review from fmount and removed request for stuggi and konan-abhi January 7, 2025 23:42
@tosky
Copy link
Contributor Author

tosky commented Jan 7, 2025

recheck

@fmount
Copy link
Contributor

fmount commented Jan 8, 2025

Thanks @tosky for starting this. It's very useful.
I see zuul based kuttl tests are failing, mostly because of missing resources of the openshift environment [1], which fails to schedule Glance Pods on a scale out operation.
I'll recheck to see if it's consistent, but in general that is a signal that we might want to review the amount of resources assigned to the crc VM.

[1] https://logserver.rdoproject.org/78/678/34557ae2934d5fe0fe2e9a1671d75d90b0a74e20/github-check/glance-operator-kuttl/bc30e93/controller/ci-framework-data/logs/openstack-k8s-operators-openstack-must-gather/namespaces/glance-kuttl-tests/events.log

@fmount
Copy link
Contributor

fmount commented Jan 8, 2025

recheck

@tosky
Copy link
Contributor Author

tosky commented Jan 8, 2025

One of the errors may not be linked to resource issues though:

<testcase classname="tests" name="glance_single" timestamp="2025-01-07T19:38:42.296545869-05:00" time="2.420" assertions="8">
<failure message="failed in step 1-deploy_glance" type="">
command "oc kustomize ../../../../config/samples/layout/single | oc apply -n..." failed, exit status 1
</failure>
</testcase>

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.

/approve
/lgtm

Copy link
Contributor

openshift-ci bot commented Jan 9, 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-ci openshift-ci bot added the approved label Jan 9, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit c5dfff0 into openstack-k8s-operators:main Jan 9, 2025
8 checks passed
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.

2 participants