Skip to content

38 terraform/backend tf add required_version ~> 1.12#108

Closed
here wants to merge 9 commits into
hackforla:mainfrom
here:38-terraform-required-version-minimum-1.12
Closed

38 terraform/backend tf add required_version ~> 1.12#108
here wants to merge 9 commits into
hackforla:mainfrom
here:38-terraform-required-version-minimum-1.12

Conversation

@here

@here here commented Jun 5, 2025

Copy link
Copy Markdown
Member

Fixes #38

What changes did you make?

  • Add required_version to incubator terraform/backend.tf

Why did you make the changes (we will use this info to test)?

@here

here commented Jun 5, 2025

Copy link
Copy Markdown
Member Author

Moving to questions / review as blocked on pull request checks. Best guess this is a permissions issue with my GitHub user?

The check job for terraform plan appears to have run successfully with the same code for RSkuma on 2025-04-14 at https://github.com/hackforla/incubator/pull/106/checks .


Pull request checks are failing with permissions errors related to aws credentials,

Error: Credentials could not be loaded, please check your action inputs: Could not load credentials from any providers

See error in Checks on pull request at https://github.com/hackforla/incubator/pull/108/checks


From source of configure-aws-credentials@v4 , additional detail recommends "it is likely but not guaranteed that the user needs but lacks this permission in their workflow."

   with:
    role-to-assume: arn:aws:iam::035866691871:role/gha-incubator
    role-session-name: ghaincubatorplan
    aws-region: us-west-2
    audience: sts.amazonaws.com

Confirmed that the .github workflow files include expected permissions for OIDC workflows as described in docs at https://docs.github.com/en/actions/security-for-github-actions/security-hardening-your-deployments/about-security-hardening-with-openid-connect#adding-permissions-settings

@here here changed the title terraform/backend tf add required_version ~> 1.12 38 terraform/backend tf add required_version ~> 1.12 Jun 18, 2025
@here

here commented Jun 18, 2025

Copy link
Copy Markdown
Member Author

Closing as replaced with pull requeset from a branch rather than fork with #109

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.

Incubator - Upgrade Terraform to the latest version

1 participant