Selaa lähdekoodia

ci(kafka): fix shellcheck errors

Zaiming (Stone) Shi 3 vuotta sitten
vanhempi
commit
ba34326010

+ 17 - 16
.ci/docker-compose-file/kafka/generate-certs.sh

@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/usr/bin/bash
 
 
 set -euo pipefail
 set -euo pipefail
 
 
@@ -17,29 +17,30 @@ cd /var/lib/secret/
 
 
 ls
 ls
 
 
-echo == Generate self-signed server and client certificates
-echo = generate CA
+echo '== Generate self-signed server and client certificates'
+echo '= generate CA'
 openssl req -new -x509 -keyout ca.key -out ca.crt -days $DAYS -nodes -subj "/C=SE/ST=Stockholm/L=Stockholm/O=brod/OU=test/CN=$HOST"
 openssl req -new -x509 -keyout ca.key -out ca.crt -days $DAYS -nodes -subj "/C=SE/ST=Stockholm/L=Stockholm/O=brod/OU=test/CN=$HOST"
 
 
-echo = generate server certificate request
-openssl req -newkey rsa:2048 -sha256 -keyout server.key -out server.csr -days $DAYS -nodes -subj "/C=SE/ST=Stockholm/L=Stockholm/O=brod/OU=test/CN=$HOST"
+echo '= generate server certificate request'
+openssl req -newkey rsa:2048 -sha256 -keyout server.key -out server.csr -days "$DAYS" -nodes -subj "/C=SE/ST=Stockholm/L=Stockholm/O=brod/OU=test/CN=$HOST"
 
 
-echo = sign server certificate
-openssl x509 -req -CA ca.crt -CAkey ca.key -in server.csr -out server.crt -days $DAYS -CAcreateserial
+echo '= sign server certificate'
+openssl x509 -req -CA ca.crt -CAkey ca.key -in server.csr -out server.crt -days "$DAYS" -CAcreateserial
 
 
-echo = generate client certificate request
-openssl req -newkey rsa:2048 -sha256 -keyout client.key -out client.csr -days $DAYS -nodes -subj "/C=SE/ST=Stockholm/L=Stockholm/O=brod/OU=test/CN=$HOST"
+echo '= generate client certificate request'
+openssl req -newkey rsa:2048 -sha256 -keyout client.key -out client.csr -days "$DAYS" -nodes -subj "/C=SE/ST=Stockholm/L=Stockholm/O=brod/OU=test/CN=$HOST"
 
 
-echo == sign client certificate
+echo '== sign client certificate'
 openssl x509 -req -CA ca.crt -CAkey ca.key -in client.csr -out client.crt -days $DAYS -CAserial ca.srl
 openssl x509 -req -CA ca.crt -CAkey ca.key -in client.csr -out client.crt -days $DAYS -CAserial ca.srl
 
 
-echo = Convert self-signed certificate to PKCS#12 format
-openssl pkcs12 -export -name $HOST -in server.crt -inkey server.key -out server.p12 -CAfile ca.crt -passout pass:$PASS
+echo '= Convert self-signed certificate to PKCS#12 format'
+openssl pkcs12 -export -name "$HOST" -in server.crt -inkey server.key -out server.p12 -CAfile ca.crt -passout pass:"$PASS"
 
 
-echo = Import PKCS#12 into a java keystore
+echo '= Import PKCS#12 into a java keystore'
 
 
-echo $PASS | keytool -importkeystore -destkeystore kafka.keystore.jks -srckeystore server.p12 -srcstoretype pkcs12 -alias $HOST -storepass $PASS
+echo $PASS | keytool -importkeystore -destkeystore kafka.keystore.jks -srckeystore server.p12 -srcstoretype pkcs12 -alias "$HOST" -storepass "$PASS"
 
 
-echo = Import CA into java truststore
 
 
-echo yes | keytool -keystore kafka.truststore.jks -alias CARoot -import -file ca.crt -storepass $PASS
+echo '= Import CA into java truststore'
+
+echo yes | keytool -keystore kafka.truststore.jks -alias CARoot -import -file ca.crt -storepass "$PASS"

+ 2 - 0
.ci/docker-compose-file/kafka/run_add_scram_users.sh

@@ -31,8 +31,10 @@ TIMEOUT=60
 
 
 echo "+++++++ Wait until Kafka ports are up ++++++++"
 echo "+++++++ Wait until Kafka ports are up ++++++++"
 
 
+# shellcheck disable=SC2016
 timeout $TIMEOUT bash -c 'until printf "" 2>>/dev/null >>/dev/tcp/$0/$1; do sleep 1; done' $SERVER $PORT1
 timeout $TIMEOUT bash -c 'until printf "" 2>>/dev/null >>/dev/tcp/$0/$1; do sleep 1; done' $SERVER $PORT1
 
 
+# shellcheck disable=SC2016
 timeout $TIMEOUT bash -c 'until printf "" 2>>/dev/null >>/dev/tcp/$0/$1; do sleep 1; done' $SERVER $PORT2
 timeout $TIMEOUT bash -c 'until printf "" 2>>/dev/null >>/dev/tcp/$0/$1; do sleep 1; done' $SERVER $PORT2
 
 
 echo "+++++++ Run config commands ++++++++"
 echo "+++++++ Run config commands ++++++++"