Skip to content

Conversation

@programmer04
Copy link
Member

@programmer04 programmer04 commented Aug 28, 2025

What this PR does / why we need it:

Without this fix, an attempt to apply, e.g. controlplane-dataplane-watchnamespaces.yaml ends with a failure like that

{"level":"error","ts":"2025-08-28T13:41:09Z","msg":"Reconciler error","controller":"controlplane","controllerGroup":"gateway-operator.konghq.com","controllerKind":"ControlPlane","ControlPlane":{"name":"controlplane-example","namespace":"default"},"namespace":"default","name":"controlplane-example","reconcileID":"f09b58aa-82b4-46b8-8169-3b82009292f2","error":"failed to patch ControlPlane status with DataPlane name dataplane-cp-watchnamespace: ControlPlane.gateway-operator.konghq.com \"controlplane-example\" is invalid: spec: Invalid value: \"object\": When dataplane target is of type 'ref' the ingressClass must be set","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).reconcileHandler\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:353\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).processNextWorkItem\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:300\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).Start.func2.1\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:202"}

@programmer04 programmer04 added this to the KO v2.0.x milestone Aug 28, 2025
@programmer04 programmer04 self-assigned this Aug 28, 2025
@programmer04 programmer04 force-pushed the fix-cp-conversion branch 2 times, most recently from dc798c2 to b30929b Compare August 28, 2025 14:01
@programmer04 programmer04 marked this pull request as ready for review August 28, 2025 14:08
@programmer04 programmer04 requested a review from a team as a code owner August 28, 2025 14:08
@programmer04 programmer04 enabled auto-merge August 28, 2025 14:09
Copy link
Member

@pmalek pmalek left a comment

Choose a reason for hiding this comment

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

@programmer04 programmer04 added this pull request to the merge queue Aug 28, 2025
Merged via the queue into main with commit d9be39c Aug 28, 2025
26 checks passed
@programmer04 programmer04 deleted the fix-cp-conversion branch August 28, 2025 14:21
@fgiudici fgiudici mentioned this pull request Aug 28, 2025
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants