← Back to issue list

store: better retry strategy for GETs.

View original Github issue

Metadata

Project
snapcraft
Number
#1212
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged after approval. Replaced existing retry logic with a configurable retry class for GET requests, enabling fine-grained control and forced retries on specific HTTP status codes. Approved with a minor naming suggestion.

Suggested action:

No scores available.

Issue body

Use the retry class which allows for more fine grained settings such as forcing a retry when a status code is returned. LP: #1671817

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after approval. Replaced existing retry logic with a configurable retry class for GET requests, enabling fine-grained control and forced retries on specific HTTP status codes. Approved with a minor naming suggestion.
qwen3.6-35b-a3b-mtp-q6 Merged an update implementing a refined retry strategy for GET requests. The change uses a dedicated retry class for fine-grained control, including forced retries on specific HTTP status codes. Approved after minor naming feedback.
qwen3.6-35b-a3b-mtp-q6 Merged an improved GET request retry strategy utilizing a dedicated retry class for finer control over status code handling. Approved after minor naming feedback. Resolves LP: #1671817.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1184 store: enable retries for store calls. snapcraft merged Merged changes enabling automatic retries for flaky store and CDN connection resets. The update registers a retry adapter with a five-attempt limit, approved by one reviewer, and modifies two files.
72%