Skip to content

Commit ecda2fa

Browse files
DEVX-2734: prebuild producer/consumer in demo client image (#1089)
* DEVX-2734: prebuild producer/consumer in demo client image. * DEVX-2734: use reload4j instead of log4j in ccloud-observability example * DEVX-2734: add explicit tag to `docker push` command Co-authored-by: Dave Troiano <[email protected]>
1 parent 19e61ec commit ecda2fa

File tree

4 files changed

+11
-11
lines changed

4 files changed

+11
-11
lines changed

ccloud-observability/Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@ WORKDIR /tmp/java
66
COPY pom.xml .
77
COPY checkstyle/suppressions.xml /tmp/java/checkstyle/suppressions.xml
88
COPY checkstyle.xml /tmp/java/checkstyle.xml
9-
RUN mvn verify --fail-never
109
# Copy in java src code
1110
COPY src/ /tmp/java/src/
11+
RUN mvn package

ccloud-observability/Makefile

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11

22
PUSH_PREFIX ?= cnfldemos
33
CONTAINER_NAME ?= ccloud-observability-client
4-
VERSION ?= 0.1.0
4+
VERSION ?= 0.2.0
55

66

77
build-image:
88
docker build -t $(PUSH_PREFIX)/$(CONTAINER_NAME):$(VERSION) .
99

1010
push-image: build-image
11-
docker push $(PUSH_PREFIX)/$(CONTAINER_NAME)
11+
docker push $(PUSH_PREFIX)/$(CONTAINER_NAME):$(VERSION)

ccloud-observability/docker-compose.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ x-network: &cap-add
99
# Note: src code and dependencies are built into cnfldemos/ccloud-observability-client
1010
# the Dockerfile for cnfldemos/ccloud-observability-client can be found in this directory--called Dockerfile
1111
x-client-base: &client-base
12-
image: cnfldemos/ccloud-observability-client:0.1.0
12+
image: cnfldemos/ccloud-observability-client:0.2.0
1313
restart: always
1414
ports:
1515
- 1234
@@ -91,4 +91,4 @@ services:
9191
environment:
9292
- CLOUD_CLUSTER=$CLOUD_CLUSTER
9393
- BOOTSTRAP_SERVERS=$BOOTSTRAP_SERVERS
94-
- SASL_JAAS_CONFIG=$SASL_JAAS_CONFIG
94+
- SASL_JAAS_CONFIG=$SASL_JAAS_CONFIG

ccloud-observability/pom.xml

+6-6
Original file line numberDiff line numberDiff line change
@@ -83,14 +83,14 @@
8383
<version>${slf4j-api.version}</version>
8484
</dependency>
8585
<dependency>
86-
<groupId>org.slf4j</groupId>
87-
<artifactId>slf4j-log4j12</artifactId>
88-
<version>${slf4j-api.version}</version>
86+
<groupId>ch.qos.reload4j</groupId>
87+
<artifactId>reload4j</artifactId>
88+
<version>${reload4j.version}</version>
8989
</dependency>
90-
<!-- Use a repackaged version of log4j with security patches. Default log4j v1.2 is a transitive dependency of slf4j-log4j12, but it is excluded in common/pom.xml -->
9190
<dependency>
92-
<groupId>io.confluent</groupId>
93-
<artifactId>confluent-log4j</artifactId>
91+
<groupId>org.slf4j</groupId>
92+
<artifactId>slf4j-reload4j</artifactId>
93+
<version>${slf4j-reload4j.version}</version>
9494
</dependency>
9595
<dependency>
9696
<groupId>com.fasterxml.jackson.core</groupId>

0 commit comments

Comments
 (0)