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

Fix userdata variables for aws-asg #31

Merged
merged 1 commit into from
Nov 3, 2022
Merged

Conversation

GMartinez-Sisti
Copy link
Contributor

Summary

Fix userdata variables for aws-asg

Description

  • Adds example for non-ha deploy
  • Fixes error when replicas are 1:
╷
│ Error: Invalid index
│
│   on ../../main.tf line 298, in resource "aws_launch_template" "launch_template":
│  298:       redis_primary_endpoint_address    = aws_elasticache_replication_group.redis[0].primary_endpoint_address,
│     ├────────────────
│     │ aws_elasticache_replication_group.redis is empty tuple
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│
│   on ../../main.tf line 299, in resource "aws_launch_template" "launch_template":
│  299:       redis_port                        = aws_elasticache_replication_group.redis[0].port,
│     ├────────────────
│     │ aws_elasticache_replication_group.redis is empty tuple
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Theses changes have been tested in:

  • Local environment

@GMartinez-Sisti GMartinez-Sisti self-assigned this Nov 3, 2022
@GMartinez-Sisti GMartinez-Sisti requested a review from a team as a code owner November 3, 2022 16:21
@GMartinez-Sisti GMartinez-Sisti merged commit 45ec70a into main Nov 3, 2022
@GMartinez-Sisti GMartinez-Sisti deleted the fix-aws-asg-user-data branch November 3, 2022 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants