PacktPublishing / Apache-Kafka-Series---Learn-Apache-Kafka-for-Beginners-v3

Code Repository for Apache Kafka Series - Learn Apache Kafka for Beginners, Published by Packt
MIT License
40 stars 34 forks source link

Conduktor container now requires a Postgres db #2

Open doug-rosser opened 1 year ago

doug-rosser commented 1 year ago

Docs at https://docs.conduktor.io/platform/configuration/database/

Posting the diff here for brevity as it's a pretty trivial patch:

$ git diff docker-compose.yml
diff --git a/kafka-beginners-course-main/conduktor-platform/docker-compose.yml b/kafka-beginners-course-main/conduktor-platform/docker-compose.yml
index 01f7e2d..09f48e7 100644
--- a/kafka-beginners-course-main/conduktor-platform/docker-compose.yml
+++ b/kafka-beginners-course-main/conduktor-platform/docker-compose.yml
@@ -51,6 +51,16 @@ services:
       - zoo1
       - kafka1

+  db:
+    image: postgres
+    restart: always
+    environment:
+      POSTGRES_PASSWORD: pgpass
+    volumes:
+      - /tmp/pg-data:/var/lib/postgresql/data
+    ports:
+      - 5432:5432
+
   conduktor-platform:
     image: conduktor/conduktor-platform:latest
     ports:
@@ -62,6 +72,7 @@ services:
         target: /opt/conduktor/platform-config.yaml
         read_only: true
     depends_on:
+      - db
       - kafka1
       - kafka-schema-registry
     environment:
@@ -69,6 +80,7 @@ services:
       KAFKA_BOOTSTRAP_SERVER: PLAINTEXT://kafka1:19092
       SR_SERVER: "http://kafka-schema-registry:8081"
       ORGANISATION_NAME: "default"
+      CDK_DATABASE_URL: "postgresql://postgres:pgpass@db:5432/postgres"
       CDK_ADMIN_EMAIL: "admin@conduktor.io"
       CDK_ADMIN_PASSWORD: "admin"
sderosiaux commented 1 year ago

Hi, it was fixed last week, see the updated version https://github.com/conduktor/kafka-beginners-course/blob/main/conduktor-platform/docker-compose.yml

Thanks for reporting this issue!