Skip to content

Conversation

@emilford
Copy link
Contributor

@emilford emilford commented Aug 6, 2024

Switch this and dependent modules to the v5.x of the Terraform AWS
provider to support the latest AWS features, like RDS' io2.

  • Fix dynamic statement block
│   on .terraform/modules/production.auth_token/secret/main.tf line 43, in data "aws_iam_policy_document" "secret":
│   43:         identifiers = [statement.arn]
│
│ This object does not have an attribute named "arn".

https://developer.hashicorp.com/terraform/language/expressions/dynamic-blocks

  • Fix user managed secret environment variables

NOTE: This cherry picks a commit from a PR @jferris opened to address the same issue.

  • Terraform was complaining about calling nonsensitive when the value
    wasn't sensitive; this forces the initial value to potentially be
    sensitive to avoid the error.
  • We can skip parsing out the environment variables for the user managed
    secret module, because they are passed in directly.

emilford and others added 2 commits August 6, 2024 10:37
Switch this and dependent modules to the v5.x of the Terraform AWS
provider to support the latest AWS features, like RDS' io2.

- Fix dynamic statement block

```
│   on .terraform/modules/production.auth_token/secret/main.tf line 43, in data "aws_iam_policy_document" "secret":
│   43:         identifiers = [statement.arn]
│
│ This object does not have an attribute named "arn".
```

https://developer.hashicorp.com/terraform/language/expressions/dynamic-blocks

- Fix user managed secret environment variables

NOTE: This cherry picks a commit from a [PR @jferris opened] to address the same issue.

  - Terraform was complaining about calling `nonsensitive` when the value
    wasn't sensitive; this forces the initial value to potentially be
    sensitive to avoid the error.
  - We can skip parsing out the environment variables for the user managed
    secret module, because they are passed in directly.

[PR @jferris opened]: #13
@clarissalimab clarissalimab merged commit bab3192 into main Jan 7, 2025
@clarissalimab clarissalimab deleted the v0.8.0 branch January 7, 2025 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants