feat(Provider): inject the base snap
Metadata
Current evaluation
Merged as commit 41cee32d, injecting the application's base snap into the provider so managed instances need not download it; fixes #952 (CRAFT-4889).
Resolution: closed resolved
Reason: PR #997 passed CI, received approval, and merged as commit 41cee32d. Its implementation injects the base snap alongside the app and explicitly fixes #952.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This injects the app's base snap into the provider along with the app itself, meaning the provider doesn't necessarily have to download the base snap.
Fixes #952 CRAFT-4889
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit 41cee32d, injecting the application's base snap into the provider so managed instances need not download it; fixes #952 (CRAFT-4889). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/craft-application#997, injecting the application's base snap into the provider alongside the app and fixing #952 (CRAFT-4889). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Injects the base snap into the provider alongside the application, eliminating separate downloads. Resolves #952 and CRAFT-4889. Approved by two reviewers with all CI checks passing. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Injects the base snap into the provider alongside the application, removing the need for separate downloads. Resolves #952 and CRAFT-4889. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-application#952
(confidence 100%)
Base-snap injection into managed instances was implemented by merged PR #997.
-
Related To:
canonical/craft-application#953
(confidence 95%)
Earlier implementation attempt superseded by the merged PR #997.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #444 feat(snaps): inject base snap from host | craft-providers | merged | Merged PR #444 in commit 03704c1, injecting each host snap's base snap and reusing an existing base; released in craft-providers 1.20.0. | |
| #953 feat: inject application's base snap into managed instances | craft-application | closed | Closed unmerged as superseded by PR #997, which merged base-snap injection as commit 41cee32d and fixed #952. | |
| #2174 build_providers: inject snaps when running from a snap | snapcraft | merged | Merged PR #2174 (commit 283ab0bb8) to inject host Snapcraft and base snaps when running from the snapcraft snap, while non-snap runs install them from the store. | |
| #4084 providers: inject current snapcraft snap into instances | snapcraft | merged | Merged PR #4084 as commit b852f9ac8, injecting the current Snapcraft snap into provider instances so parallel-installed instances can build core22 snaps. | |
| #952 Inject the application's base snap into the managed instance | craft-application | closed | Base-snap injection for managed instances was merged in PR #997 (commit 41cee32d), preventing instances from downloading the application's base snap. | |
| #1779137 build provider support to inject snaps on the host | snapcraft (launchpad) | closed | Build providers gained host-snap injection when Snapcraft runs from a snap; fixed by merged snapcore/snapcraft#2174 (commit 283ab0bb8). | |
| #622 Inject instanced host snaps to the provider | craft-providers | closed | Resolved downstream: craft-application added instanced-snap support in PR #467 (77a3582b), removing the need for a craft-providers API change. | |
| #661 feat(provider): add snap registration api | craft-application | merged | Merged #661 via commit 08853d3b, adding a provider API to register extra snaps for installation before lifecycle execution (CRAFT-4174). | |
| #143 build(deps): bump craft-providers to 1.20.0 | craft-application | merged | Merged PR #143 as commit f0349036, updating craft-providers to 1.20.0 so host-injected snaps also receive their base snap. | |
| #3583 build providers: snapcraft's new base is core20 (CRAFT-544) | snapcraft | merged | Merged PR #3583 via 2b2d7dbcf, replacing build providers' injected core18 base with core20 to match Snapcraft's new base. |