CNTRLPLANE-4016: Add hypershift e2e-aws-mgmt-techpreview CI jobs with IPI management cluster - #83096
CNTRLPLANE-4016: Add hypershift e2e-aws-mgmt-techpreview CI jobs with IPI management cluster#83096mgencur wants to merge 1 commit into
Conversation
…luster Add a new hypershift-aws-e2e-ipi-mgmt workflow that provisions a standalone OpenShift management cluster via IPI on AWS (2-tier model) instead of the standard 3-tier nested management cluster. This enables testing HyperShift with FEATURE_SET=TechPreviewNoUpgrade on the management cluster for CAPI compatibility validation. Jobs added: - Optional presubmit on main and release-5.1 - Daily periodic on release-5.1 (cron: 0 3 * * *) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@mgencur: This pull request explicitly references no jira issue. 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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mgencur The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
WalkthroughThe change adds an AWS IPI management-cluster workflow for HyperShift and registers an optional TechPreview test across main, release, and daily periodic configurations. ChangesHyperShift AWS management E2E
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant PeriodicJob
participant AWSIPIManagementCluster
participant HyperShift
participant NestedE2ETests
PeriodicJob->>AWSIPIManagementCluster: Provision AWS IPI management cluster
AWSIPIManagementCluster->>HyperShift: Install HyperShift and resolve releases
HyperShift->>NestedE2ETests: Run nested e2e tests
NestedE2ETests-->>AWSIPIManagementCluster: Gather diagnostics
Suggested reviewers: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/hold |
|
[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: |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-5.1-periodics-e2e-aws-mgmt-techpreview |
|
@mgencur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
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-main.yaml`:
- Around line 353-354: The three management-job environment blocks require the
installer input TECH_PREVIEW_NO_UPGRADE set to "true"; FEATURE_SET alone is
insufficient. Add TECH_PREVIEW_NO_UPGRADE: "true" in
ci-operator/config/openshift/hypershift/openshift-hypershift-main.yaml:353-354,
ci-operator/config/openshift/hypershift/openshift-hypershift-release-5.1.yaml:340-341,
and
ci-operator/config/openshift/hypershift/openshift-hypershift-release-5.1__periodics.yaml:242-243,
then run make update.
🪄 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: 04625010-3f95-45d7-a8fb-41713719a3a4
⛔ Files ignored due to path filters (3)
ci-operator/jobs/openshift/hypershift/openshift-hypershift-main-presubmits.yamlis excluded by!ci-operator/jobs/**ci-operator/jobs/openshift/hypershift/openshift-hypershift-release-5.1-periodics.yamlis excluded by!ci-operator/jobs/**ci-operator/jobs/openshift/hypershift/openshift-hypershift-release-5.1-presubmits.yamlis excluded by!ci-operator/jobs/**
📒 Files selected for processing (6)
ci-operator/config/openshift/hypershift/openshift-hypershift-main.yamlci-operator/config/openshift/hypershift/openshift-hypershift-release-5.1.yamlci-operator/config/openshift/hypershift/openshift-hypershift-release-5.1__periodics.yamlci-operator/step-registry/hypershift/aws/e2e/ipi-mgmt/OWNERSci-operator/step-registry/hypershift/aws/e2e/ipi-mgmt/hypershift-aws-e2e-ipi-mgmt-workflow.metadata.jsonci-operator/step-registry/hypershift/aws/e2e/ipi-mgmt/hypershift-aws-e2e-ipi-mgmt-workflow.yaml
|
/pj-rehearse pull-ci-openshift-hypershift-main-e2e-aws-mgmt-techpreview The release-5.1-periodics-e2e-aws-mgmt-techpreview failed because there's no suitable CI stream for 5.1 yet. |
|
@mgencur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
@mgencur: This pull request references CNTRLPLANE-4016 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. |
|
@mgencur: The following tests 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. |
|
/uncc |
https://redhat.atlassian.net/browse/CNTRLPLANE-4016
Summary
Adds a new
hypershift-aws-e2e-ipi-mgmtworkflow ande2e-aws-mgmt-techpreviewtest jobs to validate HyperShift e2e tests running against an IPI-provisioned management cluster withFEATURE_SET=TechPreviewNoUpgradefor CAPI management compatibility.Changes
hypershift-aws-e2e-ipi-mgmtin the step registry — provisions a standalone OpenShift cluster via IPI on AWS (2-tier model) instead of the standard 3-tier nested management cluster, installs the HyperShift operator, and runs the nested e2e test suitemainbranch (/test e2e-aws-mgmt-techpreview)release-5.1branch (/test e2e-aws-mgmt-techpreview)release-5.1(cron:0 3 * * *)All jobs use
cluster_profile: openshift-org-aws,FEATURE_SET: TechPreviewNoUpgradeon the management cluster,ENABLE_HYPERSHIFT_CERT_ROTATION_SCALE: "true", and nodepool version testing (N-1 through N-4).Always review AI generated responses prior to use.
Generated with Claude Code via openshift-developer plugin
Summary by CodeRabbit
Adds optional and periodic
e2e-aws-mgmt-techpreviewCI jobs for HyperShift onmainandrelease-5.1.The jobs use the
openshift-org-awscluster profile and thehypershift-aws-e2e-ipi-mgmtworkflow. The workflow provisions an AWS IPI management cluster, installs HyperShift, and runs nested TechPreviewNoUpgrade e2e tests.The tests cover NodePool versions 4.22 through 4.19. The
release-5.1periodic job runs daily at 03:00 UTC. Certificate rotation scaling is enabled.