ARO-HCP: Add INT westus3 e2e periodic job - #82884
Conversation
|
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 selected for processing (2)
WalkthroughThe periodic configuration adds a westus3 integration E2E job. Boskos configuration adds one free westus3 shard slot for the job. ChangesARO-HCP westus3 integration
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
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/Azure/ARO-HCP/Azure-ARO-HCP-main__periodic.yaml`:
- Around line 34-51: Run make update to regenerate CI outputs for the new
integration-e2e-parallel-westus3 periodic job, including any updated
zz_generated_metadata and generated Prow job definitions. Include all generated
changes in the commit so the new job is scheduled consistently with its source
configuration.
🪄 Autofix
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: aa928cec-cff6-44d9-a881-5eba5c9f9e65
⛔ Files ignored due to path filters (1)
ci-operator/jobs/Azure/ARO-HCP/Azure-ARO-HCP-main-periodics.yamlis excluded by!ci-operator/jobs/**
📒 Files selected for processing (1)
ci-operator/config/Azure/ARO-HCP/Azure-ARO-HCP-main__periodic.yaml
|
/pj-rehearse periodic-ci-Azure-ARO-HCP-main-periodic-integration-e2e-parallel-westus3 |
|
@geoberle: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
Add integration-e2e-parallel-westus3 test variant targeting westus3 region for INT environment e2e testing.
a3f2311 to
ccd51ba
Compare
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/lgtm |
|
/pj-rehearse periodic-ci-Azure-ARO-HCP-main-periodic-integration-e2e-parallel-westus3 |
|
@geoberle: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/rehearsals-ack |
|
/pj-rehearse ack |
|
@geoberle: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: geoberle, Prucek, roivaz 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 |
|
@geoberle: 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. |
|
/override periodic-ci-Azure-ARO-HCP-main-periodic-integration-e2e-parallel-westus3 |
|
@geoberle: geoberle unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers openshift-sustaining-engineers. 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 kubernetes-sigs/prow repository. |
cb7decc
into
openshift:main
|
@geoberle: Updated the
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 kubernetes-sigs/prow repository. |
* Add INT westus3 e2e periodic job Add integration-e2e-parallel-westus3 test variant targeting westus3 region for INT environment e2e testing. * boskos config for aro-hcp int westus3 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com> --------- Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
* Add INT westus3 e2e periodic job Add integration-e2e-parallel-westus3 test variant targeting westus3 region for INT environment e2e testing. * boskos config for aro-hcp int westus3 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com> --------- Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
PR #82884 added the westus3 e2e job only as a periodic, making it unreachable via gangway as a postsubmit. Add the matching postsubmit entry so the job can be triggered with job_execution_type=2.
* Add INT westus3 e2e periodic job Add integration-e2e-parallel-westus3 test variant targeting westus3 region for INT environment e2e testing. * boskos config for aro-hcp int westus3 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com> --------- Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
PR openshift#82884 added the westus3 e2e job only as a periodic, making it unreachable via gangway as a postsubmit. Add the matching postsubmit entry so the job can be triggered with job_execution_type=2.
Add integration-e2e-parallel-westus3 test variant targeting westus3 region for INT environment e2e testing.
Summary by CodeRabbit
Adds the
integration-e2e-parallel-westus3periodic CI job for the Azure Red Hat OpenShift main repository. Configures one free Boskos INT resource slot inwestus3for the job.