From 138c0d79e15bee3d2d2cae92a3f959aca57f73e9 Mon Sep 17 00:00:00 2001 From: Wangchong Zhou Date: Thu, 22 Aug 2019 13:19:02 -0700 Subject: [PATCH] Update db_instance.html.markdown The description of `final_snapshot_identifier` doesn't feel correct to me. When `skip_final_snapshot` is by default set to false and `final_snapshot_identifier` is omitted, the provider errored with > RDS Cluster FinalSnapshotIdentifier is required when a final snapshot is require Related: https://github.com/terraform-providers/terraform-provider-aws/issues/4910 --- website/docs/r/db_instance.html.markdown | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/website/docs/r/db_instance.html.markdown b/website/docs/r/db_instance.html.markdown index 1f5baad3c37..4c2d2bae1b1 100644 --- a/website/docs/r/db_instance.html.markdown +++ b/website/docs/r/db_instance.html.markdown @@ -118,7 +118,8 @@ this attribute will ignore differences in the patch version automatically (e.g. For supported values, see the EngineVersion parameter in [API action CreateDBInstance](https://docs.aws.amazon.com/AmazonRDS/latest/APIReference/API_CreateDBInstance.html). Note that for Amazon Aurora instances the engine version must match the [DB cluster](/docs/providers/aws/r/rds_cluster.html)'s engine version'. * `final_snapshot_identifier` - (Optional) The name of your final DB snapshot -when this DB instance is deleted. If omitted, no final snapshot will be made. +when this DB instance is deleted. Must be provided if `skip_final_snapshot` is +set to `false`. * `iam_database_authentication_enabled` - (Optional) Specifies whether or mappings of AWS Identity and Access Management (IAM) accounts to database accounts is enabled.