feat(Provider): inject the base snap
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| 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 issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2174 build_providers: inject snaps when running from a snap | snapcraft | merged | Merged. Updates build providers to inject snapcraft and its base from the host when running inside the snapcraft snap, ensuring parity with non-snap environments. Approved by reviewer and passed CI checks. | |
| #953 feat: inject application's base snap into managed instances | craft-application | closed | Injected the application's base snap into managed instances to prevent redundant downloads. After implementing the feature, addressing review feedback, and adding tests, the pull request was closed and superseded by PR #997. | |
| #2773 build providers: inject snapd snap for latest feature availability | snapcraft | merged | Merged changes that inject the snapd snap into build providers instead of core, improving feature availability and reducing size. Approved by reviewer, passed CI, and modified 4 files (+59/-12 lines). | |
| #661 feat(provider): add snap registration api | craft-application | merged | Merged addition of a snap registration API enabling providers to install extra snaps before lifecycle execution. Approved by two reviewers, changed three files with 110 additions and one deletion, resolving CRAFT-4174. | |
| #2261 build providers: inject the base for classic | snapcraft | merged | Successfully merged pull request that injects the base snap for classic confinement to configure appropriate linker paths during builds. Approved by two reviewers and passed CI checks. | |
| #1037 feat(ProviderService): don't inject snaps if the host and guest architectures differ | craft-application | merged | Merged. Updates ProviderService to install application snaps from the Snap Store instead of injecting them from the host when architectures differ, preventing cross-arch injection issues. Approved by two reviewers with all CI checks passing. |