Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(konnect): add KonnectDataPlaneGroupConfiguration reconciler #1186

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Feb 20, 2025

What this PR does / why we need it:

This PR adds support for KonnectDataPlaneGroupConfiguration CRD (added in Kong/kubernetes-configuration#307)

Some notes:

  • cloud gateway network ID is only allowed to be provided via Konnect
    • this is planned to be expanded upon demand to support namespaced ref
  • the API (https://docs.konghq.com/konnect/api/cloud-gateways/latest/#/Data-Plane%20Group%20Configurations/create-configuration) uses a PUT verb and enforces a single configuration against a Cloud Gateway. This means that only 1 KonnectCloudGatewayDataPlaneGroupConfiguration can be applied to be effective, as using more than 1 against a single Control Plane will result in configurations overriding each other
    • unless the API changes this is not planned to change

Which issue this PR fixes

Fixes #1114

Special notes for your reviewer:

Required changes from Kong/kubernetes-configuration#307

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 added this to the KGO v1.5.x milestone Feb 20, 2025
@pmalek pmalek self-assigned this Feb 20, 2025
@pmalek pmalek force-pushed the konnect-add-dataplanegroupconfiguration-reconciler branch 7 times, most recently from 8595fb9 to 4dc5f3c Compare February 21, 2025 10:41
@pmalek pmalek marked this pull request as ready for review February 21, 2025 10:41
@pmalek pmalek requested a review from a team as a code owner February 21, 2025 10:41
Copy link
Member

@programmer04 programmer04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall LGTM, some nits

@pmalek pmalek force-pushed the konnect-add-dataplanegroupconfiguration-reconciler branch from adb4c1b to 615a938 Compare February 21, 2025 13:02
@pmalek pmalek requested a review from programmer04 February 21, 2025 13:02
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.

Add Dedicated Cloud Gateways DataPlane group configuration reconciler
2 participants