Merge pull request #12 from techops-services/helm-vars-replace

Fix helm vars replace

Author: eskp

.github/workflows/deploy.yaml

@@ -77,11 +77,20 @@ jobs:
 
           docker push $ECR_REGISTRY/$AWS_ECR_NAME:migrator
 
+      - name: Replace variables in the Helm values file
+        id: replace-vars
+        if: ${{ !contains(github.event.head_commit.message , '[skip deploy]') }}
+        env:
+          ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
+          IMAGE_TAG: ${{ steps.vars.outputs.sha_short }}
+        run: |
+          sed -i 's|${ECR_REGISTRY}|'"$ECR_REGISTRY"'|g' $HELM_FILE
+          sed -i 's|${IMAGE_TAG}|'"$IMAGE_TAG"'|g' $HELM_FILE
+
       - name: Deploying Service to Kubernetes with Helm
         id: deploy
         uses: bitovi/github-actions-deploy-eks-helm@v1.2.10
         with:
-          values: image.repository=${{ steps.login-ecr.outputs.registry }}/${{ env.AWS_ECR_NAME }},image.tag=${{ steps.vars.outputs.sha_short }}
           cluster-name: ${{ env.CLUSTER_NAME }}
           config-files: ${{ env.HELM_FILE }}
           chart-path: techops-services/common

deploy/staging/values.yaml

@@ -33,7 +33,7 @@ httpBasicAuth:
 
 image:
   repository: ${ECR_REGISTRY}/keeperhub-staging
-  tag: latest
+  tag: ${IMAGE_TAG}
   pullPolicy: Always
 
 deployment: