setId before throwing exception for dynamoImportFailure for tainting

hsiam261 · hsiam261 · commit 41d42970ddb7 · 2023-12-13T03:08:30.000+06:00
Otherwise the table would get created but state won't manage it which
would then require manual intervention to fix.
diff --git a/internal/service/dynamodb/table.go b/internal/service/dynamodb/table.go
@@ -624,6 +624,7 @@ func resourceTableCreate(ctx context.Context, d *schema.ResourceData, meta inter
 
 		importArn := importTableOutput.(*dynamodb.ImportTableOutput).ImportTableDescription.ImportArn
 		if _, err = waitImportComplete(ctx, conn, *importArn, d.Timeout(schema.TimeoutCreate)); err != nil {
+			d.SetId(tableName)
 			return create.AppendDiagError(diags, names.DynamoDB, create.ErrActionCreating, ResNameTable, d.Id(), err)
 		}
 	} else {

Original file line number	Diff line number	Diff line change
`@@ -624,6 +624,7 @@ func resourceTableCreate(ctx context.Context, d *schema.ResourceData, meta inter`
`624`	`624`
`625`	`625`	`importArn := importTableOutput.(*dynamodb.ImportTableOutput).ImportTableDescription.ImportArn`
`626`	`626`	`if _, err = waitImportComplete(ctx, conn, *importArn, d.Timeout(schema.TimeoutCreate)); err != nil {`
	`627`	`+ d.SetId(tableName)`
`627`	`628`	`return create.AppendDiagError(diags, names.DynamoDB, create.ErrActionCreating, ResNameTable, d.Id(), err)`
`628`	`629`	`}`
`629`	`630`	`} else {`