Skip to content

fix: set KonnectGatewayControlPlane endpoints after create - #3474

Draft
pmalek wants to merge 2 commits into
mainfrom
set-konnectgatewaycontrolplane-endpoints-after-create
Draft

fix: set KonnectGatewayControlPlane endpoints after create#3474
pmalek wants to merge 2 commits into
mainfrom
set-konnectgatewaycontrolplane-endpoints-after-create

Conversation

@pmalek

@pmalek pmalek commented Mar 3, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it:

Set the KonnectGatewayControlPlane endpoints immediately after create not waiting until next reconciliation and subsequent update in

switch e := any(ent).(type) {
case *configurationv1.KongConsumer:
updated, isProblem = handleKongConsumerSpecific(ctx, cl, e)
case *konnectv1alpha2.KonnectGatewayControlPlane:
updated, err = handleKonnectGatewayControlPlaneSpecific(ctx, sdk, e)
if err != nil {
return false, ctrl.Result{}, err
}
default:
}

This allows to get the KonnectGatewayControlPlane status filled in the same reconciliation, not forcing the requeue and handling that in handleTypeSpecific. Moreover this saves an API Konnect call (ops.GetControlPlaneByID) which was required there.

Which issue this PR fixes

Fixes #

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect significant changes

@pmalek pmalek self-assigned this Mar 3, 2026
@pmalek
pmalek force-pushed the set-konnectgatewaycontrolplane-endpoints-after-create branch 2 times, most recently from febdf72 to c9975ee Compare March 3, 2026 17:13
@pmalek
pmalek force-pushed the set-konnectgatewaycontrolplane-endpoints-after-create branch 3 times, most recently from eac4b6f to 0899100 Compare March 4, 2026 21:09
@pmalek
pmalek force-pushed the set-konnectgatewaycontrolplane-endpoints-after-create branch from 0899100 to 2b8250c Compare March 17, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant