-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
Adding rds snapshots #2062
Adding rds snapshots #2062
Conversation
Can you merge/rebase from Thanks! |
…g-rds-snapshots * 'master' of github.com:reverbdotcom/terraform: (524 commits) docs: tweaks to RELEASING Minor change to docs Update CHANGELOG.md Update DynamoDB example docs to remove non-key attributes; update test to remove non-key attribute from attribute set to prevent infinite planning loops Update CHANGELOG.md use /usr/bin/env bash provider/aws: fix go vet provider/aws: ignore providers with Meta nil update CHANGELOG provider/aws: Code cleanups for Spot Requests provider/aws: fix db_subnet acc test Fixing the tests Fixes issue hashicorp#2568 Update CHANGELOG.md Update CHANGELOG.md fixes typo Fixed void Azure network config bug. provider/aws: ecs task definition is deregistered correctly provider/azure: fixup storage service test provider/docker: [tests] change images ...
@catsby ok pushed! Looking forward to being able to use this in master. Cheers. |
Everything checks out, I can still make dbs, dbs, with replicas, and now restore from a snapshot, thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Adding support for the ability to create an RDS instance from a previously saved DB snapshot. This pr adds the config option(snapshot_identifier) to the aws_db_instance resource and the conditional logic for handling this type of use case.