-
-
Notifications
You must be signed in to change notification settings - Fork 643
docs: Exclude local zones from aws_availability_zones data source in examples when retrieving list of available zones
#378
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
base: master
Are you sure you want to change the base?
Conversation
|
|
||
| data "aws_availability_zones" "available" {} | ||
| data "aws_availability_zones" "available" { | ||
| filter { |
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.
can we add a note so folks know why we do this https://github.com/terraform-aws-modules/terraform-aws-eks/blob/c41b58277ab3951eca8d11863edf178135ec7654/examples/eks-auto-mode/main.tf#L6
|
if we're going to add it, lets add it to all examples to be consistent |
aws_availability_zones data source in examples when retrieving list of available zones
|
@bryantbiggs I have updated everything you asked for, and ran a fresh |
Description
This change excludes local zones that the user may have opted into in their region of choice
Motivation and Context
Without this change, the examples may attempt to deploy to local zones, which will cause failures of the example apply
Breaking Changes
none
How Has This Been Tested?
examples/*to demonstrate and validate my change(s)/examples/completeexamples/*projectspre-commit run -aon my pull request