Merge pull request #126 from unboxed/fix-matrix-deploy

Fix issue of new matrix deploy
unboxed · Aug 11, 2023 · 4679051 · 4679051
2 parents 79e2762 + c2d1208
commit 4679051
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.github/workflows/deploy-environment.yml b/.github/workflows/deploy-environment.yml
@@ -24,7 +24,7 @@ jobs:
     environment: ${{ inputs.environment-name }}
 
     steps:
-      - name: Checkout
+      - name: Checkout code
         uses: actions/checkout@v3
 
       - name: Get github commit sha
@@ -49,7 +49,7 @@ jobs:
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v2
 
-      - name: Build, tag, and push ${{ inputs.environment-name }} image to Amazon ECR
+      - name: Build, tag, and push ${{ inputs.environment-name }} image to Amazon ECR based github sha
         id: build-image
         env:
           ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
@@ -97,9 +97,9 @@ jobs:
         run: |
           echo "uri=$ECR_REGISTRY/$ECR_REPOSITORY:$IMAGE_TAG" >>$GITHUB_OUTPUT
 
-      - name: Download task definition for db_migrate and strip unused properties
+      - name: Download task definition for db_migrate, strip unused properties and update image to latest image_arn
         env:
-          IMAGE_ARN: ${{ steps.ecr-image.outputs.image }}
+          IMAGE_ARN: ${{ steps.ecr-image.outputs.uri }}
         run: |
           aws ecs describe-task-definition --task-definition paapi-db-migrate-${{ inputs.environment-name }} --query taskDefinition | \
           jq -r 'del(.compatibilities, .taskDefinitionArn, .requiresAttributes, .revision, .status, .registeredAt, .registeredBy)' | \