Skip to content
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

docs: Update AWS backend doc's #5133

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tomellis91
Copy link

Updated the assume_role config on the backed to match the updated syntax

what

The latest changes / updates to terraform have slightly changes the syntax. The PR updates the doc's to align with the Terraform Docs - backend/s3/role_arn

Updated the assume_role config on the backed to match the updated syntax

Signed-off-by: Tom Ellis <tom.e@mryum.com>
@tomellis91 tomellis91 requested review from a team as code owners December 2, 2024 01:54
@tomellis91 tomellis91 requested review from jamengual, lukemassa and nitrocode and removed request for a team December 2, 2024 01:54
@github-actions github-actions bot added the docs Documentation label Dec 2, 2024
@tomellis91 tomellis91 changed the title Update provider-credentials.md docs: Update AWS backend doc's Dec 2, 2024
@X-Guardian
Copy link
Contributor

Thanks for this @tomellis91, but the new assume_role block is only valid for Terraform version 1.6.0 and above. Can you update the docs to include example of both the legacy properties and the new assume_role block?

@X-Guardian X-Guardian added the waiting-on-response Waiting for a response from the user label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation waiting-on-response Waiting for a response from the user
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants