Skip to content
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

support more nodes than there are AZs defined #28

Merged
merged 2 commits into from
Feb 3, 2021

Conversation

gusse
Copy link
Contributor

@gusse gusse commented Feb 2, 2021

for loop to produce a list with the same amount of AZs as there are cache nodes. element() supports "wrap around" making it loop over the existing elements in var.availability_zones even if there aren't enough unique AZs for each node.

what

  • Different method to populate the list of AZs that needs to match the number of cache nodes defined.

why

  • Should be able to run more nodes than there are AZs defined in variable. Or for example more than 6 nodes (max number of AZs in any AWS region) in us-east-1 without providing a list with duplicate AZ names.

references

for loop to produce a list with the same amount of AZs as there are cache nodes. element() supports "wrap around" making it loop over the existing elements in var.availability_zones even if there aren't enough unique AZs for each node.
@gusse gusse requested review from a team as code owners February 2, 2021 15:17
@gusse gusse requested review from jamengual and woz5999 and removed request for a team February 2, 2021 15:17
@gusse gusse requested a review from a team as a code owner February 2, 2021 15:17
@SweetOps
Copy link
Contributor

SweetOps commented Feb 3, 2021

/test all

@SweetOps SweetOps added the bug 🐛 An issue with the system label Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Defining more cache nodes than you have availability zones is not supported
3 participants