openshift/cluster-capi-operator: skip post-install test for manifests-gen changes#81452
openshift/cluster-capi-operator: skip post-install test for manifests-gen changes#81452damdo wants to merge 1 commit into
Conversation
…nstall for manifests-gen changes The e2e-aws-capi-techpreview-post-install test was missing manifests-gen/ from its pipeline_skip_if_only_changed pattern, causing it to run unnecessarily on PRs that only touch manifests-gen/ files. Update the skip pattern to match the one used by all other tests in the cluster-capi-operator config.
|
Caution Review failedAn error occurred during the review process. Please try again later. WalkthroughThis PR updates the ChangesSkip regex update
Estimated code review effort: 1 (Trivial) | ~5 minutes Related PRs: None identified. Suggested labels: ci, config Suggested reviewers: None identified. 🐰 PoemA regex hop, from branch to branch, ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: damdo The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/pj-rehearse ack |
|
@damdo: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[REHEARSALNOTIFIER]
Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals. Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@damdo: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
Summary
manifests-gen/to thepipeline_skip_if_only_changedpattern for thee2e-aws-capi-techpreview-post-installtest inopenshift/cluster-capi-operatorconfigs (main, 4.23, 5.0, 5.1)manifests-gen/filesTest plan
e2e-aws-capi-techpreview-post-installis skipped on a manifests-gen-only PR after mergeSummary by CodeRabbit
Updated the
openshift/cluster-capi-operatorCI configuration acrossmain,4.23,5.0, and5.1so thee2e-aws-capi-techpreview-post-installjob now skips runs when onlymanifests-gen/files change.This aligns that job’s
pipeline_skip_if_only_changedbehavior with the rest of the cluster-capi-operator test suite, preventing unnecessary execution on manifests-gen-only PRs.