-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Transient error: google_project.project: Error retrieving App Engine application ... 503 #2814
Comments
We now generate the app engines resource code in Magic Modules, the file would be here: https://github.com/GoogleCloudPlatform/magic-modules/blob/master/third_party/terraform/resources/resource_app_engine_application.go |
@emilymye Thanks for the info! Will |
Yup, that's the plan. @chrisst may have dealt with those retries for 2.x going forward but tagging to make sure. |
Correct, as of https://github.com/terraform-providers/terraform-provider-google/pull/2568/files we should retry on 503's for all CRUD requests made by generated resources. |
Closing due to @chrisst's comment, feel free to reopen if the issue is still occurring. |
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. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks! |
Community Note
Terraform Version
terraform v0.11.7
Affected Resource(s)
Terraform Configuration Files
omitted, issue only occurs in our CI job
Debug Output
omitted, issue only occurs in our CI job
Panic Output
omitted, issue only occurs in our CI job
Expected Behavior
Terraform should not fail due to a transient 503 error.
Actual Behavior
Terraform occasionally fails with:
Steps to Reproduce
omitted, issue only occurs in our CI job
Important Factoids
omitted, issue only occurs in our CI job
References
Should Apps.Get (call site) be wrapped with
retryTime
? I've made a similar PR before (https://github.com/terraform-providers/terraform-provider-google/pull/1713/files) but since the Apps.Get call has been removed in master/2.0.0 (#2386), I can't see if/how to make this change.The text was updated successfully, but these errors were encountered: