Repository navigation
default AZ for the region - #12
Conversation
| description = "a mapping of the default availability zones for each AWS region" | ||
|
|
||
| default = { | ||
| us-east-1 = "us-east-1a,us-east-1b,us-east-1c,us-east-1e" |
There was a problem hiding this comment.
Nice this looks good.
This reminds me, we may want to re-work how we're doing the VPC networking in this case since we'd also need the CIDR blocks depending on the count of availability zones.
We could definitely achieve it, but we'll need a bit more to get there, something like:
count = "${length($split(",", var.availability_zones))}"
|
one note about the different az-counts, lgtm otherwise! |
60c3207 to
5fc69f2
Compare
|
I revisited the implementation, gave up on trying to generate stuff dynamically and instead added documentation to help understand what needs to be set. |
|
Sounds fair to me. Yeah, I wouldn't want to break too many of the abstractions just to handle this other case |
@segmentio/infra
This adds a mapping of default availability zones for each AWS region, that way when a
stackmodule is instantiated, only the region can be specified and the config dynamically figures out what AZ to use.Note that the AZ may be changing from time to time so we'll likely have to edit these values in the future.