From 0855d0a983cd2ba0bce172e64f8bb636450b4c3f Mon Sep 17 00:00:00 2001
From: Staffan Olsson <staffan@repos.se>
Date: Fri, 13 Sep 2024 12:11:39 +0200
Subject: [PATCH] breaking: topic job config should now be whitespace (not
 comma) separated

because redpanda uses comma in for example cleanup.policy values
---
 kafka/topic-job/kafka-topic-job.yaml | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/kafka/topic-job/kafka-topic-job.yaml b/kafka/topic-job/kafka-topic-job.yaml
index 6d361e7..60b5815 100644
--- a/kafka/topic-job/kafka-topic-job.yaml
+++ b/kafka/topic-job/kafka-topic-job.yaml
@@ -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:
@@ -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;