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.

Sign up for GitHub

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

Jump to bottom

resource/aws_cloud9_environment_ec2: Retry creation for IAM eventual consistency #3651

Merged
merged 1 commit into from
Mar 7, 2018
Merged

resource/aws_cloud9_environment_ec2: Retry creation for IAM eventual consistency #3651

merged 1 commit into from
Mar 7, 2018

Conversation

bflad
Copy link
Contributor

@bflad bflad commented Mar 7, 2018

To address the following test failure:

=== RUN   TestAccAWSCloud9EnvironmentEc2_allFields
--- FAIL: TestAccAWSCloud9EnvironmentEc2_allFields (308.07s)
	testing.go:513: Step 0 error: Error applying: 1 error(s) occurred:
		
		* aws_cloud9_environment_ec2.test: 1 error(s) occurred:
		
		* aws_cloud9_environment_ec2.test: NotFoundException: User arn:aws:iam::*******:user/tf_acc_cloud9_env_7kfkcqqz does not exist.
			status code: 400, request id: 3a531ada-2138-11e8-9203-1925f346b668

=== RUN   TestAccAWSCloud9EnvironmentEc2_importBasic
--- PASS: TestAccAWSCloud9EnvironmentEc2_importBasic (147.96s)
=== RUN   TestAccAWSCloud9EnvironmentEc2_basic
--- PASS: TestAccAWSCloud9EnvironmentEc2_basic (163.00s)
=== RUN   TestAccAWSCloud9EnvironmentEc2_allFields
--- PASS: TestAccAWSCloud9EnvironmentEc2_allFields (227.17s)

@bflad bflad added enhancement Requests to existing resources that expand the functionality or scope. service/cloud9 Issues and PRs that pertain to the cloud9 service. labels Mar 7, 2018
@ghost ghost added the size/S Managed by automation to categorize the size of a PR. label Mar 7, 2018
@bflad bflad requested a review from appilon March 7, 2018 14:45
appilon
appilon approved these changes Mar 7, 2018
View reviewed changes
Copy link
Contributor

@appilon appilon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@bflad bflad merged commit 42021c5 into master Mar 7, 2018
@bflad bflad deleted the t-aws_cloud9_environment_ec2-user-notfoundexception branch March 7, 2018 15:09
bflad added a commit that referenced this pull request Mar 7, 2018
@bflad
Update CHANGELOG for #3651
0eaf01e
@ghost
Copy link

ghost commented Apr 7, 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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@appilon appilon appilon approved these changes

Assignees
No one assigned
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/cloud9 Issues and PRs that pertain to the cloud9 service. size/S Managed by automation to categorize the size of a PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bflad @appilon