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

provider/aws: Add a new aws_rds_snapshot data source #10287

Closed
edmorley opened this issue Nov 22, 2016 · 8 comments · Fixed by #10291
Closed

provider/aws: Add a new aws_rds_snapshot data source #10287

edmorley opened this issue Nov 22, 2016 · 8 comments · Fixed by #10291

Comments

@edmorley
Copy link

edmorley commented Nov 22, 2016

#10017 recently added the aws_ebs_snapshot data source, fixing #8828.

It would be great if we could have something similar for AWS RDS snapshots, which could be used like so:

data "aws_rds_snapshot" "example" {
  most_recent = true
  filter {
    name = "name"
    values = ["foo-prod"]
  }
  owners = ["self"]
}

resource "aws_db_instance" "foo-dev-rds" {
    identifier = "foo-dev"
    snapshot_identifier = "${data.aws_rds_snapshot.example.id}"
    # ...
}

This would allow for the workflow of:

  • taint existing development/staging RDS instance
  • re-run terraform to recreate dev/stage RDS instance from latest snapshot of prod RDS

...without having to hard-code (and continually update) RDS snapshot identifier names like "rds:foo-prod-2016-11-30-07-05" in the Terraform config.

@edmorley edmorley changed the title provider/aws: Add a new aws_rds_snapshot resource & data source provider/aws: Add a new aws_rds_snapshot data source Nov 22, 2016
@stack72
Copy link
Contributor

stack72 commented Nov 30, 2016

Hi @edmorley

I added the PR above - please note that this specific use case is only for db-snapshots NOT db-cluster-snapshots

That second usecase is on the way :)

Paul

@stack72
Copy link
Contributor

stack72 commented Nov 30, 2016

Closed via #10291

@stack72 stack72 closed this as completed Nov 30, 2016
@edmorley
Copy link
Author

Many thanks!

@joshuaspence
Copy link
Contributor

But #10291 hasn't been merged yet?

@gtmtech
Copy link

gtmtech commented Mar 23, 2017

Did someone forget about this @stack72 ? It seems to have been closed, but not merged, so... not really closed(?)

@stack72
Copy link
Contributor

stack72 commented Mar 23, 2017

Hi @gtmtech

This will be finished this week - I promise!

Paul

@Seraf
Copy link

Seraf commented Apr 9, 2017

Hello, is there some update ? Any chance to get this with the 0.9.3 please ? =)
It would be really awesome !

@ghost
Copy link

ghost commented Apr 14, 2020

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.

@ghost ghost locked and limited conversation to collaborators Apr 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants