Skip to content

Commit

Permalink
update deployment to be more configurable via secrets and variables
Browse files Browse the repository at this point in the history
  • Loading branch information
alismx committed Oct 15, 2024
1 parent 6f74a20 commit 16cbd35
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
11 changes: 4 additions & 7 deletions .github/workflows/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,7 @@ permissions:
contents: read

env:
aws_region: us-east-1
environment: ${{ github.event.inputs.environment }}
owner: "skylight"
project: "dibbs"

jobs:
terraform:
Expand All @@ -45,16 +42,16 @@ jobs:
with:
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
role-session-name: githubDeploymentWorkflow
aws-region: ${{ env.aws_region }}
aws-region: ${{ secrets.AWS_REGION }}

- name: Terraform
env:
ENVIRONMENT: ${{ env.environment }}
BUCKET: ${{ secrets.TFSTATE_BUCKET }}
DYNAMODB_TABLE: ${{ secrets.TFSTATE_DYNAMODB_TABLE }}
REGION: ${{ env.aws_region }}
OWNER: ${{ env.owner }}
PROJECT: ${{ env.project }}
REGION: ${{ secrets.AWS_REGION }}
OWNER: ${{ vars.OWNER }}
PROJECT: ${{ vars.PROJECT }}
shell: bash
run: |
echo "ENVIRONMENT=$ENVIRONMENT" >> .env
Expand Down
2 changes: 1 addition & 1 deletion terraform/implementation/ecs/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -35,5 +35,5 @@ module "ecs" {
# internal = false

# If the intent is to disable authentication, set ecr_viewer_app_env to "test" (default is "prod")
ecr_viewer_app_env = "test"
# ecr_viewer_app_env = "test"
}

0 comments on commit 16cbd35

Please sign in to comment.