Skip to content

config: add stolostron/capi-tests CAPZ e2e jobs#3377

Open
RadekCap wants to merge 2 commits intoopenshift:mainfrom
RadekCap:add-capz-qe-jobs
Open

config: add stolostron/capi-tests CAPZ e2e jobs#3377
RadekCap wants to merge 2 commits intoopenshift:mainfrom
RadekCap:add-capz-qe-jobs

Conversation

@RadekCap
Copy link
Copy Markdown

@RadekCap RadekCap commented Mar 27, 2026

Summary

  • Add capz-qe section to openshift-customizations.yaml to register stolostron/capi-tests CAPZ e2e jobs in Sippy
  • Registers the periodic job (periodic-ci-stolostron-capi-tests-configure-prow-mgmt-periodics-capz-e2e and pull-ci-stolostron-capi-tests-configure-prow-mgmt-capz-e2e) by exact name

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Enabled CAPZ end-to-end testing in CI: automated E2E validation for CAPZ components now runs on pull requests and periodic schedules to improve QA coverage and catch regressions earlier.

Register the CAPZ QE periodic and presubmit jobs in Sippy so they
appear in the dashboard.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci-robot
Copy link
Copy Markdown

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: automatic mode

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 034c2b85-bf3b-4cdc-b1fb-2576526a3cd4

📥 Commits

Reviewing files that changed from the base of the PR and between 5c5db95 and 5670282.

📒 Files selected for processing (1)
  • config/openshift-customizations.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • config/openshift-customizations.yaml

Walkthrough

Added a new top-level releases.capz-qe section to config/openshift-customizations.yaml enabling two CAPZ-related jobs by setting their jobs[...] entries to true.

Changes

Cohort / File(s) Summary
Release Configuration
config/openshift-customizations.yaml
Added releases.capz-qe release entry and set job enablement flags: releases.capz-qe.jobs.periodic-ci-stolostron-capi-tests-configure-prow-mgmt-periodics-capz-e2e: true and releases.capz-qe.jobs.pull-ci-stolostron-capi-tests-configure-prow-mgmt-capz-e2e: true.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Go Error Handling ✅ Passed The PR only modifies config/openshift-customizations.yaml, a YAML configuration file, with no Go source code present, making the Go Error Handling check not applicable.
Sql Injection Prevention ✅ Passed Pull request modifies only a YAML configuration file with job names used for map lookups and string comparisons, not SQL query construction, so no SQL injection vulnerabilities are present.
Excessive Css In React Should Use Styles ✅ Passed PR modifies only config/openshift-customizations.yaml for CAPZ e2e jobs, not React components.
Single Responsibility And Clear Naming ✅ Passed PR modifies only configuration files and documentation with no code changes, so the custom check for Single Responsibility and Clear Naming does not apply.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci openshift-ci bot requested review from deepsm007 and sosiouxme March 27, 2026 09:31
@openshift-ci-robot
Copy link
Copy Markdown

Scheduling required tests:
/test e2e

Copy link
Copy Markdown

@marek-veber marek-veber 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 Indicates that a PR is ready to be merged. label Mar 27, 2026
@RadekCap
Copy link
Copy Markdown
Author

/assign dgoodwin

@RadekCap
Copy link
Copy Markdown
Author

/retest

Replace the overly broad regexp with an exact job entry for
pull-ci-stolostron-capi-tests-configure-prow-mgmt-capz-e2e.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 30, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Mar 30, 2026

New changes are detected. LGTM label has been removed.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Mar 30, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: marek-veber, RadekCap
Once this PR has been reviewed and has the lgtm label, please ask for approval from dgoodwin. For more information see the Code Review Process.

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

Details 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-robot
Copy link
Copy Markdown

Scheduling required tests:
/test e2e

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Mar 30, 2026

@RadekCap: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

@RadekCap
Copy link
Copy Markdown
Author

Can anybody please do review here? Thank you. This is currently blocking our next work.

I wouldn't ping you all but I don't know where to ask properly and I don't want to ping you personally on Slack or elsewhere.

@deads2k
@deepsm007
@dgoodwin
@neisw
@petr-muller
@stbenjam
@xueqzhan
@sosiouxme
@smg247

periodic-ci-Azure-ARO-HCP-main-periodic-prod-brazilsouth-e2e-parallel: true
periodic-ci-Azure-ARO-HCP-main-periodic-prod-centralindia-e2e-parallel: true
periodic-ci-Azure-ARO-HCP-main-periodic-prod-uksouth-e2e-parallel: true
capz-qe:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Specifying 'capz-qe' here will not create a release. You should open a card with TRT to discuss what you need.

For the 'pull' job you could add it to the presubmits section.

Even then we likely need to update the test suites to include github.com/rcap/capi-tests/test

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants