-
-
Notifications
You must be signed in to change notification settings - Fork 559
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
feat!: Add support for creating ECS service and container definition #76
Merged
Merged
Changes from 37 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
d45632c
feat: Add support for ECS service
bryantbiggs 1b0bb21
feat: Add support for ECS task definition
bryantbiggs bcae52d
feat: Add support for ECS service IAM role
bryantbiggs 718353c
feat: Add support for ECS task execution and ECS tasks IAM roles
bryantbiggs e6c17dd
feat: Add support for ECS task sets
bryantbiggs 1f5f485
feat: Add support for container definition sub-module
bryantbiggs a4f3fcd
feat: Update service module to latest spec and integrate `container-d…
bryantbiggs 5255af5
chore: Update examples to use service sub-module
bryantbiggs c59b22a
feat: Add support for service autoscaling
bryantbiggs 986e2b2
feat: Setting defaults based on docs and ECS guidance
bryantbiggs 114430e
feat: Add support for ECS service connect
bryantbiggs 5750dff
feat: Add support for timeouts and triggers on ECS service
bryantbiggs 7cb25cf
refactor: Switch to default to ignoring scaling and allow ignoring ta…
bryantbiggs e2d3a92
fix: Use `lookup()` for service connect namespace since this can be a…
bryantbiggs 3e3106f
fix: Use `null` for default value on capacity provider strategy to av…
bryantbiggs eea18bc
fix: Set default cluster capacity provider weight to `null`
bryantbiggs 5c56117
fix: Set launch type correctly when not an external deployment
bryantbiggs 1259a58
feat: Add support for creating shared task execution role plus secret…
bryantbiggs c3614f3
fix: Add output for shared task execution role
bryantbiggs 6867416
fix: Ensure Sids are unique
bryantbiggs dd81a01
feat: Add support for creating log group for cluster
bryantbiggs 3c35e5f
feat: Add support for creating security group for service/task
bryantbiggs ef28bab
feat: Output the `family` attribute for task definition
bryantbiggs 9ab549e
feat: Allow for enable/disabling creation of CloudWatch logging and t…
bryantbiggs 36b68eb
fix: Ensure that the latest task definition is used
bryantbiggs dd756ba
fix: Correct logging configuration for container definition
bryantbiggs 1a07577
chore: Use SSM for fluentbit image
bryantbiggs c7600d4
fixed service connect port_name parsing (#2)
nvpnathan dd6c56b
feat: Add support for extending IAM policy permissions with generic `…
bryantbiggs 70d4dd5
fix: Correct spelling errors and version mis-match
bryantbiggs 8039894
chore: Update documentation
bryantbiggs be70968
chore: Update readme
bryantbiggs 73acc1d
fix: Ensure default values are present which matches managed policy f…
bryantbiggs c92aadf
Update modules/service/README.md
bryantbiggs 37103e3
Update main.tf
bryantbiggs ad37f81
chore: Add docs
bryantbiggs d3a4ff3
Update DESIGN.md (#3)
arvindsoni80 1db3647
Apply suggestions from code review
bryantbiggs 16442e0
feat: Add support for generating Terragrunt wrappers
bryantbiggs 98218e4
refactor: Move root cluster into module; add both cluster and service…
bryantbiggs c6fef3b
chore: Updates from PR review
bryantbiggs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Please add this hook after terraform_fmt: