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

resource/aws_db_instance: Add deletion_protection argument #6011

Merged
merged 1 commit into from
Oct 3, 2018

Conversation

bflad
Copy link
Contributor

@bflad bflad commented Sep 27, 2018

I will submit an AWS Support case regarding the unexpected behavior with CreateDBInstanceReadReplica, but that should not hold up this enhancement from review.

AWS Support has confirmed the CreateDBInstanceReadReplica issue and noted that it will be fixed at some point in the future.

Closes #6006

Changes proposed in this pull request:

  • Add deletion_protection argument to aws_db_instance resource

Output from acceptance testing:

--- PASS: TestAccAWSDBInstance_basic (431.32s)
--- PASS: TestAccAWSDBInstance_DeletionProtection (452.56s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DeletionProtection (1346.38s)

@bflad bflad added the enhancement Requests to existing resources that expand the functionality or scope. label Sep 27, 2018
@bflad bflad requested a review from a team September 27, 2018 03:54
@ghost ghost added size/L Managed by automation to categorize the size of a PR. documentation Introduces or discusses updates to documentation. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. labels Sep 27, 2018
@bflad bflad added the service/rds Issues and PRs that pertain to the rds service. label Sep 27, 2018
```
--- PASS: TestAccAWSDBInstance_basic (431.32s)
--- PASS: TestAccAWSDBInstance_DeletionProtection (452.56s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DeletionProtection (1346.38s)
```
@bflad bflad force-pushed the f-aws_db_instance-deletion_protection branch from 83388ca to 6851c6d Compare September 27, 2018 14:26
@ghost ghost added size/L Managed by automation to categorize the size of a PR. documentation Introduces or discusses updates to documentation. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. labels Sep 27, 2018
@ozbillwang
Copy link

Same comment as #6010

#6010 (comment)

@bflad
Copy link
Contributor Author

bflad commented Oct 3, 2018

@ozbillwang long form answer in #6010 (comment)

@bflad bflad added this to the v1.39.0 milestone Oct 3, 2018
@bflad bflad merged commit 3b0f65d into master Oct 3, 2018
@bflad bflad deleted the f-aws_db_instance-deletion_protection branch October 3, 2018 16:45
bflad added a commit that referenced this pull request Oct 3, 2018
@ghost
Copy link

ghost commented Oct 3, 2018

This has been released in version 1.39.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "aws" {
	version = "~> 1.39.0"
}
# ... other configuration ...

@ghost
Copy link

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

@ghost ghost locked and limited conversation to collaborators Apr 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Introduces or discusses updates to documentation. enhancement Requests to existing resources that expand the functionality or scope. service/rds Issues and PRs that pertain to the rds service. size/L Managed by automation to categorize the size of a PR. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RDS Deletion Protection API Support (aws_db_instance)
3 participants