-
Notifications
You must be signed in to change notification settings - Fork 139
Retry uploads on (connection) errors #158
Comments
Another example:
|
Any update on this? Another example (from AppVeyor):
|
I've also seen quite a few similar connection errors from our CircleCI builds. My current solution for the issue has been running
I know it isn't pretty but I haven't seen it failing yet. |
I got the same issue
|
Hi all.
|
Dear maintainer(s): Would a PR to use a 3rd party "retryable" decorator, https://tenacity.readthedocs.io/en/latest/ be accepted? |
I implemented a retry for requests here and retry number is configurable via args: https://github.com/nmoinvaz/codecov-python/commit/9632dc655ed378ece67bf54861a300dd1e732aef If this is something that would be accepted I can submit a PR. |
Looks like this is a long-running issue. What about the solution from @nmoinvaz? Could it be added or not? What is needed to finally get a solution into codecov-python? |
Retries have been added in 07583f3 by @thomasrockhu, thanks! However I've still seen this now:
From this it is not clear if there have been any retries at all (should get logged probably), and it might have worked better with a delay between retries (#293, #295). |
Looking at codecov bash uploader it appears to now try 5 times, sleeping for 2s in between. |
I believe this issue is resolved now. It does delay between retries and also displays each time it retries. |
In version 1.18.0 and earlier, >> term.split_seqs(term.move_right(333) + 'xyz', maxsplit=1) ['\x1b[333C', '333', 'xyz'] This is a bug, it duplicates the matched parameter, this is now corrected: ['\x1b[3C', 'xyz'] Previously, we documented "same arguments as "re.split", so we must also implement maxsplit and flags. Also, - fix flake8 linting of tests by moving fixtures to conftest.py, fixes "unused" or "re-definition from import" errors. - version stamp blessed/__init__.py like a codegen step i guess - remove run_codecov.py, its been fixed upstream codecov/codecov-python#158 (comment)
* Bugfix for term.split_seqs('term.right(3)') In version 1.18.0 and earlier, >> term.split_seqs(term.move_right(333) + 'xyz', maxsplit=1) ['\x1b[333C', '333', 'xyz'] This is a bug, it duplicates the matched parameter, this is now corrected: ['\x1b[3C', 'xyz'] Previously, we documented "same arguments as "re.split", so we must also implement maxsplit and flags. Also, - fix flake8 linting of tests by moving fixtures to conftest.py, fixes "unused" or "re-definition from import" errors. - version stamp blessed/__init__.py like a codegen step i guess - remove run_codecov.py, its been fixed upstream codecov/codecov-python#158 (comment)
Example: https://ci.appveyor.com/project/pytestbot/pytest/build/1.0.5141/job/ycpyxb9yvrckho21#L402 (RemoteDisconnected)
codecov-bash tries 3 times already, sleeping for 30s inbetween.
The text was updated successfully, but these errors were encountered: