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

Use Docker official Apache Zookeeper image for OpenShift instead of Debezium version #891

Conversation

michalvavrik
Copy link
Member

@michalvavrik michalvavrik commented Oct 2, 2023

Summary

See quarkus-qe/quarkus-test-suite#1431 (comment), I'm not sure about versioning at all, but I gave up searching for it after 15 mins. Can't see any compatibility matrix, IIRC some version of Zookeeper used to be shipped with Strimzi and now something. Let's verify it experimentally.

Please check the relevant options

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Dependency update
  • Refactoring
  • Breaking change (fix or feature that would cause existing functionality to change)
  • This change requires a documentation update
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Example scenarios has been updated / added
  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@michalvavrik
Copy link
Member Author

run tests

@michalvavrik
Copy link
Member Author

Failures definitely related, I don't have time for it, so make this draft.

@michalvavrik michalvavrik marked this pull request as draft October 2, 2023 14:07
@rsvoboda
Copy link
Member

rsvoboda commented Nov 6, 2023

@michalvavrik any plans around this? This looks like important one for other platforms

@michalvavrik
Copy link
Member Author

@michalvavrik any plans around this? This looks like important one for other platforms

Agreed, this should be done. I'll add it to my list on this week.

@michalvavrik michalvavrik force-pushed the feature/use-official-docker-zookeeper-img branch from 889d7ed to f2aec9f Compare November 6, 2023 18:38
@michalvavrik
Copy link
Member Author

run tests

@michalvavrik
Copy link
Member Author

it's interesting, because I run OCP tests locally and they passed; difference must be that I run it with released version, I'll check why it doesn't work with a snapshot later

@michalvavrik
Copy link
Member Author

michalvavrik commented Dec 22, 2023

I tried to adapt https://quarkus.io/version/main/guides/kafka-reactive-getting-started Docker file to OCP but so far completely failed, all relevant examples in docs and GH are for operator https://strimzi.io/documentation/ https://github.com/strimzi/strimzi-kafka-operator/tree/0.38.0/examples https://github.com/strimzi/ https://strimzi.io/quickstarts/

I'll keep trying, but so far I don't know how to work around Error syncing load balancer: failed to ensure load balancer: error creating LB floatingip: Resource not found: [POST https://reducted:reducted/v2.0/floatingips], error message: {"NeutronError": {"type": "ExternalGatewayForFloatingIPNotFound", "message": "External network 0ac88691-d42e-4b85-8195-1b1206b30318 is not reachable from subnet 74877b27-8eb5-47e6-917e-0dc8dd5ffe49. Therefore, cannot associate Port 26eb6d1a-87c1-467d-a056-aec5bd96337b with a Floating IP.", "detail": ""}}: {"NeutronError": {"type": "ExternalGatewayForFloatingIPNotFound", "message": "External network 0ac88691-d42e-4b85-8195-1b1206b30318 is not reachable from subnet 74877b27-8eb5-47e6-917e-0dc8dd5ffe49. Therefore, cannot associate Port 26eb6d1a-87c1-467d-a056-aec5bd96337b with a Floating IP.", "detail": ""}}. So it's unlikely I'll succeed soon.

@rsvoboda
Copy link
Member

Please talk to @ozangunalp. Kafka docs seem to be outdated, see for example quarkusio/quarkus#37687 and quarkusio/quarkus#37685

@michalvavrik
Copy link
Member Author

Please talk to @ozangunalp. Kafka docs seem to be outdated, see for example quarkusio/quarkus#37687 and quarkusio/quarkus#37685

I'll do it in 2024.

@michalvavrik
Copy link
Member Author

Apache Kafka 3.6.1 is out and ZooKeeper mode has been deprecated in 3.5. There is a production ready migration from ZooKeeper to KRaft and the Zookeeper mode will be removed in Apache Kafka 4.0 (which is happening in 2024). Might be good idea to explore KRaft instead.

@michalvavrik
Copy link
Member Author

I had fun with Kafka in last days so I read a lot of its doc and I am definitely sure Zookeeper must go, so I'm closing this PR.

@michalvavrik michalvavrik deleted the feature/use-official-docker-zookeeper-img branch April 20, 2024 11:41
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.

2 participants