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

Allow using an externally defined db parameter group #34

Merged

Conversation

gimoh
Copy link
Contributor

@gimoh gimoh commented Sep 15, 2017

Add two optional input variables, one to indicate an external parameter group should be used instead of the built-in one, and the other to specify the name of the group to use. For now, a single variable doesn't appear to be sufficient due to hashicorp/terraform#12570

Fixes #29

@gstlt
Copy link
Contributor

gstlt commented Sep 15, 2017

Cool stuff! :)

@antonbabenko
Copy link
Member

LGTM

@antonbabenko antonbabenko merged commit 6a41a0f into terraform-community-modules:master Sep 15, 2017
@antonbabenko
Copy link
Member

@gimoh
Copy link
Contributor Author

gimoh commented Sep 15, 2017

yay 🎉 cheers guys!

@gimoh gimoh deleted the external-param-group branch September 16, 2017 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants