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 support for replicate_source_db variable #113

Closed
adamantike opened this issue Apr 26, 2021 · 2 comments · Fixed by #120
Closed

Add support for replicate_source_db variable #113

adamantike opened this issue Apr 26, 2021 · 2 comments · Fixed by #120

Comments

@adamantike
Copy link

Describe the Feature

Currently, this module cannot be used to create a read replica, as the replicate_source_db variable is not exposed. cloudposse/terraform-aws-rds-replica seems to be the alternative, but hasn't received any updates in over two years.

Use Case

Using this same module both for creating a RDS primary database, and its read replicas.

Describe Ideal Solution

Add the replicate_source_db variable in this module, and consider if there are variables that only must be provided to the aws_db_instance resource when this option is used.

Alternatives Considered

cloudposse/terraform-aws-rds-replica seems to be the alternative module provided by CloudPosse, but hasn't received any updates in over two years.

@nitrocode
Copy link
Member

@adamantike have you been able to use the other module to create a read replica ?

If we add replicate_source_db variable to this module (and other inputs when this is used), would that not render the other module obsolete ?

@adamantike
Copy link
Author

adamantike commented Jul 19, 2021

@nitrocode, I think it will! However, that module hasn't been updated in more than two years, so it makes sense to either update it with all the same feature this module currently provides (plus, support for newer Terraform versions), or centralize both functionalities here.

As stated in aws_db_instance docs:

NOTE: Removing the replicate_source_db attribute from an existing RDS Replicate database managed by Terraform will promote the database to a fully standalone database.

So, promoting replicas would be simpler just by using the same module, and not requiring Terraform state reimports.

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 a pull request may close this issue.

2 participants