Fixing assume role

truemark · Dec 11, 2023 · 5900e46 · 5900e46
1 parent 0b3ebff
commit 5900e46
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -25,7 +25,8 @@ jobs:
       - name: Configure AWS Credentials
         uses: aws-actions/configure-aws-credentials@v2
         with:
-          role-to-assume: "${{ secrets.aws_assume_role }}"
+          # TODO Convert to secret
+          role-to-assume: "${{ vars.AWS_ASSUME_ROLE_DEPLOY }}"
           aws-region: "us-east-2"
 
       - name: CDK Deploy
@@ -34,13 +35,6 @@ jobs:
           pnpm i --frozen-lockfile
           cdk -c env=${{ inputs.environment }} deploy --require-approval never
 
-      # TODO Get rid of separate role
-      - name: Configure AWS Credentials
-        uses: aws-actions/configure-aws-credentials@v2
-        with:
-          role-to-assume: "${{ vars.AWS_ASSUME_ROLE_DEPLOY }}"
-          aws-region: "us-east-2"
-
       - name: Setup Cluster Access
         run: |
           aws eks update-kubeconfig --name ${{ vars.CLUSTER_NAME }} --kubeconfig ./kubeconfig