-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix Terraform credential error #226
Conversation
Terraform Lint/Plan ResultsTerraform Format and Style 🖌
|
Terraform Lint/Plan ResultsTerraform Format and Style 🖌
|
I don't really know why this broke in the first place, the only breaking change I can see in the Terraform changelogs is that they changed the order in which AWS creds are loaded, but we don't have any other AWS creds? Downgrading works at least. In the future, we'll probably just use some free-tier opentofu provider for this so we don't have to worry about Terraform breaking randomly. |
Turns out the issue was that external PRs like #225 don't have access to our Github Actions secrets, which make the tests fail. This can be ignored for smaller PRs, but this means that bigger PRs that need thorough testing need to be submitted from the local repo, and cannot come from a fork. |
Was getting the following error from Github Actions on Terraform 1.9.3:
Reverting to last known working minor version.