-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Taskcluster checkout step can fail due to "The remote end hung up unexpectedly" #14617
Comments
I saw this again today in #15037: @jgraham should I report this to the Taskcluster bug tracker, or should we just add retries? I suspect that both Travis CI and Azure Pipelines use a GitHub token for the clone step, although it's not clear if this is a quota issue or just the network dropping packets. |
This is in our code, not in the TaskCluster code, and it's a git clone, not an API access, so tokens don't seem relevant. I think the only thing we can do is add retries. |
It happened in https://tools.taskcluster.net/groups/ND6VrV-QRHeu9jeBJ7RUHw/tasks/EMRPZhU7QnS9pcoukCdzuQ/runs/0/logs/public%2Flogs%2Flive.log too, also for #15037. @jgraham is there an existing retry script we can use? I imagine there must be one in Mozilla CI if that also depends on mozdownload? |
That doesn't rely on mozdownload, and the errors we are seeing here aren't related to mozdownload. That said I'm sure that someone will have a library for retrying a command with exponential backoff e.g. https://pypi.org/project/backoff/ for Python (of course this is not Python, but shell). |
This is in our code, no? It's |
Yes, it's in our code. Other CI systems do the checkout before control reaches "user code" and I suspect they do something to be more resilient, or we'd see a lot of these errors on Travis and Azure too. |
This didn't keep happening, closing. |
In https://tools.taskcluster.net/groups/XDEY8H3YSBSfN2-rUbft8g (for #13886) the wpt-firefox-nightly-results-without-changes task failed like this:
This is the first time I've seen this, but we may need retry logic around the checkout.
@Hexcles @lukebjerring FYI.
The text was updated successfully, but these errors were encountered: