Skip to content

Commit da4e25c

Browse files
author
Max Azatian
committed
CI fix 4 | disabling SASL for Kafka+Zookeper
+ update arch file: mermaid instead of plantuml for better rendering
1 parent 66ff9a8 commit da4e25c

File tree

2 files changed

+256
-223
lines changed

2 files changed

+256
-223
lines changed

.github/workflows/tests.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,11 @@ jobs:
4747
yq eval '.services.mongo.environment.MONGO_INITDB_ROOT_USERNAME = "testroot"' -i docker-compose.ci.yaml
4848
yq eval '.services.mongo.environment.MONGO_INITDB_ROOT_PASSWORD = "testpassword"' -i docker-compose.ci.yaml
4949
50-
# Disable SASL authentication for Kafka in CI (remove KAFKA_OPTS and volume mounts for JAAS files)
50+
# Disable SASL authentication for Kafka and Zookeeper in CI
5151
yq eval 'del(.services.kafka.environment.KAFKA_OPTS)' -i docker-compose.ci.yaml
52-
yq eval '.services.kafka.volumes = [.services.kafka.volumes[] | select(. | contains("kafka_client_jaas.conf") | not)]' -i docker-compose.ci.yaml
53-
yq eval '.services.kafka.volumes = [.services.kafka.volumes[] | select(. | contains("kafka_jaas.conf") | not)]' -i docker-compose.ci.yaml
52+
yq eval 'del(.services.zookeeper.environment.KAFKA_OPTS)' -i docker-compose.ci.yaml
53+
yq eval '.services.kafka.volumes = [.services.kafka.volumes[] | select(. | contains("jaas.conf") | not)]' -i docker-compose.ci.yaml
54+
yq eval '.services.zookeeper.volumes = [.services.zookeeper.volumes[] | select(. | contains("/etc/kafka/secrets") | not)]' -i docker-compose.ci.yaml
5455
5556
# For the cert-generator service
5657
# Check if extra_hosts exists, if not create it as a list

0 commit comments

Comments
 (0)