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

chore(kafka): use apache/kafka official image #1480

Merged
merged 1 commit into from
Apr 6, 2024

Conversation

tranngoclam
Copy link
Contributor

Changes

Merge Requirements

For new features contributions please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@tranngoclam tranngoclam requested a review from a team March 24, 2024 16:01
@austinlparker austinlparker added the helm-update-required Requires an update to the Helm chart when released label Mar 25, 2024
@austinlparker
Copy link
Member

What's the difference between these two images?

@tranngoclam
Copy link
Contributor Author

What's the difference between these two images?

@austinlparker it's just more official, under Apache. Additionally:

  • size is smaller than the confluentinc/cp-kafka: 370MB vs 833MB
  • java version is newer (21)

More details in: https://cwiki.apache.org/confluence/display/KAFKA/KIP-975%3A+Docker+Image+for+Apache+Kafka

Copy link
Member

@mviitane mviitane left a comment

Choose a reason for hiding this comment

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

Runs fine. Also, KRaft seems to be enabled by default.
kafka | Running in KRaft mode...

@austinlparker austinlparker merged commit 5cef0e7 into open-telemetry:main Apr 6, 2024
26 of 27 checks passed
AlexPSplunk pushed a commit to splunk/edu-opentelemetry-demo that referenced this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm-update-required Requires an update to the Helm chart when released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants