migrate strimzi duck types and client bindings from v1beta2 to v1#6604
Merged
squakez merged 3 commits intoapache:mainfrom May 5, 2026
Merged
migrate strimzi duck types and client bindings from v1beta2 to v1#6604squakez merged 3 commits intoapache:mainfrom
squakez merged 3 commits intoapache:mainfrom
Conversation
… Kafka resources Kafka setup fails with errors like 'no matches for kind KafkaNodePool' because Strimzi CRDs are not fully established when Kafka resources are applied. Added explicit waits for CRDs and operator readiness to prevent race condition. Also updated KEDA installation to use server-side apply to avoid annotation size issues.
…a2 to v1 Strimzi v1beta2 APIs are no longer supported and have been replaced by v1. This commit migrates the Camel-K strimzi duck types from `v1beta2` to `v1`. - Renames duck types package and references from `v1beta2` to `v1` - Updates schema registration to point to the `v1` API group - Regenerates Kubernetes clientset and deepcopy functions for `strimzi/v1` - Refactors `pkg/util/bindings` provider to utilize the new `KafkaV1` client
squakez
approved these changes
May 4, 2026
Contributor
squakez
left a comment
squakez
left a comment
There was a problem hiding this comment.
LGTM, thanks for the contribution!
This was
linked to
issues
May 4, 2026
Closed
Contributor
Author
Your welcome, |
Contributor
|
We're always happy to receive contributions :) Feel free to have a look at "good first contributions" labeled issues: https://github.com/apache/camel-k/issues?q=is%3Aissue%20state%3Aopen%20label%3A%22good%20first%20issue%22 |
Contributor
|
There's some minor lint issue: you can locally run |
squakez
approved these changes
May 4, 2026
|
|
||
| kubectl create namespace kafka | ||
| kubectl create -f 'https://strimzi.io/install/latest?namespace=kafka' -n kafka | ||
| kubectl create namespace kafka --dry-run=client -o yaml | kubectl apply -f - |
Contributor
There was a problem hiding this comment.
Noticed this one. Why not creating the namespace normally as before?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue : #6596
Renamed pkg/apis/duck/strimzi/v1beta2 package to v1.
Updated schema registration and binding resolution (KafkaV1beta2 to KafkaV1).
Regenerated Kubernetes clientsets, informers, and deepcopy functions.
Testing Unit tests in pkg/util/bindings successfully pass using the updated KafkaV1 mocks: go test -v ./pkg/util/bindings -run Strimzi