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

#6114 Using UBI base image for Kafka Streams quickstart #400

Closed
wants to merge 1 commit into from

Conversation

gunnarmorling
Copy link
Contributor

RocksDB (a dependency of Kafka Streams) isn't compatible with Alpine.
Hence Kafka Streams cannot be used with the default Dockerfile.jvm as
generated by Quarkus atm.
The quickstart guide will be updated accordingly and this example is
updated to verify things work with UBI + installed JDK.

Check list:

Your pull request:

  • targets the development branch
  • uses the 999-SNAPSHOT version of Quarkus
  • has tests (mvn clean test)
  • works in native (mvn clean package -Pnative)
  • has native tests (mvn clean verify -Pnative)
  • makes sure the documentation must not be updated
  • links the documentation update pull request (if needed)
  • updates or creates the README.md file (with build and run instructions)
  • For new quickstart, is located in the directory component-quickstart
  • For new quickstart, is added to the root pom.xml and README.md

Docs PR will follow up in a bit.

RocksDB (a dependency of Kafka Streams) isn't compatible with Alpine.
Hence Kafka Streams cannot be used with the default Dockerfile.jvm as
generated by Quarkus atm.
The quickstart guide will be updated accordingly and this example is
updated to verify things work with UBI + installed JDK.
@gunnarmorling
Copy link
Contributor Author

Hey @cescoffier, do we still need this one? I remember you were about to do some similar change on a more broad level anyways?

@cescoffier
Copy link
Member

No, but we would need to update all the Dockerfile we have here with the new UBI-based one.

@gsmet
Copy link
Member

gsmet commented Feb 4, 2020

This one is not needed anymore, @cescoffier fixed it globally.

@gsmet gsmet closed this Feb 4, 2020
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