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

add transit encryption variable for elasticache cluster resource #64

Closed

Conversation

jfineberg
Copy link

No description provided.

@jfineberg jfineberg requested review from a team as code owners November 8, 2023 18:51
@jfineberg jfineberg closed this Nov 8, 2023
@jfineberg jfineberg deleted the add-transit-encryption-var branch November 8, 2023 18:54
@jfineberg
Copy link
Author

Sorry! Intended to open this on a forked repo. If this looks useful, I can open it again with a proper PR template.

@LuisDeHaro
Copy link
Contributor

@jfineberg @dferramosi this would actually be very helpful to have and let folks decide wether or not to enable Encryption in Transit. I'm trying to create new clusters with transit_encryption_enabled but can't do that with the current module as is.

I can open a new PR with the exact same changes as well. @dferramosi wdyt?

@dferramosi
Copy link

I'm not a cloudposse approver, not sure how i was able to approve - but yeah, that sounds great!

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