Skip to content

Commit

Permalink
breaking: topic job config should now be whitespace (not comma) separ…
Browse files Browse the repository at this point in the history
…ated

because redpanda uses comma in for example cleanup.policy values
  • Loading branch information
solsson committed Sep 13, 2024
1 parent 73060c5 commit 0855d0a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions kafka/topic-job/kafka-topic-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ spec:
yolean.se/kafka-bootstrap: bootstrap.kafka:9092
yolean.se/kafka-topic-name: ""
yolean.se/kafka-topic-config: >-
max.message.bytes=524288,
retention.bytes=-1,
retention.ms=-1,
max.message.bytes=524288
retention.bytes=-1
retention.ms=-1
yolean.se/kafka-topic-partitions: "1"
yolean.se/kafka-topic-replicas: "-1"
spec:
Expand All @@ -29,7 +29,7 @@ spec:
[ -n "$TOPIC_CONFIG" ] || exit 1
function config_args {
FLAG=$1
for C in $(echo "$TOPIC_CONFIG" | tr ',' ' '); do echo -n " $FLAG $C"; done
for C in $TOPIC_CONFIG; do echo -n " $FLAG $C"; done
echo ''
}
until rpk cluster --brokers $KAFKA_BOOTSTRAP info -b -c; do sleep 1; done;
Expand Down

0 comments on commit 0855d0a

Please sign in to comment.