[helm] Using config map for STS configuration entries

Author: morazow

helm/templates/configmap.yaml

@@ -53,3 +53,25 @@ data:
     {{- end }}
 
     {{- end }}
+
+  entrypoint.sh: |
+    #!/bin/sh
+    set -e
+
+    COMPONENT="$1"
+
+    FLUSS_SERVER_ID=${POD_NAME##*-}
+    cp /opt/conf/server.yaml $FLUSS_HOME/conf
+
+    BIND_LISTENERS="INTERNAL://${POD_IP}:{{ .Values.listeners.internal.port }}, CLIENT://${POD_IP}:{{ .Values.listeners.client.port }}"
+    ADVERTISED_LISTENERS="CLIENT://${POD_NAME}.${COMPONENT}-server-hs.${POD_NAMESPACE}.svc.cluster.local:{{ .Values.listeners.client.port }}"
+
+    echo "" >> $FLUSS_HOME/conf/server.yaml
+    echo "bind.listeners: ${BIND_LISTENERS}" >> $FLUSS_HOME/conf/server.yaml
+    echo "advertised.listeners: ${ADVERTISED_LISTENERS}" >> $FLUSS_HOME/conf/server.yaml
+
+    if [ "$COMPONENT" = "tablet" ]; then
+      echo "tablet-server.id: ${FLUSS_SERVER_ID}" >> $FLUSS_HOME/conf/server.yaml
+    fi
+
+    exec bin/${COMPONENT}-server.sh start-foreground

helm/templates/sts-coordinator.yaml

@@ -71,19 +71,8 @@ spec:
               containerPort: {{ .Values.listeners.client.port }}
           command:
             - "/bin/sh"
-            - "-c"
-            - |
-              export FLUSS_SERVER_ID=${POD_NAME##*-} && \
-              cp /opt/conf/server.yaml $FLUSS_HOME/conf && \
-
-              BIND_LISTENERS="INTERNAL://${POD_IP}:{{ .Values.listeners.internal.port }}, CLIENT://${POD_IP}:{{ .Values.listeners.client.port }}" && \
-              ADVERTISED_LISTENERS="CLIENT://${POD_NAME}.coordinator-server-hs.${POD_NAMESPACE}.svc.cluster.local:{{ .Values.listeners.client.port }}" && \
-
-              echo "" >> $FLUSS_HOME/conf/server.yaml && \
-              echo "bind.listeners: ${BIND_LISTENERS}" >> $FLUSS_HOME/conf/server.yaml && \
-              echo "advertised.listeners: ${ADVERTISED_LISTENERS}" >> $FLUSS_HOME/conf/server.yaml && \
-
-              bin/coordinator-server.sh start-foreground
+            - "/opt/conf/entrypoint.sh"
+            - "coordinator"
           livenessProbe:
             failureThreshold: 100
             timeoutSeconds: 1

helm/templates/sts-tablet.yaml

@@ -67,20 +67,8 @@ spec:
               containerPort: {{ .Values.listeners.client.port }}
           command:
             - "/bin/sh"
-            - "-c"
-            - |
-              export FLUSS_SERVER_ID=${POD_NAME##*-} && \
-              cp /opt/conf/server.yaml $FLUSS_HOME/conf && \
-
-              BIND_LISTENERS="INTERNAL://${POD_IP}:{{ .Values.listeners.internal.port }}, CLIENT://${POD_IP}:{{ .Values.listeners.client.port }}" && \
-              ADVERTISED_LISTENERS="CLIENT://${POD_NAME}.tablet-server-hs.${POD_NAMESPACE}.svc.cluster.local:{{ .Values.listeners.client.port }}" && \
-
-              echo "" >> $FLUSS_HOME/conf/server.yaml && \
-              echo "tablet-server.id: ${FLUSS_SERVER_ID}" >> $FLUSS_HOME/conf/server.yaml && \
-              echo "bind.listeners: ${BIND_LISTENERS}" >> $FLUSS_HOME/conf/server.yaml && \
-              echo "advertised.listeners: ${ADVERTISED_LISTENERS}" >> $FLUSS_HOME/conf/server.yaml && \
-
-              bin/tablet-server.sh start-foreground
+            - "/opt/conf/entrypoint.sh"
+            - "tablet"
           livenessProbe:
             failureThreshold: 100
             timeoutSeconds: 1

helm/tests/security_test.yaml

@@ -274,11 +274,11 @@ tests:
           path: data["server.yaml"]
           pattern: 'security\.sasl\.enabled\.mechanisms: PLAIN,PLAIN'
   - it: writes server yaml using expected destination path
-    template: templates/sts-tablet.yaml
+    template: templates/configmap.yaml
     asserts:
       - matchRegex:
-          path: spec.template.spec.containers[0].command[2]
+          path: data["entrypoint.sh"]
           pattern: 'cp /opt/conf/server\.yaml \$FLUSS_HOME/conf'
       - matchRegex:
-          path: spec.template.spec.containers[0].command[2]
+          path: data["entrypoint.sh"]
           pattern: '>> \$FLUSS_HOME/conf/server\.yaml'