build providers: prevent snap refreshing in build environment
Metadata
Current evaluation
Merged to prevent snap refreshing in the build environment by holding refresh for one day. Adjusted per reviewer feedback to avoid incomplete manifests and address versioning string comparisons. Approved and merged.
Suggested action: —
No scores available.
Issue body
hold refresh for 1 day for installed snaps
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to prevent snap refreshing in the build environment by holding refresh for one day. Adjusted per reviewer feedback to avoid incomplete manifests and address versioning string comparisons. Approved and merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Prevents snap refreshing in the build environment by holding refresh for one day. Updated based on feedback to avoid incomplete manifests. Noted potential versioning string comparison issues with rc tags. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Prevents snap refreshing in the build environment by holding refresh for one day. Updated to avoid manifest issues, with a noted caveat regarding rc versioning breaking string comparisons. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2507 build providers: improve handling in snap logic | snapcraft | merged | Merged. Improved snap provider logic by cleaning up channel warnings, removing unnecessary sudo calls, and silencing noisy refresh hold commands. Approved, passed CI, and modified 5 files (+131/-73 lines). | |
| #3184 build providers: check revision before switching | snapcraft | merged | Merged a fix adding a revision check before channel switching in build providers. Resolves incorrect switching behavior caused by snapd changes. Approved by reviewer, passed CI, and merged. | |
| #3146 build providers: snap sw to channels if injecting | snapcraft | merged | Merged fix that prevents automatic channel switching to latest/stable when injecting snaps with revision assertions during build provider setup. Approved by reviewer, passed CI, and integrated. | |
| #3130 build providers: don't show an error if there are no auto-refresh changes | snapcraft | merged | Merged a fix suppressing a false snap watch error when no auto-refresh changes exist. Adding a question mark to the change type ensures a success exit code. Approved by reviewers and passed CI. | |
| #1692 fix: refresh build snaps if already installed | craft-parts | open | Refreshes build snaps when already installed to reflect channel changes, fixing #496. Has 2 approvals, 1 unresolved review comment, and mostly passing CI. | |
| #3617 snap: do not refresh already installed snaps | snapcraft | merged | Merged changes to prevent refreshing already installed snaps by removing two lines of code. Approved by one reviewer and integrated into the main branch. |