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

[DPE-3233] Apache Kafka <> OCI Factory Integration #155

Merged

Conversation

deusebio
Copy link
Contributor

@deusebio deusebio commented Apr 4, 2024

Ping the @canonical/rocks team.


Description

We have recently created a Rock for packaging Apache Kafka. The binaries are the same as the ones used in Charmed Kafka images, but here we just reduce only to the Apache Kafka, leaving outside all the extra components and dependencies.

The idea of the image would be to replace the current Apache Kafka images in DockerHub that are not integrated with Rock and OCI Factory pipeline. With respect to that image, there is a UX difference, namely that the new image only provides a way to run Kafka. The previous image was also allowing to run ZooKeeper, but given we already have an Official ZooKeeper image, I would generally advice to use that one instead, without packaging here those binaries/entrypoints as well. In the documentation I provide guidance on how Kafka can easily be deployed along side a ZooKeeper container (using the Canonical offical ZooKeeper image)

@deusebio deusebio changed the title [DPE-3233] Apache ZooKeeper <> OCI Factory Integration [DPE-3233] Apache Kafka <> OCI Factory Integration Apr 4, 2024
@deusebio deusebio force-pushed the dpe-3233-kafka-oci-integration branch from 99b62c0 to f5e5d06 Compare April 4, 2024 21:11
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

LGTM ty!

@cjdcordeiro cjdcordeiro self-assigned this Apr 8, 2024
@cjdcordeiro cjdcordeiro merged commit 41d5513 into canonical:main Apr 8, 2024
25 checks passed
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