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.

Sign up for GitHub

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

Jump to bottom

No ElastiCache subnet group name output #58

Closed
jekstrom opened this issue Jan 18, 2018 · 2 comments
Closed

No ElastiCache subnet group name output #58

jekstrom opened this issue Jan 18, 2018 · 2 comments

Comments

@jekstrom
Copy link
Contributor

ElastiCache cluster has a subnet_group_name argument, but no subnet_group_id argument, so the output variable elasticache_subnet_group is not useful in this regard.

Should a elasticache_subnet_group_name output be added?
@antonbabenko
Copy link
Member

I think you are right. We should export all available attributes.

PR is welcome (as usual) :)

jekstrom added a commit to jekstrom/terraform-aws-vpc that referenced this issue Jan 19, 2018
@jekstrom
Issue terraform-aws-modules#58: Add ElastiCache subnet group name out…
79fa50d 
…put.
@jekstrom jekstrom mentioned this issue Jan 19, 2018
Merged
antonbabenko pushed a commit that referenced this issue Jan 21, 2018
@jekstrom @antonbabenko
Issue #58: Add ElastiCache subnet group name output. (#60)
fa68b2f
@github-actions
Copy link

github-actions bot commented Nov 6, 2022

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@antonbabenko @jekstrom