Merge pull request #39190 from aleksandarknezevic/f-ecr-policy-iam-role

fixed iam role in ecr repository
hashicorp · Sep 9, 2024 · ccf99b0 · ccf99b0
2 parents 2c552c3 + 607339e
commit ccf99b0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/39190.txt b/.changelog/39190.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+resource/aws_ecr_repository_policy: Fix retry logic handling eventual consistency of newly created IAM roles
+```
diff --git a/internal/service/ecr/repository_policy.go b/internal/service/ecr/repository_policy.go
@@ -77,7 +77,7 @@ func resourceRepositoryPolicyPut(ctx context.Context, d *schema.ResourceData, me
 
 	_, err = tfresource.RetryWhenIsAErrorMessageContains[*types.InvalidParameterException](ctx, propagationTimeout, func() (interface{}, error) {
 		return conn.SetRepositoryPolicy(ctx, input)
-	}, "Invalid repository policy provided")
+	}, "Principal not found")
 
 	if err != nil {
 		return sdkdiag.AppendErrorf(diags, "putting ECR Repository Policy (%s): %s", repositoryName, err)