-
Notifications
You must be signed in to change notification settings - Fork 0
ci: Add workflow to launch a tools container. #30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
6124a3c
to
10456c2
Compare
Plan output for service config
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a GitHub workflow to launch a tools container within the infrastructure, alongside refactoring OpenTofu setup code and updating infrastructure configurations.
- Adds a new GitHub workflow
launch-tools.yaml
for running containers in ECS environments - Refactors OpenTofu setup logic into a reusable GitHub action to reduce duplication
- Updates Terraform configurations to support zero database instances and expose new outputs
Reviewed Changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
.github/workflows/launch-tools.yaml |
New workflow for launching tools containers in ECS environments |
.github/actions/setup-opentofu/action.yaml |
New reusable action for OpenTofu setup and initialization |
.github/workflows/plan.yaml |
Refactored to use the new setup-opentofu action |
.github/workflows/deploy.yaml |
Refactored to use the new setup-opentofu action |
tofu/modules/system/variables.tf |
Updated database instance count validation to allow zero instances |
tofu/modules/system/outputs.tf |
Added task security group ID output |
tofu/config/service/variables.tf |
Added database instance count variable with validation |
tofu/config/service/outputs.tf |
Added multiple new outputs for container configuration |
tofu/config/service/main.tf |
Updated to use new database instance count variable and image tag local |
tofu/config/service/locals.tf |
New file containing image tag calculation logic |
tofu/modules/ephemeral_service/versions.tf |
Updated Docker provider version from ~> 3.6 to ~> 3.7 |
tofu/modules/ephemeral_service/docker.tf |
Added auth_config for docker_image and reordered auth_config fields |
Files not reviewed (1)
- tofu/config/service/.terraform.lock.hcl: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
10456c2
to
9728505
Compare
No description provided.