[release-4.22] OCPBUGS-98627: Skip Azure topology LB scope override for ARO HCP IngressController - #8998
Conversation
…logy PR openshift#7821 (CNTRLPLANE-2172) added code to force InternalLoadBalancer scope on the guest-cluster IngressController for Azure clusters with Private or PublicAndPrivate topology. This was correct for self-managed Azure but also affected ARO HCP clusters, which use shared ingress and should keep ExternalLoadBalancer scope (or defer to the annotation set by Cluster Service). Add an IsAroHCPByHCP guard so the topology-based scope override only applies to self-managed Azure. ARO HCP clusters now respect the IngressControllerLoadBalancerScope annotation propagated from the HostedCluster, matching the pattern already used in router.go. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Cover the ARO HCP guard added in the previous commit: - PublicAndPrivate topology → ExternalLoadBalancer (not Internal) - Private topology → ExternalLoadBalancer (shared ingress) - Annotation override to Internal is still respected Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@openshift-cherrypick-robot: Detected clone of Jira Issue OCPBUGS-98571 with correct target version. Will retitle the PR to link to the clone. 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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@openshift-cherrypick-robot: This pull request references Jira Issue OCPBUGS-98627, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. 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. |
|
/approve clean backport |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bryan-cox, openshift-cherrypick-robot 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 |
|
/jira refresh |
|
@bryan-cox: This pull request references Jira Issue OCPBUGS-98627, which is valid. The bug has been moved to the POST state. 7 validation(s) were run on this bug
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. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release-4.22 #8998 +/- ##
=============================================
Coverage 35.87% 35.87%
=============================================
Files 774 774
Lines 94752 94753 +1
=============================================
+ Hits 33993 33997 +4
+ Misses 57979 57977 -2
+ Partials 2780 2779 -1
🚀 New features to boost your workflow:
|
|
/hold |
|
/test e2e-aks |
|
/pipeline required |
|
Scheduling tests matching the |
|
/uncc |
|
/hold cancel |
|
/test e2e-aws |
|
/verified later @bryan-cox |
|
@csrwng: This PR has been marked to be verified later by 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. |
|
@openshift-cherrypick-robot: 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. |
fb3b1a3
into
openshift:release-4.22
|
@openshift-cherrypick-robot: Jira Issue OCPBUGS-98627: All pull requests linked via external trackers have merged: This pull request has 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 openshift-eng/jira-lifecycle-plugin repository. |
Add AWS CPO image overrides for 4.22.0-4.22.6 to fix KMS key rotation and related issues. 4.22.7 does not need an override: both PRs (openshift#8972, openshift#8998) merged before the 4.22.7 development cutoff (2026-07-22). - 4.22.0-4.22.6: PR openshift#8972, PR openshift#8998 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add AWS CPO image overrides for 4.22.0-4.22.6 to fix KMS key rotation and related issues. 4.22.7 does not need an override: both PRs (openshift#8972, openshift#8998) merged before the 4.22.7 development cutoff (2026-07-22). - 4.22.0-4.22.6: PR openshift#8972, PR openshift#8998 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Fix included in release 4.22.0-0.nightly-2026-07-15-214410 |
Add Azure CPO image overrides to fix KMS key rotation (OCPBUGS-98220, PR #8972) and ARO HCP ingress LB scope (OCPBUGS-98627, PR #8998). This supersedes the previous CNTRLPLANE-3619/3656 overrides for 4.22.0-4.22.3: the new Konflux image (commit fb3b1a3) includes all four PRs (#8721, #8790, #8972, #8998). 4.22.7 does not need an override: all PRs merged before the development cutoff (2026-07-22). Refs: OCPBUGS-98695 Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com> Commit-Message-Assisted-by: Claude (via Claude Code)
Add Azure CPO image overrides to fix KMS key rotation (OCPBUGS-98220, PR openshift#8972) and ARO HCP ingress LB scope (OCPBUGS-98627, PR openshift#8998). This supersedes the previous CNTRLPLANE-3619/3656 overrides for 4.22.0-4.22.3: the new Konflux image (commit fb3b1a3) includes all four PRs (openshift#8721, openshift#8790, openshift#8972, openshift#8998). 4.22.7 does not need an override: all PRs merged before the development cutoff (2026-07-22). Refs: OCPBUGS-98695 Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com> Commit-Message-Assisted-by: Claude (via Claude Code)
Add Azure CPO image overrides to fix KMS key rotation (OCPBUGS-98220, PR openshift#8972) and ARO HCP ingress LB scope (OCPBUGS-98627, PR openshift#8998). This supersedes the previous CNTRLPLANE-3619/3656 overrides for 4.22.0-4.22.3: the new Konflux image (commit fb3b1a3) includes all four PRs (openshift#8721, openshift#8790, openshift#8972, openshift#8998). 4.22.7 does not need an override: all PRs merged before the development cutoff (2026-07-22). Refs: OCPBUGS-98695 Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com> Commit-Message-Assisted-by: Claude (via Claude Code)
This is an automated cherry-pick of #8992
/assign openshift-ci-robot