ROSAENG-59003 | chore: Clusters cleaner prow onboarding - #82577
ROSAENG-59003 | chore: Clusters cleaner prow onboarding#82577jfrazierRedHat wants to merge 3 commits into
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughAdds master-branch CI configuration, repository ownership, and repository-scoped Prow plugin, trigger, and Tide settings for Changesrosa-clusters-cleaner repository onboarding
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@ci-operator/config/openshift-online/rosa-clusters-cleaner/openshift-online-rosa-clusters-cleaner-master.yaml`:
- Around line 16-24: Regenerate the CI Operator artifacts for the new test
configuration by running the repository’s make update/make jobs target, ensuring
generated ci-operator/jobs entries for openshift-online/rosa-clusters-cleaner
are created and committed alongside the source config.
In
`@core-services/prow/02_config/openshift-online/rosa-clusters-cleaner/_pluginconfig.yaml`:
- Around line 1-4: Update the approve configuration for repository
openshift-online/rosa-clusters-cleaner so require_self_approval is set to true,
matching the required ROSA plugin approval policy.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: fef5d687-39d1-4c65-89a2-abedc26e6d88
📒 Files selected for processing (3)
ci-operator/config/openshift-online/rosa-clusters-cleaner/openshift-online-rosa-clusters-cleaner-master.yamlcore-services/prow/02_config/openshift-online/rosa-clusters-cleaner/_pluginconfig.yamlcore-services/prow/02_config/openshift-online/rosa-clusters-cleaner/_prowconfig.yaml
| approve: | ||
| - repos: | ||
| - openshift-online/rosa-clusters-cleaner | ||
| require_self_approval: false |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Require self-approval for this ROSA repository.
Set require_self_approval to true; false does not match the ROSA GitLab review strategy and weakens the repository’s approval policy.
Proposed fix
- require_self_approval: false
+ require_self_approval: trueBased on learnings, ROSA-owned plugin configurations under core-services/prow/02_config/openshift-online must set approve.require_self_approval: true.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| approve: | |
| - repos: | |
| - openshift-online/rosa-clusters-cleaner | |
| require_self_approval: false | |
| approve: | |
| - repos: | |
| - openshift-online/rosa-clusters-cleaner | |
| require_self_approval: true |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In
`@core-services/prow/02_config/openshift-online/rosa-clusters-cleaner/_pluginconfig.yaml`
around lines 1 - 4, Update the approve configuration for repository
openshift-online/rosa-clusters-cleaner so require_self_approval is set to true,
matching the required ROSA plugin approval policy.
Source: Learnings
3564c28 to
d928f49
Compare
d928f49 to
20dfca3
Compare
d654952 to
ebc4742
Compare
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gacalixto, jfrazierRedHat 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 |
ebc4742 to
b73503f
Compare
|
New changes are detected. LGTM label has been removed. |
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/test test |
|
/retest |
|
@jfrazierRedHat hi, based on the rehearsal failure, you need to fix your Makefile's test target to point to the correct package path, likely |
|
/retest |
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
b73503f to
478e8fd
Compare
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
478e8fd to
6978d33
Compare
|
@jfrazierRedHat: |
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: your |
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse ack |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
6978d33 to
9d878a4
Compare
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
9d878a4 to
747624a
Compare
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/retest |
747624a to
31a48d7
Compare
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
31a48d7 to
40c4ba2
Compare
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse pull-ci-openshift-online-rosa-clusters-cleaner-master-test |
|
@jfrazierRedHat: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
@jfrazierRedHat: The following test failed, say
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 by CodeRabbit
This PR onboards
openshift-online/rosa-clusters-cleanerto OpenShift CI and Prow for themasterbranch:make testin asrccontainer with standard resource settings and the configured release build image.approvedandlgtmlabels and excludes blocked or invalid pull requests.