-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
examples/ecs-alb: Replace template_file data sources #23395
examples/ecs-alb: Replace template_file data sources #23395
Conversation
Update the examples to use the new templatefile() function. The template provider has been deprecated since 0.12 and is not available on all platforms, such as arm64.
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.
Welcome @democracytoday 👋
It looks like this is your first Pull Request submission to the Terraform AWS Provider! If you haven’t already done so please make sure you have checked out our CONTRIBUTING guide and FAQ to make sure your contribution is adhering to best practice and has all the necessary elements in place for a successful approval.
Also take a look at our FAQ which details how we prioritize Pull Requests for inclusion.
Thanks again, and welcome to the community! 😃
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.
Looks good to me! Thank you for your contribution! I'll get this merged in once the current release process finishes up and the checks have completed! 🚀
Update: looks like the checks didn't like some of the formatting of |
Great, thanks for handling the formatting changes! |
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. |
Update the examples to use the "new"
templatefile()
function.The template provider has been deprecated since 0.12 and is not available on all platforms, such as arm64.
Community Note
Relates #22739
Output from acceptance testing: N/A, this is only a documentation update.