TheKevJames / coveralls-python / 46972d53-78bd-4dfd-9b73-f537593a6d65

96%

master: 96%

Build Type

Pull #760

circleci

Committed by TheKevJames

Commit Message

feat(api): retry transient HTTP failures

Route API POSTs through a requests Session with a urllib3 Retry adapter so
transient failures can be retried with exponential backoff and jitter. Only
connection errors, read timeouts, 429s and 5xx responses are retried; a 422 is
a configuration error and is never retried. Retries are off by default and are
configurable via the --retries flag, the COVERALLS_RETRIES env var, or a
retries key in .coveralls.yml, on every command that talks to the API.

The API HTTP tests move to the responses library for higher fidelity, which
replaces the previous fake "resubmission" tests with real retry coverage.

Closes #572

changelog:
API requests can now retry transient HTTP failures (connection errors, read
timeouts, 429 and 5xx responses) with exponential backoff and jitter. Retries
are off by default; enable with ``--retries``, ``COVERALLS_RETRIES``, or a
``retries`` key in ``.coveralls.yml``.

Pull Request Pull Request #760: feat(api): retry transient HTTP failures

Coverage Stats

158 of 164 branches covered (96.34%)

Branch coverage included in aggregate %.

36 of 39 new or added lines in 3 files covered. (92.31%)

7 existing lines in 1 file now uncovered.

576 of 598 relevant lines covered (96.32%)

5.78 hits per line