-
Improvement
-
Resolution: Fixed
-
Minor
-
None
We should make sure the plugin does retries on ingestion when timeouts/errors happen.
We should add retries to the OkHttp client config we use in the plugin. It looks like OkHttp already does some retries out-of-the-box: https://medium.com/inloopx/okhttp-is-quietly-retrying-requests-is-your-api-ready-19489ef35ace
In these 3 scenarios, retries are handled automatically by Okhttp: https://square.github.io/okhttp/4.x/okhttp/okhttp3/-ok-http-client/-builder/retry-on-connection-failure/
Based on the server response, we can also add additional logic to perform retries.
Implementation:
We can do 3 retries with a delay of 2 seconds between each try when we receive 5XX from the ingestion API.
[JENKINS-58544] Implement HTTP client retries
Rafal Myslek
created issue -
| Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
Rafal Myslek
made changes -
| Status | Original: In Progress [ 3 ] | New: In Review [ 10005 ] |
Rafal Myslek
made changes -
| Resolution | New: Fixed [ 1 ] | |
| Status | Original: In Review [ 10005 ] | New: Resolved [ 5 ] |
Rafal Myslek
made changes -
| Status | Original: Resolved [ 5 ] | New: Closed [ 6 ] |