Skip to content

ARO-HCP: Add INT westus3 e2e postsubmit job - #83267

Open
geoberle wants to merge 1 commit into
openshift:mainfrom
geoberle:worktree-add-westus3-postsubmit-v2
Open

ARO-HCP: Add INT westus3 e2e postsubmit job#83267
geoberle wants to merge 1 commit into
openshift:mainfrom
geoberle:worktree-add-westus3-postsubmit-v2

Conversation

@geoberle

@geoberle geoberle commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

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.

Summary by CodeRabbit

Adds an INT westus3 ARO-HCP e2e postsubmit job. The job uses the persistent E2E workflow and sets ALLOWED_LOCATIONS=westus3.

The job is reachable through Gangway with job_execution_type=2.

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.
@roivaz

roivaz commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

/approve
/lgtm

@openshift-ci

openshift-ci Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geoberle, roivaz

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

The pull request process is described 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 openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 11, 2026
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 11, 2026
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@geoberle: pj-rehearse could not automatically process this event because the request waited in queue for longer than 5 minutes. Use /pj-rehearse to trigger rehearsals manually.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 93c8b63e-4048-4d94-9d83-a841b3cd666c

📥 Commits

Reviewing files that changed from the base of the PR and between 52a5dd4 and f1fd1d1.

⛔ Files ignored due to path filters (1)
  • ci-operator/jobs/Azure/ARO-HCP/Azure-ARO-HCP-main-postsubmits.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (1)
  • ci-operator/config/Azure/ARO-HCP/Azure-ARO-HCP-main__e2e.yaml

Walkthrough

The E2E configuration adds a postsubmit integration job for the public deployment in westus3. The job uses the persistent E2E workflow and sets westus3 as the allowed location.

Changes

West US 3 E2E integration

Layer / File(s) Summary
West US 3 postsubmit job
ci-operator/config/Azure/ARO-HCP/Azure-ARO-HCP-main__e2e.yaml
Adds the integration-e2e-parallel-westus3 postsubmit job with persistent E2E workflow settings, westus3 configuration, concurrency controls, failure reporting, and GCP capability.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: roivaz


Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Container-Privileges ❓ Inconclusive The initial repository query returned no diff, so the changed manifest content is not yet verified. Inspect the target YAML and its repository revision to verify whether the added job contains any flagged privilege settings.
✅ Passed checks (14 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the addition of the INT westus3 ARO-HCP E2E postsubmit job.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed The PR adds only static CI job identifiers and YAML configuration; it introduces no Ginkgo It(), Describe(), Context(), or When() test titles.
Test Structure And Quality ✅ Passed Not applicable: the pull request changes only CI YAML and generated job YAML; no Ginkgo It blocks or cluster-test code require review under these criteria.
Microshift Test Compatibility ✅ Passed The PR changes only CI YAML configuration and generated job metadata; it adds no Ginkgo tests or test code that requires MicroShift compatibility review.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The pull request adds only CI YAML job configuration; it adds no Ginkgo tests or test code requiring SNO compatibility review.
Topology-Aware Scheduling Compatibility ✅ Passed The PR changes only CI test configuration and a generated Prow postsubmit job; no deployment, operator, controller, or topology-sensitive scheduling constraints were added.
Ote Binary Stdout Contract ✅ Passed The patch changes only two CI YAML files; it adds no OTE binary or process-level code and no stdout writes.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The PR changes only E2E and generated postsubmit YAML; it adds no Ginkgo test declarations, IPv4 literals, or external-network test logic.
No-Weak-Crypto ✅ Passed The 104 added lines only define the westus3 e2e job and generated Prow configuration; no weak algorithms, custom crypto, or secret comparisons appear.
No-Sensitive-Data-In-Logs ✅ Passed Added YAML contains only region/profile values and standard secret references; no literal credentials. The persistent command disables xtrace before reading or using the client secret.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@geoberle

Copy link
Copy Markdown
Contributor Author

/pj-rehearse branch-ci-Azure-ARO-HCP-main-e2e-integration-e2e-parallel-westus3

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@geoberle: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci

openshift-ci Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

@geoberle: 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.

@geoberle

Copy link
Copy Markdown
Contributor Author

/pj-rehearse list

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@geoberle: job(s): branch-ci-Azure-ARO-HCP-main-e2e-integration-e2e-parallel-westus3 either don't exist or were not found to be affected, and cannot be rehearsed

@geoberle

Copy link
Copy Markdown
Contributor Author

/pj-rehearse ack

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@geoberle: your /pj-rehearse request was not processed because the request waited in queue for longer than 5 minutes. Please retry in a few minutes.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants