fix: Enable transit_encryption_enabled for all avialble engines now that Elasticache supports this
#56
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cmanzur
changed the title
cmanzur
changed the title
transit_encryption_enabled for all engines
bryantbiggs
changed the title
transit_encryption_enabled for all enginestransit_encryption_enabled for all avialble engines now that Elasticache supports this
bryantbiggs
approved these changes
Oct 7, 2025
antonbabenko
pushed a commit
that referenced
this pull request
Oct 7, 2025
## [1.10.1](v1.10.0...v1.10.1) (2025-10-07) ### Bug Fixes * Enable `transit_encryption_enabled` for all avialble engines now that Elasticache supports this ([#56](#56)) ([2ee85ee](2ee85ee))
|
This PR is included in version 1.10.1 🎉 |
|
This was supposed to be released as a breaking change. As soon the module version was updated, clusters were trying to be recreated because of this default variable. |
bryantbiggs
added a commit
to bryantbiggs/terraform-aws-elasticache
that referenced
this pull request
Oct 7, 2025
…gines now that Elasticache supports this terraform-aws-modules#56
3 tasks
|
apologies for that - the removal of the prior restriction appears to clash with the current default behavior of the module to enforce encryption we'll revert for now in #57 and save this change for an upcoming breaking change that will be released under a new major version number |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Support transit_encryption_enabled for all Engines
Motivation and Context
transit_encryption_enabledwas disabled by default for Redis Engine.Resolves #55
Breaking Changes
No
How Has This Been Tested?
examples/*to demonstrate and validate my change(s)examples/*projectspre-commit run -aon my pull request