remove default for BROKERS in Dockerfiles (fixes #138)

deviceinsight · Nov 8, 2022 · 9503741 · 9503741
1 parent 8e4f6fd
commit 9503741
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- [#138](https://github.com/deviceinsight/kafkactl/issues/138) remove default value for `BROKERS` in Dockerfiles
 
 ## 3.0.0 - 2022-09-30
 ### Changed

diff --git a/Dockerfile.Scratch b/Dockerfile.Scratch
@@ -1,10 +1,8 @@
-
 FROM alpine:latest as certs
 RUN apk --update --no-cache add ca-certificates && update-ca-certificates
 
 FROM scratch
 ENV USER docker
-ENV BROKERS localhost:9092
 COPY kafkactl /
 COPY --from=certs /etc/ssl/certs/ca-certificates.crt /etc/ssl/certs/
 ENTRYPOINT ["/kafkactl"]
diff --git a/Dockerfile.Ubuntu b/Dockerfile.Ubuntu
@@ -1,6 +1,5 @@
 FROM ubuntu:latest
 ENV USER docker
-ENV BROKERS localhost:9092
 COPY kafkactl /usr/local/bin
 RUN apt-get update && apt-get install -y bash-completion