Skip to content

Commit

Permalink
merge: #3974
Browse files Browse the repository at this point in the history
3974: fix: remove secrets step as it is no longer needed r=sprutton1 a=sprutton1

Missed this when I cleaned up how we did secrets last time around. It's unnecessary now.

Co-authored-by: Scott Prutton <[email protected]>
  • Loading branch information
si-bors-ng[bot] and sprutton1 authored Jun 12, 2024
2 parents e729dd7 + 6a3d4c5 commit 40d79e8
Showing 1 changed file with 3 additions and 16 deletions.
19 changes: 3 additions & 16 deletions .github/workflows/deploy-stack.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,21 +37,8 @@ jobs:
environment: shared
secrets: inherit

secrets:
needs: invalidate-cache
runs-on: ubuntu-latest
outputs:
aws_access_key: ${{ steps.prepare.outputs.aws_access_key }}
aws_secret_key: ${{ steps.prepare.outputs.aws_secret_key }}
steps:
- name: prepare
id: prepare
run: |
echo "aws_access_key=aws_${{ github.event.inputs.environment }}_access_key" >> "$GITHUB_OUTPUT"
echo "aws_secret_key=aws_${{ github.event.inputs.environment }}_secret_key" >> "$GITHUB_OUTPUT"
deploy-service:
needs: secrets
needs: invalidate-cache
if: ${{ github.event.inputs.service != 'all' }}
uses: ./.github/workflows/deploy-service.yml
with:
Expand All @@ -61,7 +48,7 @@ jobs:
secrets: inherit

deploy-services:
needs: secrets
needs: invalidate-cache
if: ${{ github.event.inputs.service == 'all' }}
strategy:
fail-fast: false
Expand All @@ -75,7 +62,7 @@ jobs:
secrets: inherit

set-init:
needs: secrets
needs: invalidate-cache
uses: ./.github/workflows/set-init-version.yml
with:
environment: ${{ github.event.inputs.environment }}
Expand Down

0 comments on commit 40d79e8

Please sign in to comment.