build providers: prevent snap refreshing in build environment
Metadata
Current evaluation
Merged PR #3102 as commit 2ff9fcd10, adding a one-day refresh hold for installed snaps to prevent automatic refreshes in build environments.
Resolution: closed resolved
Reason: The approved change was merged into Snapcraft via commit 2ff9fcd10. It replaced the proposed manifest-injection approach with a one-day refresh hold for installed snaps.
Evaluated by: mr-cal-bot
No scores available.
Issue body
hold refresh for 1 day for installed snaps
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #3102 as commit 2ff9fcd10, adding a one-day refresh hold for installed snaps to prevent automatic refreshes in build environments. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #3102 in commit 2ff9fcd10: build providers now hold refresh for one day on installed snaps, preventing snap refreshes in build environments. | |
| 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. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #182 bases: disable automatic snap refreshes | craft-providers | merged | Merged PR #182 (commit 562257a, CRAFT-1518) disables automatic snap refreshes in build providers by running `snap set system refresh.hold=24h` during setup() and warmup(), fixing #174 and LP#1998100. | |
| #2507 build providers: improve handling in snap logic | snapcraft | merged | Merged snapcraft#2507 as commit 841b41fa3, improving build-provider snap logic, removing unnecessary sudo calls, and silencing refresh-hold noise. | |
| #3617 snap: do not refresh already installed snaps | snapcraft | merged | Merged PR #3617 via commit 22040ffc5, stopping Snapcraft from refreshing already installed snaps; implementation commit 9d4129398. | |
| #161 snap: do not refresh already installed snaps | craft-parts | merged | Merged craft-parts#161 via 704ae4c6, porting snapcraft#3617 to prevent already-installed snaps from being refreshed. | |
| #198 buildd: add try/except block for snap refresh commands | craft-providers | merged | Merged PR #198 added exception handling around buildd snap refresh commands, preventing refresh failures from interrupting build setup. |