snap: do not refresh already installed snaps
Metadata
Current evaluation
Merged PR #3617 via commit 22040ffc5, stopping Snapcraft from refreshing already installed snaps; implementation commit 9d4129398.
Resolution: closed resolved
Reason: PR #3617 was approved and merged as commit 22040ffc5; implementation commit 9d4129398 removed the two lines responsible for refreshing installed snaps.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #3617 via commit 22040ffc5, stopping Snapcraft from refreshing already installed snaps; implementation commit 9d4129398. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in PR #3617, removing two lines so snapcraft no longer refreshes already installed snaps; the same fix was ported to the legacy branch in #3618. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged changes to prevent refreshing already installed snaps by removing two lines of code. Approved by one reviewer and integrated into the main branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main branch. Implements logic to skip refreshing snaps that are already installed. Integrated by maintainer sergiusens without additional comments or revisions. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#3618
(confidence 100%)
Commit b7c08a876 ported the same no-refresh fix to the legacy branch and was merged via commit c4ebfc54f.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #3618 snap: do not refresh already installed snaps | snapcraft | merged | Merged #3618 (commit c4ebfc54f), porting #3617 to snapcraft_legacy; it removed the refresh step so already installed snaps remain unchanged. | |
| #3102 build providers: prevent snap refreshing in build environment | snapcraft | merged | Merged PR #3102 as commit 2ff9fcd10, adding a one-day refresh hold for installed snaps to prevent automatic refreshes in build environments. | |
| #3502 snaps: do not validate snaps before install/refresh (Fixes LP#1901733) | snapcraft | merged | Merged #3502 (commit 7ab76e084), removing redundant pre-install/refresh validation because store API branch checks already cover it; fixes LP#1901733. | |
| #151 snap installer: do not remove previous snap file | craft-providers | merged | Merged PR #151 in commit 297c452, removing unnecessary deletion of existing snap files before uploading replacements. | |
| #4747 snapcraft 8.2.0.post3+gitc9ab9d1d from edge doesn't support refresh-mode: ignore-running | snapcraft | closed | Snapcraft added refresh-mode: ignore-running via merged PR #4749 (a76f85a78), then backported it through PR #4800 (f6451b6cd). |