Skip to content

Kafka-15444: Native docker image for Apache Kafka(KIP-974) #15927

Merged
omkreddy merged 9 commits into
apache:trunkfrom
kagarwal06:native-image-KAFKA-15444
May 22, 2024
Merged

Kafka-15444: Native docker image for Apache Kafka(KIP-974) #15927
omkreddy merged 9 commits into
apache:trunkfrom
kagarwal06:native-image-KAFKA-15444

Conversation

@kagarwal06

@kagarwal06 kagarwal06 commented May 12, 2024

Copy link
Copy Markdown
Contributor

This PR aims to add Docker Image for GraalVM based Native Kafka Broker as per the following KIP - https://cwiki.apache.org/confluence/display/KAFKA/KIP-974%3A+Docker+Image+for+GraalVM+based+Native+Kafka+Broker

This PR adds the following functionalities:

  1. Ability to build the docker image for Native Apache Kafka
    • Dockerfile
    • Launch script
    • metadata configs required by graalVM native-image: link
  2. Add Kafka startup ability in the KafkaDockerWrapper.scala
  3. ABility to build and test the image - integrated with the existing JVM docker image framework.

Testing:
I modified the internal system tests pipeline to run System tests via vagrant on native iAK. The test results were comparable with the System tests results of AK.
I will raise the PR to integrate the native AK with System tests post this PR merge.

Post this PR:

  1. PR to integrate the native AK with System tests.
  2. Integration with Github action jobs
  3. Add Examples
  4. Add Documentation
  5. Update release process

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@omkreddy omkreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR. Can we add README to docker/native and add few details about the image and native-image-configs (unless we will be covering in another PR)?

Also will the current Github workflows work with current PR changes ?

@kagarwal06

Copy link
Copy Markdown
Contributor Author

Also will the current Github workflows work with current PR changes ?

The github workflows uses the build, test and release scripts.
I have integrated the native docker image with these scripts and tested them locally. They are working fine.
I will raise a separate PR to integrate this image in the github action workflows.

Can we add README to docker/native and add few details about the image and native-image-configs

Sure. I will add this Readme in the documentation PR.

Comment thread docker/native/Dockerfile
Comment thread docker/native/Dockerfile
Comment thread docker/native/launch

@omkreddy omkreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@kagarwal06 Thanks for the PR. LGTM

@omkreddy omkreddy merged commit 271c04b into apache:trunk May 22, 2024
rreddy-22 pushed a commit to rreddy-22/kafka-rreddy that referenced this pull request May 24, 2024
…5927)

This PR aims to add Docker Image for GraalVM based Native Kafka Broker as per the following KIP - https://cwiki.apache.org/confluence/display/KAFKA/KIP-974%3A+Docker+Image+for+GraalVM+based+Native+Kafka+Broker

This PR adds the following functionalities:

Ability to build the docker image for Native Apache Kafka
   - Dockerfile
   - Launch script
   - metadata configs required by graalVM native-image: link
Add Kafka startup ability in the KafkaDockerWrapper.scala
Ability to build and test the image - integrated with the existing JVM docker image framework.
chiacyu pushed a commit to chiacyu/kafka that referenced this pull request Jun 1, 2024
…5927)

This PR aims to add Docker Image for GraalVM based Native Kafka Broker as per the following KIP - https://cwiki.apache.org/confluence/display/KAFKA/KIP-974%3A+Docker+Image+for+GraalVM+based+Native+Kafka+Broker

This PR adds the following functionalities:

Ability to build the docker image for Native Apache Kafka
   - Dockerfile
   - Launch script
   - metadata configs required by graalVM native-image: link
Add Kafka startup ability in the KafkaDockerWrapper.scala
Ability to build and test the image - integrated with the existing JVM docker image framework.
TaiJuWu pushed a commit to TaiJuWu/kafka that referenced this pull request Jun 8, 2024
…5927)

This PR aims to add Docker Image for GraalVM based Native Kafka Broker as per the following KIP - https://cwiki.apache.org/confluence/display/KAFKA/KIP-974%3A+Docker+Image+for+GraalVM+based+Native+Kafka+Broker

This PR adds the following functionalities:

Ability to build the docker image for Native Apache Kafka
   - Dockerfile
   - Launch script
   - metadata configs required by graalVM native-image: link
Add Kafka startup ability in the KafkaDockerWrapper.scala
Ability to build and test the image - integrated with the existing JVM docker image framework.
gongxuanzhang pushed a commit to gongxuanzhang/kafka that referenced this pull request Jun 12, 2024
…5927)

This PR aims to add Docker Image for GraalVM based Native Kafka Broker as per the following KIP - https://cwiki.apache.org/confluence/display/KAFKA/KIP-974%3A+Docker+Image+for+GraalVM+based+Native+Kafka+Broker

This PR adds the following functionalities:

Ability to build the docker image for Native Apache Kafka
   - Dockerfile
   - Launch script
   - metadata configs required by graalVM native-image: link
Add Kafka startup ability in the KafkaDockerWrapper.scala
Ability to build and test the image - integrated with the existing JVM docker image framework.
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