Skip to content

Commit

Permalink
Address test issues; PR review.
Browse files Browse the repository at this point in the history
  • Loading branch information
korenyoni committed Aug 19, 2024
1 parent ad7ba2a commit 03e5e04
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 3 deletions.
1 change: 1 addition & 0 deletions .github/workflows/test-matrix-2-levels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
- uses: ./
id: current
with:
iam-auth-enabled: false
install-atmos: false
atmos-config-path: "./tests"
nested-matrices-count: '2'
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/test-matrix-3-levels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
- uses: ./
id: current
with:
iam-auth-enabled: false
install-atmos: false
atmos-config-path: "./tests"
nested-matrices-count: '3'
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/test-no-changes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ jobs:
- uses: ./
id: current
with:
iam-auth-enabled: false
install-atmos: true
atmos-config-path: "./tests"

Expand Down
1 change: 1 addition & 0 deletions .github/workflows/test-positive.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
- uses: ./
id: current
with:
iam-auth-enabled: false
install-atmos: false
atmos-config-path: "./tests"

Expand Down
8 changes: 5 additions & 3 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,10 @@ inputs:
description: Whether to include dependents of affected stacks in the output
required: false
default: "false"
iam-auth-enabled:
description: Whether to assume the Terraform plan IAM role prior to running atmos commands
required: false
default: "true"
install-jq:
description: Whether to install jq
required: false
Expand Down Expand Up @@ -113,9 +117,6 @@ runs:
echo "group-by=$(atmos describe config -f json | jq -r '.integrations.github.gitops.matrix["group-by"]')" >> $GITHUB_OUTPUT
echo "sort-by=$(atmos describe config -f json | jq -r '.integrations.github.gitops.matrix["sort-by"]')" >> $GITHUB_OUTPUT
echo "aws-region=$(atmos describe config -f json | jq -r '.integrations.github.gitops["artifact-storage"].region')" >> $GITHUB_OUTPUT
echo "terraform-state-role=$(atmos describe config -f json | jq -r '.integrations.github.gitops["artifact-storage"].role')" >> $GITHUB_OUTPUT
echo "terraform-state-table=$(atmos describe config -f json | jq -r '.integrations.github.gitops["artifact-storage"].table')" >> $GITHUB_OUTPUT
echo "terraform-state-bucket=$(atmos describe config -f json | jq -r '.integrations.github.gitops["artifact-storage"].bucket')" >> $GITHUB_OUTPUT
echo "terraform-plan-role=$(atmos describe config -f json | jq -r '.integrations.github.gitops.role.plan')" >> $GITHUB_OUTPUT
- name: Install Terraform
Expand Down Expand Up @@ -149,6 +150,7 @@ runs:
working-directory: base-ref

- name: Configure Plan AWS Credentials
if: ${{ inputs.iam-auth-enabled == 'true' }}
uses: aws-actions/configure-aws-credentials@v4.0.2
with:
aws-region: ${{ steps.config.outputs.aws-region }}
Expand Down

0 comments on commit 03e5e04

Please sign in to comment.