providers: allow parallel installations when not running as a snap
Metadata
Current evaluation
Merged PR #4088 as commit cb6f3857e, adding a snap-runtime check so non-snap installations can run in parallel.
Resolution: closed resolved
Reason: PR #4088 was approved and merged into main as commit cb6f3857e. It gates SNAP_INSTANCE_NAME lookup on is_snapcraft_running_from_snap(), allowing non-snap executions to use the default name safely.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [X] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [X] Have you successfully run `make lint`?
- [X] Have you successfully run `pytest tests/unit`?
-----
Minor follow up to https://github.com/snapcore/snapcraft/pull/4084 to allow check if snapcraft is running as a snap before reading the envvar `SNAP_INSTANCE_NAME`.
(CRAFT-1660)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4088 as commit cb6f3857e, adding a snap-runtime check so non-snap installations can run in parallel. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #4088 merged into main, checking whether Snapcraft is a snap before reading SNAP_INSTANCE_NAME, allowing parallel provider installations outside snap execution (CRAFT-1660). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged into main. Checks if snapcraft runs as a snap before reading SNAP_INSTANCE_NAME, enabling parallel installations outside the snap. Follows PR #4084, approved, passed CI, 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Adds a runtime check to verify if Snapcraft runs as a snap before reading SNAP_INSTANCE_NAME, enabling parallel installations for non-snap environments. Follows up on PR #4084 with full test coverage. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcore/snapcraft#4084
(confidence 100%)
Direct predecessor that introduced injection of the current Snapcraft snap into build instances.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5085 fix: allow parallel installations of signed snapcraft snaps | snapcraft | merged | Merged PR #5085 fixed parallel installation of signed Snapcraft snaps, resolving #4683 and #4927; #5087 backported the fix to hotfix/7.5. | |
| #411 Support snap parallel installs | craft-application | closed | Parallel-installed snap injection was fixed by PR #467 (commit 77a3582b); #411 was closed as fixed, with a separate craft-providers gap noted. |