CNTRLPLANE-2850: hypershift: Add aws v2 periodics#81878
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:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (3)
WalkthroughAdds the ChangesHypershift AWS E2E v2 scheduling
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
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 |
|
/pj-rehearse |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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/hypershift/openshift-hypershift-release-4.22__periodics.yaml`:
- Around line 147-153: The generated Prow artifacts are missing for both new v2
AWS periodic jobs. Regenerate and validate the jobs defined in
ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.22__periodics.yaml
lines 147-153 and
ci-operator/config/openshift/hypershift/openshift-hypershift-release-5.0__periodics.yaml
lines 163-169 using make registry-metadata && make ci-operator-config && make
jobs, ensuring both e2e-v2-aws definitions produce corresponding generated jobs.
🪄 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: Enterprise
Run ID: 0f245720-a4e9-4f68-9298-3b1bc9950a7a
⛔ Files ignored due to path filters (2)
ci-operator/jobs/openshift/hypershift/openshift-hypershift-release-4.22-periodics.yamlis excluded by!ci-operator/jobs/**ci-operator/jobs/openshift/hypershift/openshift-hypershift-release-5.0-periodics.yamlis excluded by!ci-operator/jobs/**
📒 Files selected for processing (4)
ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.22__periodics.yamlci-operator/config/openshift/hypershift/openshift-hypershift-release-5.0__periodics.yamlcore-services/release-controller/_releases/release-ocp-4.22-ci.jsoncore-services/release-controller/_releases/release-ocp-5.0-ci.json
|
@ironcladlou: your |
|
/pj-rehearse |
|
@ironcladlou: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
468ca7d to
b666710
Compare
|
/pj-rehearse |
|
@ironcladlou: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
b666710 to
33765f4
Compare
|
/pj-rehearse |
|
@ironcladlou: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/retitle CNTRLPLANE-2850: hypershift: Add aws v2 periodics |
|
@ironcladlou: This pull request references CNTRLPLANE-2850 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
33765f4 to
37cec54
Compare
|
/pj-rehearse |
|
@ironcladlou: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/approve |
|
/rehearsals-ack |
|
/pj-rehearse ack |
|
@ironcladlou: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
Add the hypershift AWS v2 periodics to begin collecting baseline signal. Fixes https://redhat.atlassian.net/browse/CNTRLPLANE-2850
37cec54 to
cbc9b3b
Compare
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse skip |
|
@ironcladlou: 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: bryan-cox, csrwng, ironcladlou 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 |
|
@ironcladlou: 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. |
Add the hypershift AWS v2 periodics to begin collecting baseline signal. Fixes https://redhat.atlassian.net/browse/CNTRLPLANE-2850
Add the hypershift AWS v2 periodics to begin collecting baseline signal.
Fixes https://redhat.atlassian.net/browse/CNTRLPLANE-2850
Summary by CodeRabbit
Adds scheduled HyperShift AWS v2 baseline CI jobs for the 4.22, 4.23, and 5.0 release streams. Each job runs four times daily on the
hypershift-awscluster profile using thehypershift-aws-e2e-v2workflow, expanding upgrade verification coverage for these release configurations.