Enhancement CNTRLPLANE-3527: Add Router to ServicePublishingStrategy for non-cloud platform support#2024
Conversation
|
Skipping CI for Draft Pull Request. |
|
@vsolanki12: This pull request references CNTRLPLANE-3527 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 epic 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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
…atforms Signed-off-by: Vimal Solanki <vsolanki@redhat.com>
cd86889 to
1cf0499
Compare
Why was this change needed?
The HCP private router Service is unconditionally created as
LoadBalancerregardless of platform.On management clusters without cloud load-balancer support (Agent, KubeVirt, None platforms), the
Service stays
Pendingindefinitely, blocking route status propagation and KAS service resolution.OCPBUGS-77856 identified this issue. PR openshift/hypershift#8439 provided an initial fix using
platform auto-detection, but was closed after review identified that ClusterIP used for NodePort
services is not externally reachable.
What changes are being made?
This enhancement proposes adding
Routeras a newServiceTypeinspec.services[], extendingthe existing
ServicePublishingStrategyMappingpattern to cover the private router. This givesoperators explicit control over the service type and external address used to reach HCP routes.
Key changes:
RouterServiceType constant in the HostedCluster APINodePortPublishingStrategy(address + port fields)Routerentry defaults toLoadBalancerReferences