-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Flaky System Tests #1796
Comments
Ouch. I haven't seen a 403 like that before, isn't 429 supposed to be for rate limiting? Exponential backoff would hopefully resolve that. The majority of the errors we've come across are due to eventual consistency though. |
@jgeewax I'm not sure how to move on this? |
Nice! Let me know if I can help in anyway. I see in the docs for bigquery as well as for storage that the 403 response code is used for a lot of situations. |
@jgeewax, do we need to up our quota limits? It looks like our normal number of builds is getting close to the limits?
|
@daspecster Those BigQuery rate limiting issues are fixed via the backoff/retry stuff added in #2050. |
@dhermes and I talked briefly about this issue. It seems like the tests are failing pretty consistently now.
What can we do to make this library or the tests more resilient?
Some of the issues seem to be account related?.
Maybe @jgeewax knows?
I think this is what @dhermes and I talked about last week. Is this where you were talking about adding/changing the retrying/exponential backoff?
The text was updated successfully, but these errors were encountered: