Skip to content

Commit

Permalink
test: Provide IAM permission to `cloudformation:DescribeStackResource…
Browse files Browse the repository at this point in the history
…s` (#4786)
  • Loading branch information
jonathan-innis authored Oct 11, 2023
1 parent 94aba36 commit 42ff5dc
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 4 deletions.
10 changes: 9 additions & 1 deletion .github/actions/e2e/create-cluster/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ runs:
cmd="create"
eksctl get cluster --name ${{ inputs.cluster_name }} && cmd="upgrade"
eksctl ${cmd} cluster -f - <<EOF
cat << EOF >> clusterconfig.yaml
---
apiVersion: eksctl.io/v1alpha5
kind: ClusterConfig
Expand Down Expand Up @@ -146,6 +146,14 @@ runs:
wellKnownPolicies:
ebsCSIController: true
EOF
eksctl ${cmd} cluster -f clusterconfig.yaml
# We need to call these update iamserviceaccount commands again since the "eksctl upgrade cluster" action
# doesn't handle updates to IAM serviceaccounts correctly when the roles assigned to them change
eksctl update iamserviceaccount -f clusterconfig.yaml --approve
- name: tag oidc provider of the cluster
if: always()
shell: bash
Expand Down
7 changes: 4 additions & 3 deletions .github/workflows/e2e-matrix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,10 @@ jobs:
e2e-upgrade:
uses: ./.github/workflows/e2e-upgrade.yaml
with:
# This version matches the steps of the newest version of the install-eksctl action
# which will take in the eksctl_version into the composite action
from_git_ref: 3519331035579ac0caf66a7f5a5282a2fef9b409
# This version matches the steps of the newest version that contains the additional step
# of deploying the instance profile so that the pre-upgrade and post-upgrade create-cluster
# actions have the same number of steps and don't fail during post-cleanup
from_git_ref: 62c25a3ea85c7d00165e60a913fff1ec7c1f29fd
to_git_ref: ${{ inputs.git_ref }}
region: ${{ inputs.region }}
k8s_version: ${{ inputs.k8s_version }}
Expand Down
1 change: 1 addition & 0 deletions test/cloudformation/iam_cloudformation.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,7 @@ Resources:
- cloudformation:DeleteStack
- cloudformation:DescribeChangeSet
- cloudformation:DescribeStackEvents
- cloudformation:DescribeStackResources
- cloudformation:ExecuteChangeSet
- cloudformation:GetTemplate
- cloudformation:GetTemplateSummary
Expand Down

0 comments on commit 42ff5dc

Please sign in to comment.