mirror of
https://github.com/Sea-Haven-Industries/.github.git
synced 2026-10-02 10:53:18 +00:00
Fix CFN execution role transform permission and pip install path
- Add cloudformation:CreateChangeSet on aws:transform/* to the shared CFN execution role (required for SAM's Serverless transform) - Remove working-directory from pip install step so it finds requirements.txt at repo root (not just cdk-dir)
This commit is contained in:
parent
9a8d1f7736
commit
fffc08da43
2 changed files with 10 additions and 1 deletions
1
.github/workflows/cd-cdk.yaml
vendored
1
.github/workflows/cd-cdk.yaml
vendored
|
|
@ -57,7 +57,6 @@ jobs:
|
||||||
|
|
||||||
- name: Install Python dependencies
|
- name: Install Python dependencies
|
||||||
if: ${{ inputs.python-version != '' }}
|
if: ${{ inputs.python-version != '' }}
|
||||||
working-directory: ${{ inputs.cdk-dir }}
|
|
||||||
run: |
|
run: |
|
||||||
for req in $(find . -name requirements.txt -not -path '*/node_modules/*'); do
|
for req in $(find . -name requirements.txt -not -path '*/node_modules/*'); do
|
||||||
pip install -r "$req"
|
pip install -r "$req"
|
||||||
|
|
|
||||||
|
|
@ -54,6 +54,16 @@ Resources:
|
||||||
- arn:aws:iam::aws:policy/AmazonEventBridgeFullAccess
|
- arn:aws:iam::aws:policy/AmazonEventBridgeFullAccess
|
||||||
- arn:aws:iam::aws:policy/AmazonSESFullAccess
|
- arn:aws:iam::aws:policy/AmazonSESFullAccess
|
||||||
- arn:aws:iam::aws:policy/IAMFullAccess
|
- arn:aws:iam::aws:policy/IAMFullAccess
|
||||||
|
Policies:
|
||||||
|
- PolicyName: cloudformation-transforms
|
||||||
|
PolicyDocument:
|
||||||
|
Version: "2012-10-17"
|
||||||
|
Statement:
|
||||||
|
- Effect: Allow
|
||||||
|
Action:
|
||||||
|
- cloudformation:CreateChangeSet
|
||||||
|
Resource:
|
||||||
|
- arn:aws:cloudformation:us-east-1:aws:transform/*
|
||||||
|
|
||||||
# ---------------------------------------------------------------------------
|
# ---------------------------------------------------------------------------
|
||||||
# SAM deploy roles (5 repos)
|
# SAM deploy roles (5 repos)
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue