forked from kiran-sama/terraform-aws-elasticache-redis
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Merge upstream changes into fork #17
Open
WestonReed
wants to merge
96
commits into
verkada:master
Choose a base branch
from
cloudposse:main
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
* No number_cache_clusters if cluster_mode_enabled See #64 * No explicit AZs in cluster mode * Updated README.md Co-authored-by: actions-bot <[email protected]>
* Fix typo on example for 'automatic_failover' * Updated README.md Co-authored-by: actions-bot <[email protected]>
## What 1. Update Version Pinning for Terraform to support 0.13 ## Why 1. This is a relatively minor update that the CloudPosse module already likely supports. 1. This allows module consumers to not individually update our Terraform module to support Terraform 0.13.
* add_kms_key_id: Allow user to supply their own kms_key_id Change-Id: I23d1288851301328afaa61686b42d8376d303415 * Updated README.md Co-authored-by: Andy Hibbert <[email protected]> Co-authored-by: actions-bot <[email protected]>
…e53-cluster-hostname to 0.6.0 (#81)
…date to `context.tf` (#84)
* added eggress * updated example * Updated README.md Co-authored-by: actions-bot <[email protected]>
Co-authored-by: actions-bot <[email protected]>
… to v0.9.0 (#99) Co-authored-by: Renovate Bot <[email protected]>
… to v0.10.0 (#101) * chore(deps): update terraform cloudposse/route53-cluster-hostname/aws to v0.10.0 * Auto Format Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: cloudpossebot <[email protected]>
* Add snapshot name argument * Update variables.tf Co-authored-by: Vladimir <[email protected]>
* add ability to restore cluster from snapshots * Auto Format * Update variables.tf Co-authored-by: Vladimir <[email protected]> * update readme * Auto Format Co-authored-by: Sung Kang <[email protected]> Co-authored-by: cloudpossebot <[email protected]> Co-authored-by: Vladimir <[email protected]>
… to v0.10.1 (#105) * chore(deps): update terraform cloudposse/route53-cluster-hostname/aws to v0.10.1 * Auto Format Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: cloudpossebot <[email protected]>
…ped to 0.13.0 when needed, readme updated (#107)
Co-authored-by: cloudpossebot <[email protected]> Co-authored-by: actions-bot <[email protected]>
* allow empty list of availability_zones availability_zones parameter shouldn't be required * enable more nodes to be created than AZs * Updated README.md Co-authored-by: actions-bot <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Default var.transit_encryption_enabled to false * trans enc to true if token unless explicitly set * Update variables.tf * Auto Format * Trigger validate-codeowners Co-authored-by: cloudpossebot <[email protected]> Co-authored-by: PePe Amengual <[email protected]>
* Default transit_encryption_enabled to true due to bridgecrew * Update README.yaml * Auto Format * Trigger validate-codeowners Co-authored-by: cloudpossebot <[email protected]> Co-authored-by: PePe Amengual <[email protected]>
* fix: make the security group description configurable * Add null value in the variable description * Auto Format Co-authored-by: nitro <[email protected]> Co-authored-by: cloudpossebot <[email protected]>
Co-authored-by: Joe Niland <[email protected]> Co-authored-by: actions-bot <[email protected]>
* Add TF attribute member-clusters to available outputs * make init && make github/init * README * README updates
* feat: allow final snapshot creation * Auto Format Co-authored-by: cloudpossebot <[email protected]>
Co-authored-by: max-lobur <[email protected]>
* feat: Allow configuring auth_token_update_strategy * break: Upgrade AWS provider to at least 5.27.0 --------- Co-authored-by: Igor Rodionov <[email protected]>
* feat: add support for redis serverless * update docs * fix outputs * update docs * improve checks * update docs * make dns and outputs work with serverless * make address work * fix description and empty cache_usage_limits entries * prevent empty cache_usage_limits block * rework serverless_cache_usage_limits * update docs * Update main.tf Co-authored-by: Benjamin Smith <[email protected]> * Update main.tf Co-authored-by: Benjamin Smith <[email protected]> --------- Co-authored-by: Benjamin Smith <[email protected]>
* feat: Allow configuring transit_encryption_mode * fix(tests): bump redis to 7.x * fix(tests): set redis to 7.2 * fix(tests): set family to redis7 * fix(tests): set engine version to 7 * Update examples/complete/fixtures.us-east-2.tfvars * chore: Update README.md --------- Co-authored-by: Veronika Gnilitska <[email protected]> Co-authored-by: nitrocode <[email protected]>
…n_enabled is false (#238) * fix: Default transit_encryption_mode to null if var.transit_encryption_enabled is false * refactor: Change transit_encryption_mode default to null * chore: Remove not needed conditional
Co-authored-by: Igor Rodionov <[email protected]>
Bumps the go_modules group with 1 update in the /test/src directory: [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter). Updates `github.com/hashicorp/go-getter` from 1.7.4 to 1.7.5 - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Changelog](https://github.com/hashicorp/go-getter/blob/main/.goreleaser.yml) - [Commits](hashicorp/go-getter@v1.7.4...v1.7.5) --- updated-dependencies: - dependency-name: github.com/hashicorp/go-getter dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Required when running in cluster mode as there is no reader endpoint Co-authored-by: Kevin <[email protected]>
This change allows setting the `engine` variable, to allow using Valkey as the cache engine. No breaking changes are expected, as the default value for the `engine` variable is set to `redis`. Also, the minimum AWS provider version is not changed as using Redis does not require using a newer provider.
#221) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Merge upstream changes into fork