Support import on aws_backup_vault resource

[martcollins]

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"
}

[programmer04]

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.

[bflad]

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!

[martcollins]

Thanks @programmer04, awesome!

[bflad]

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.

[ghost]

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!