-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: Support Multi Region Cluster Name argument #12
feat: Support Multi Region Cluster Name argument #12
Conversation
modules/multi-region-cluster/main.tf
Outdated
@@ -0,0 +1,21 @@ | |||
resource "aws_memorydb_multi_region_cluster" "this" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should leave this sub-module out for now - we don't want to re-wrap just a single resource, it doesn't really provide any value. for now, I think the state of multi-region cluster would be this standalone resource plus multiple instantiations of the memorydb module
examples/multi-region/main.tf
Outdated
@@ -0,0 +1,237 @@ | |||
provider "aws" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets leave out this example for now as well
## [2.3.0](v2.2.0...v2.3.0) (2025-01-07) ### Features * Support Multi Region Cluster Name argument ([#12](#12)) ([1a7c7ce](1a7c7ce))
This PR is included in version 2.3.0 🎉 |
@antonbabenko : Can you please advise, when this version will be available on Terraform registry ? As currently it shows 2.2.0 as the latest ![]() |
@shiv4nsh Fixed. |
Description
Support multi-region memorydb cluster name argument.
Motivation and Context
multi_region_cluster
resource hashicorp/terraform-provider-aws#40376Breaking Changes
No.
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request