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

provider/aws: Increase timeout for creating IAM role #7733

Merged
merged 1 commit into from
Aug 18, 2016
Merged

provider/aws: Increase timeout for creating IAM role #7733

merged 1 commit into from
Aug 18, 2016

Conversation

dtolnay
Copy link
Contributor

@dtolnay dtolnay commented Jul 20, 2016

I am seeing terraform apply fail with:

Error creating IAM Role my-role: timeout while waiting for state to become 'success'. last error: %!s()

(The message is fixed in #7732.)

All subsequent terraform applys fail with:

Error creating IAM Role my-role: EntityAlreadyExists: Role with name my-role already exists.

@dtolnay
Copy link
Contributor Author

dtolnay commented Aug 18, 2016

@stack72 is there anything I can do to move this along?

@radeksimko
Copy link
Member

Hi @dtolnay
thanks for the PR.

I took some time to read through the git log and related issues to understand why we need the retry logic here in the 1st place, because I was pretty sure this is not the reason:

// IAM roles can take ~30 seconds to propagate in AWS:

Even though the statement is true on its own, it's not why we'd retry iam:CreateRole, this would be a reason for retrying creation of any resource that depends on this IAM Role (e.g. EC2 instance, ECS service, etc.).

The reason we retry iam:CreateRole here is because there may be other IAM resources, e.g. IAM Users referenced in the Role assume policy and these IAM Users take time to propagate.

IAM Role with assume policy that doesn't reference any other IAM resource should not require any retry (ain't saying we should put it away though).

It may all sound like nitpicking a 1-2 line comment, but I felt the need for explaining this as I was confused by that one. 😃
I will update the comment in a separate PR.

@radeksimko radeksimko merged commit 46c858b into hashicorp:master Aug 18, 2016
@dtolnay dtolnay deleted the roletime branch August 18, 2016 15:34
kwilczynski pushed a commit to kwilczynski/terraform that referenced this pull request Aug 18, 2016
@ghost
Copy link

ghost commented Apr 23, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants