← Back to issue list

fix: prevent races in cached inspection results

View original Github issue

Metadata

Project
fetch-service
Number
#409
Type
pull request
State
merged
Author
cmatsuoka
Labels
Created
Updated
Closed

Current evaluation

Merged fix preventing race conditions in cached inspection results. Artifacts are now added to the session only after asynchronous inspection completes, ensuring valid reuse during fast download retries. Approved by two reviewers, passed CI, and merged.

Suggested action:

No scores available.

Issue body

Add artifacts to the current session only after the asynchronous inspection is completed, making sure inspection results are valid before reusing them in case of fast download retries of the same artifact. Fixes #399 - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged fix preventing race conditions in cached inspection results. Artifacts are now added to the session only after asynchronous inspection completes, ensuring valid reuse during fast download retries. Approved by two reviewers, passed CI, and merged.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#489 fix: check session exists before completing inspection fetch-service merged Merged fix for a race condition where inspection completion overlaps with session timeout. Adds a session check before finalizing inspection, ignoring invalid results. Resolves #486. Noble CI failure fixed in PR #490.
70%
#536 fix: increase inspection timeout to 5 minutes fetch-service merged Merged after two approvals. Increases the default inspection timeout from 60 to 300 seconds to handle large artifacts, with plans to make it configurable. Passed CI checks and integrated into the codebase.
70%