Open sbutler opened 2 years ago
Please merge this, need it urgently. Thank you!
any news on this PR?
How about now? I believe our team is now getting bitten by this and I'd rather not have to revert our changes to manage Service Catalog Portfolio shares..
I'm not sure if this is has the same root cause, but I'm getting the following error when doing multiple concurrent deassociations (for example when changing the elements of the for_each):
InvalidStateException: Cannot process more than one portfolio share action at the same time for account ####. Try again later.
Please consider this as well in a possible fix.
Community Note
Terraform CLI and Terraform AWS Provider Version
Affected Resource(s)
Terraform Configuration Files
Debug Output
https://gist.github.com/sbutler/a0c33af877aed4e5c9ac05d6658edeea
Expected Behavior
The portfolio should be shared to all the OU's.
Actual Behavior
One or more might be created, but some will get a ThrottlingException and not be shared.
Steps to Reproduce
terraform apply
Important Factoids
Although I could not find documentation, experimentation revealed:
CreatePortfolioShare
API call can succeed, but the laterDescribePortfolioShareStatus
call can return an error in theShareDetail.ShareErrors[*].Error
field. After adding some enhanced error logging I could see this wasThrottlingException
in my example.