Skip to content

Commit cce5f56

Browse files
committed
Improve traceability of AWS API calls in test runs
1 parent cf7d0a3 commit cce5f56

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

.github/workflows/module-test.yaml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,6 @@ jobs:
3838
chmod +x ${HOME}/.local/bin/*
3939
4040
echo "${HOME}/.local/bin" >> $GITHUB_PATH
41-
- name: Get AWS credentials
42-
uses: aws-actions/[email protected]
43-
with:
44-
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
45-
role-duration-seconds: 3600
46-
aws-region: ${{ secrets.AWS_REGION }}
47-
role-session-name: GithubActionsSession
4841
- name: Set cluster ID
4942
env:
5043
RUN_ID: "${{ github.job }}"
@@ -53,6 +46,13 @@ jobs:
5346
CLUSTER_ID=$(echo $RANDOM | md5sum | head -c 8)
5447
echo "Using cluster ID ${CLUSTER_ID}"
5548
echo "CLUSTER_ID=$CLUSTER_ID" >> $GITHUB_ENV
49+
- name: Get AWS credentials
50+
uses: aws-actions/[email protected]
51+
with:
52+
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
53+
role-duration-seconds: 3600
54+
aws-region: ${{ secrets.AWS_REGION }}
55+
role-session-name: GithubActionsSession-${{ env.CLUSTER_ID }}
5656
- name: Create infrastructure
5757
id: create-infrastructure
5858
env:
@@ -66,7 +66,7 @@ jobs:
6666
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
6767
role-duration-seconds: 3600
6868
aws-region: ${{ secrets.AWS_REGION }}
69-
role-session-name: GithubActionsSession
69+
role-session-name: GithubActionsSession-${{ env.CLUSTER_ID }}
7070
- name: Run tests - Module
7171
env:
7272
DOCKER_BUILDKIT: 1
@@ -85,7 +85,7 @@ jobs:
8585
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
8686
role-duration-seconds: 3600
8787
aws-region: ${{ secrets.AWS_REGION }}
88-
role-session-name: GithubActionsSession
88+
role-session-name: GithubActionsSession-${{ env.CLUSTER_ID }}
8989
- name: Run tests - Cleanup
9090
if: always()
9191
env:
@@ -103,7 +103,7 @@ jobs:
103103
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
104104
role-duration-seconds: 3600
105105
aws-region: ${{ secrets.AWS_REGION }}
106-
role-session-name: GithubActionsSession
106+
role-session-name: GithubActionsSession-${{ env.CLUSTER_ID }}
107107
- name: Cleanup environment
108108
if: always()
109109
env:

hack/lib/generate-aws-creds.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
echo "Generating temporary AWS credentials..."
22

3-
ACCESS_VARS=$(aws sts assume-role --role-arn $ASSUME_ROLE --role-session-name eks-workshop-shell --output json | jq -r '.Credentials | "export AWS_ACCESS_KEY_ID=\(.AccessKeyId) AWS_SECRET_ACCESS_KEY=\(.SecretAccessKey) AWS_SESSION_TOKEN=\(.SessionToken)"')
3+
ACCESS_VARS=$(aws sts assume-role --role-arn $ASSUME_ROLE --role-session-name ${EKS_CLUSTER_NAME}-shell --output json | jq -r '.Credentials | "export AWS_ACCESS_KEY_ID=\(.AccessKeyId) AWS_SECRET_ACCESS_KEY=\(.SecretAccessKey) AWS_SESSION_TOKEN=\(.SessionToken)"')
44

55
# TODO: This should probably not use eval
66
eval "$ACCESS_VARS"

0 commit comments

Comments
 (0)