snap installer: do not remove previous snap file
Metadata
Current evaluation
Merged PR #151 in commit 297c452, removing unnecessary deletion of existing snap files before uploading replacements.
Resolution: closed resolved
Reason: PR #151 was approved and merged as commit 297c452. The snap installer now lets the push operation overwrite existing snap files, simplifying the provider and avoiding unnecessary deletion work.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
#### Overview
Do not remove the previous snap file (i.e. `/tmp/<snap-name>.snap`) before pushing a new snap file into the instance.
Pushing a file into an instance overwrites the previous file, so there is no need to delete the previous file.
#### Reasons to remove
1. This isn't done in `snapcraft_legacy/internal/build_providers`
2. It doesn't need to be done
3. It adds time and complexity
(CRAFT-1370)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #151 in commit 297c452, removing unnecessary deletion of existing snap files before uploading replacements. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR removing unnecessary deletion of previous snap file before push, simplifying the snap installer logic. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. |