snap_installer: trim suffix from snap names
Metadata
Current evaluation
Merged PR #251 as commit adc7860, trimming snap-name suffixes from store installs, temporary filenames, and instance config, with integration tests added.
Resolution: closed resolved
Reason: PR canonical/craft-providers#251 was merged as commit adc7860. It fixes suffixed snap handling and adds integration coverage for instance configuration and store installation.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
When installing a snap in an instance, do not use the snap name suffix (if present) when installing from the store, in temporary filenames, or in the instance config file.
This is a bug fix following up on CRAFT-1660, where the instance config files were not trimming the snap name suffix:
Previous `/etc/craft-instance.conf`:
```
compatibility_tag: snapcraft-buildd-base-v0.0
snaps:
snapcraft:
revision: '8931'
source: host
snapcraft_foo:
revision: '8928'
source: host
snapcraft_bar:
revision: '8931'
source: host
```
New `/etc/craft-instance.conf`:
```
snaps:
snapcraft:
revision: '8931'
source: host
```
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #251 as commit adc7860, trimming snap-name suffixes from store installs, temporary filenames, and instance config, with integration tests added. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #251 (commit adc7860), trimming snap suffixes from store installs, temporary filenames, and instance configuration. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Trimmed snap name suffixes during store installation, temp filenames, and instance config generation, resolving CRAFT-1660. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #152 snap installer: refactor | craft-providers | merged | Merged PR #152 in commit 0e51d9a, refactoring snap_installer without functional changes and adding integration coverage for dangerously installed snaps. | |
| #121 snap installer: refactor installation logic | craft-providers | merged | Merged PR #121 (commit 8b7dede, merge 58abe9b), refactoring snap installation to install or refresh from the store, support host/store switching, and expand tests. | |
| #564 fix(test): remove env var set by snapped testing programs | craft-application | merged | Merged PR #564 (commit f854aac9) clears SNAP_INSTANCE_NAME before provider tests, preventing snapped runners such as VS Code from changing the expected instance name. | |
| #513 Add snaps/name inspector | fetch-service | closed | Added the Snapcraft snap names inspector in merged PR #517 (commit 85f220e), resolving the LP test-build rejection. |