-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update Kafka versions to mirror Strimzi kafka operator repository #34
Conversation
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Do we really need Kafka 3.7.2? We will never use it anywhere else as Strimzi. |
Well, I am just updating x.y.max(z) as having a convention in it. I mean I can remove such a version but then we could have another discussion about version less than 3.7.2 (i.e.,
If we think it's useless to keep those versions then I can remove it. |
Is anyone using it? I don't think any Strimzi project uses anything else than the latest version. |
I would follow the support matrix of Strimzi, which also supports just two latest minor versions. I follow this pattern in test-clients as well - that's because we are running tests with all of the supported versions of Kafka by Strimzi. |
I thought Quarkus was using some older versions, but looking at the codebase, they are using just the latest version or the option to inject image via a constructor parameter (which would use different versions). |
So, based on my findings are we okay with shrinking those versions to mirror the |
+1 |
+1 from me as well |
1 similar comment
+1 from me as well |
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Thanks for the input I have updated the PR with description etc. |
This PR updates our Kafka versions to mirror Strimzi Kafka Operator repository-supported versions. For the past two years, we haven't seen any benefit in supporting multiple Kafka versions, as most users use the latest version. For those reasons, we decided to support only SKO versions (typically 2-3 versions).