use --set-string for tag

asuc-octo · Sep 29, 2024 · 2d97918 · 2d97918
1 parent 8a5cf88
commit 2d97918
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/cd-dev.yaml b/.github/workflows/cd-dev.yaml
@@ -27,7 +27,7 @@ jobs:
       - name: Set vars
         id: vars
         run: |
-          echo "sha_short=\"$(git rev-parse --short ${{ github.sha }})\"" >> $GITHUB_OUTPUT
+          echo "sha_short=$(git rev-parse --short ${{ github.sha }})" >> $GITHUB_OUTPUT
 
   build-push:
     name: Build and Push Images
@@ -56,8 +56,8 @@ jobs:
             helm install bt-dev-app-${{ needs.compute-sha.outputs.sha_short }} ./app --namespace=bt \
               --set env=dev \
               --set ttl=${{ inputs.ttl }} \
-              --set frontend.image.tag=${{ needs.compute-sha.outputs.sha_short }} \
-              --set backend.image.tag=${{ needs.compute-sha.outputs.sha_short }} \
+              --set-string frontend.image.tag=${{ needs.compute-sha.outputs.sha_short }} \
+              --set-string backend.image.tag=${{ needs.compute-sha.outputs.sha_short }} \
               --set host=${{ needs.compute-sha.outputs.sha_short }}.stanfurdtime.com \
               --set mongoUri=mongodb://bt-dev-mongo-mongodb.bt.svc.cluster.local:27017/bt \
               --set redisUri=redis://bt-dev-redis-master.bt.svc.cluster.local:6379 \

diff --git a/infra/app/templates/cleanup.yaml b/infra/app/templates/cleanup.yaml
@@ -11,6 +11,6 @@ spec:
       containers:
         - name: cleanup
           image: alpine/helm
-          command: ['sh', '-c', 'sleep ${{ `{{` }} {{ .Values.ttl }} * 60 * 60 {{ `}}` }}; helm uninstall {{ .Release.Name }}']
+          command: ['sh', '-c', 'sleep $(( {{ .Values.ttl }} * 60 * 60 )); helm uninstall {{ .Release.Name }}']
       restartPolicy: Never
 {{ end }}