-
-
Notifications
You must be signed in to change notification settings - Fork 603
feat: Add support for restartPolicy #231
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 restartPolicy #231
Conversation
It now works fine (fixed the validation test on var.restart_policy) |
@bryantbiggs is the "Optional object type attributes are experimental" fail a problem? I believe it's long been in Terraform core (since 1.3) |
it was introduced in Terraform 1.3, but on |
Cheers @bryantbiggs. |
This is amazing! When will a new tag release be done? So we can start using it? |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
ECS now supports the ability to restart containers when they crash.
Supporting documentation:
Motivation and Context
Fixes #230
Breaking Changes
Shouldn't be breaking as older terraform provider versions will just ignore restartPolicy and a default value is provided with enabled = false (default behaviour when it's absent)
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request