-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
aws_account_alternate_contact not working if contact was not previously set #22186
Comments
Seeing the same errors on |
@jansepke Thanks for reporting this! @davidferguson-cr Thank your for the additional details. I have added eventual consistency in #23516 to hopefully fix this issue. It is difficult for us to test since we do not have new test accounts without a previously set alternate contact. However, my best guess is that there is a longer delay with a new account to create the alternate contact. That would explain the error you are seeing since there was no wait between creation and reading, which is where you get the error. After you get a chance to test #23516 (should be part of v4.4), please let me know if this is resolved. |
@YakDriver I've tested with Previously, with |
This is also consistently inconsistent for me. Earlier I did 69 contact adds and 9 failed, just now I did 9 and 5 of them failed. I double-checked one of the accounts in question this last time and the contacts that failed just aren't showing up at all. Looking at the code in the provider it seems correct - it's checking for errors returned from the SDK. Looking at the SDK it's using the same request.Send() method that every other service uses. This seems like an issue with the API itself silently discarding requests? Edit: Opened a support case w/ AWS. |
According to AWS they've deployed a fix for this on their side. Haven't had a chance to test it myself yet. |
Tested and all works fine. Thanks all! |
Marking this issue as stale due to inactivity. This helps our maintainers find and focus on the active issues. If this issue receives no comments in the next 30 days it will automatically be closed. Maintainers can also remove the stale label. If this issue was automatically closed and you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thank you! |
There have been signficant changes since this was first raised leaving this in an unknown state:
To shift focus to things we know are problems currently, I'm going to close this issue. Please don't hestitate to raise a new issue if this isn't resolved. |
Warning This issue has been closed, meaning that any additional comments are hard for our team to see. Please assume that the maintainers will not see them. Ongoing conversations amongst community members are welcome, however, the issue will be locked after 30 days. Moving conversations to another venue, such as the AWS Provider forum, is recommended. If you have additional concerns, please open a new issue, referencing this one where needed. |
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. |
Community Note
Terraform CLI and Terraform AWS Provider Version
Affected Resource(s)
Terraform Configuration Files
Please include all Terraform configurations required to reproduce the bug. Bug reports without a functional reproduction may be closed without investigation.
Debug Output
Panic Output
Expected Behavior
When the alternate contact of that type does not exist yet in AWS I would expect terraform to set the contact details for that type without errors.
Actual Behavior
Error: error reading Account Alternate Contact (/BILLING): couldn't find resource
Steps to Reproduce
terraform apply
Important Factoids
The resource
aws_account_alternate_contact
was introduced in #21789. When a contact is empty and needs to be created by terraform the AWS API will return aResourceNotFoundException
. This is not handled in this resource. A possible solution could be to check inresourceAlternateContactRead
if the resource is new and if we got a not found error.References
The text was updated successfully, but these errors were encountered: