← Back to issue list

fix(i/lxd): make lxd download inspector more strict

View original Github issue

Metadata

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

Current evaluation

Merged a fix making the LXD download inspector format verification stricter to prevent snap assertion download rejections. Approved by two reviewers, passed CI checks, and modified six files with 79 additions and 12 deletions.

Suggested action: —

No scores available.

Issue body

Snap assertion downloads may trigger rejections from the lxd download inspector. Make format verification more strict to avoid this. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a fix making the LXD download inspector format verification stricter to prevent snap assertion download rejections. Approved by two reviewers, passed CI checks, and modified six files with 79 additions and 12 deletions.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#462 fix(inspectors/lxd): detect releases image request fetch-service merged Merged fix for the LXD rootfs inspector to correctly detect release images alongside daily snapshots. Approved by two reviewers with all CI checks passing. Resolves FETCH-28 across three modified files.
75%
#427 feat(i/lxd): simple streams download inspector for lxd fetch-service merged Merged implementation adding Simple Streams Download inspection to verify rootfs image integrity by validating digests against product listings. Approved by two reviewers, passed all CI checks, and modified 12 files with 2133 additions.
75%
#1984 fix(snap): check if LXD is actually installed charmcraft merged Merged fix updating configure and remove hooks to verify LXD installation before proceeding. Prevents unwanted LXD snap installations on Ubuntu that cause delays. Approved by two reviewers, passed CI, and resolves #1982.
75%
#504 feat: add inspector for lxd instance types fetch-service merged Merged. Introduces an inspector for LXD instance types metadata from images.lxd.canonical.com, resolving issue #509. Approved by two reviewers with all CI checks passing. Changes span seven files with 770 additions.
72%
#1199 cleanbuild: packaging independent detection. snapcraft merged Merged changes making LXD detection packaging-independent for snap and manual installs. Detection logic moved from lifecycle to the lxd module, eliminating the translation layer. Approved, passed CI, and merged across four files.
72%
#566 fix(inspectors/lxd): allow dot in image date fetch-service merged Merged fix to the LXD inspector allowing dots in image dates within download URLs. Approved by two reviewers, passed all CI checks, and updated four files to correct date parsing logic.
71%
#3112 spread: remove dead code for lxd setup and add debug prints snapcraft merged Merged removal of redundant elif dead code in LXD setup logic and addition of debug prints for LXD snap install failures. Approved by one reviewer, passed CI checks, and integrated.
71%
#557 fix(inspectors/lxd): add 26.04 to the list of supported versions fetch-service merged Merged a quick fix adding Ubuntu 26.04 to the LXD inspector supported versions list. Approved by three reviewers, passed all CI checks, and changed one file. The author noted the hardcoded list should eventually be moved to a configuration file.
70%
#2222 lxd: support new style snap injection snapcraft merged Merged after implementing new style snap injection for LXD. Refactored provider logic, normalized code, and removed test duplication. Approved by two reviewers, passed CI, and verified via edge channel testing.
70%