Skip to content

Commit

Permalink
fix: Pass parameters to cdk command
Browse files Browse the repository at this point in the history
`npm run` by default treats all parameters as its own, and *ignores*
unknown parameters <npm/cli#6638>.
  • Loading branch information
l0b0 committed Sep 25, 2023
1 parent 1eed107 commit 3929ddf
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,8 @@ jobs:
uses: aws-actions/configure-aws-credentials@v4.0.0
with:
aws-region: ap-southeast-2
role-to-assume: ${{ github.ref == 'refs/heads/master' && 'TBD' || 'arn:aws:iam::623931144233:role/ContinuousIntegration-GitHubActionsRole4F1BBA26-WDE0O3ZN93A6' }}
role-to-assume: ${{ github.ref == 'refs/heads/master' && 'arn:aws:iam::434775598764:role/ContinuousIntegration-GitHubActionsRole4F1BBA26-U1I7SY11QA7D' || 'arn:aws:iam::623931144233:role/ContinuousIntegration-GitHubActionsRole4F1BBA26-WDE0O3ZN93A6' }}

- name: Deploy
if: github.repository == 'linz/hydro-aws-infrastructure'
run: npm run cdk deploy --require-approval=never --role-arn=${{ github.ref == 'refs/heads/master' && 'TBD' || 'arn:aws:iam::623931144233:role/ContinuousIntegration-CiExecRoleAA6BFF70-CTEJ6J3L1NVU' }} --verbose
run: npm run cdk deploy -- --method=direct --require-approval=never --role-arn=${{ github.ref == 'refs/heads/master' && 'arn:aws:iam::434775598764:role/ContinuousIntegration-CiExecRoleAA6BFF70-1SL2099R0R4A2' || 'arn:aws:iam::623931144233:role/ContinuousIntegration-CiExecRoleAA6BFF70-CTEJ6J3L1NVU' }} --strict --verbose

0 comments on commit 3929ddf

Please sign in to comment.