Terraform-provider-aws: Support import on aws_backup_vault resource

Created on 31 May 2019  ·  5Comments  ·  Source: hashicorp/terraform-provider-aws

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

The aws_backup_vault resource doesn't yet allow import. This would be useful for users with existing backup vaults. Issue found using version 2.12 of the provider.

$ terraform import aws_backup_vault.test-vault test-vault
aws_backup_vault.test-vault: Importing from ID "test-vault"...

Error: aws_backup_vault.test-vault (import id: test-vault): import aws_backup_vault.test-vault (id: test-vault): resource aws_backup_vault doesn't support import

New or Affected Resource(s)

  • aws_backup_vault

Potential Terraform Configuration

resource "aws_backup_vault" "test-vault" {
  name = "TestVault"
}
enhancement good first issue servicbackup

All 5 comments

My PR #9041 adds it, but there is a bug in AWS API for non-existent backup_vault, returns the misleading error code (403 instead of 404). Bug's been reported to AWS here and should be fixed, but it is independent of merging this PR.

Support for importing aws_backup_vault resources has been merged and will be released in version 2.16.0 of the Terraform AWS Provider, likely tomorrow. 👍 Thanks, @programmer04!

Thanks @programmer04, awesome!

This has been released in version 2.16.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

Was this page helpful?
0 / 5 - 0 ratings