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

feat: add iam-role argument #316

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

Conversation

a-hat
Copy link

@a-hat a-hat commented Feb 28, 2024

Pull Request

Related Github Issues

  • [none]

Description

Adds a new argument --iam-role which passes the given role ARN to terragrunt (see https://terragrunt.gruntwork.io/docs/reference/cli-options/#terragrunt-iam-role).

Security Implications

  • [none]

System Availability

  • [none]

@Almenon
Copy link
Collaborator

Almenon commented Feb 29, 2024

terragrunt-atlantis-config just generates a config, so it doesn't need a IAM role, but I could be mistaken?

@a-hat
Copy link
Author

a-hat commented Feb 29, 2024

terragrunt-atlantis-config just generates a config, so it doesn't need a IAM role, but I could be mistaken?

@Almenon Thanks for your feedback! Unfortunately it does. As far as I understand, to generate the config terragrunt is called, which will need to retrieve the terraform state from the remote location. In our case this is a AWS backend, and we need to assume the role to access it.

@Almenon
Copy link
Collaborator

Almenon commented Mar 3, 2024

I also have a AWS backend. I'm able to generate the config without AWS credentials.

@a-hat
Copy link
Author

a-hat commented Mar 4, 2024

I also have a AWS backend. I'm able to generate the config without AWS credentials.

If I try to generate the config without the assume role parameter, I get 4xx errors accessing the AWS API. I don't know exactly what terragrunt does, maybe it tries to retrieve outputs of a module from the state.

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.

None yet

2 participants