Skip to content

Commit

Permalink
[kafka] remove KRaft mode support workarounds
Browse files Browse the repository at this point in the history
as the latest confluentinc/cp-kafka:7.4.0-1-ubi8
already provides built-in support
  • Loading branch information
styblope committed May 5, 2023
1 parent 9a1b07a commit c107224
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 28 deletions.
6 changes: 1 addition & 5 deletions src/kafka/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,6 @@ ADD https://github.com/open-telemetry/opentelemetry-java-instrumentation/release
RUN chmod go+r /tmp/opentelemetry-javaagent.jar

USER appuser
WORKDIR /tmp
COPY ./src/kafka/clusterID clusterID
COPY ./src/kafka/update_run.sh update_run.sh

ENV KAFKA_LISTENERS=PLAINTEXT://0.0.0.0:9092,CONTROLLER://0.0.0.0:9093
ENV KAFKA_CONTROLLER_QUORUM_VOTERS='1@0.0.0.0:9093'
Expand All @@ -28,5 +25,4 @@ ENV KAFKA_AUTO_CREATE_TOPICS_ENABLE=true
ENV KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR=1
ENV KAFKA_TRANSACTION_STATE_LOG_REPLICATION_FACTOR=1
ENV KAFKA_OPTS="-javaagent:/tmp/opentelemetry-javaagent.jar -Dotel.jmx.target.system=kafka-broker"

ENTRYPOINT ["/bin/sh", "-c", "ls -lh /tmp && /tmp/update_run.sh && /etc/confluent/docker/run"]
ENV CLUSTER_ID=ckjPoprWQzOf0-FuNkGfFQ
5 changes: 1 addition & 4 deletions src/kafka/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,5 @@
This is used as a message queue service to connect the checkout service with
the accounting and fraud detection services.

## KRaft: Kafka without ZooKeeper

Kafka is run in KRaft mode. The `update_run.sh` script is used to configure
the system to run Kafka in this mode. Environment variables are substituted at
Kafka is run in KRaft mode. Environment variables are substituted at
deploy-time.
1 change: 0 additions & 1 deletion src/kafka/clusterID

This file was deleted.

18 changes: 0 additions & 18 deletions src/kafka/update_run.sh

This file was deleted.

0 comments on commit c107224

Please sign in to comment.