fix: allow parallel installations of signed snapcraft snaps
Metadata
Current evaluation
Merged a fix enabling parallel installations of signed snapcraft snaps, resolving issues #4683 and #4927. Approved by two reviewers, passed CI checks, and integrated.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox run -m lint`?
- [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Fixes a bug where parallel installations of snapcraft would not work if the snapcraft snap was signed.
I recommend reviewing by commit as the refactor commit is noisy.
Fixes #4683
Fixes #4927
(CRAFT-3259)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix enabling parallel installations of signed snapcraft snaps, resolving issues #4683 and #4927. Approved by two reviewers, passed CI checks, and integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix enabling parallel installations of signed snapcraft snaps, resolving issues #4683 and #4927. The change corrects a bug where snap signing previously blocked concurrent instance execution. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix enabling parallel installations of signed snapcraft snaps, resolving issues #4683 and #4927. The change allows multiple signed snapcraft versions to coexist without conflicts. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5087 fix: allow parallel installations of signed snapcraft snaps (7.5) | snapcraft | merged | Merged into hotfix/7.5 to fix parallel installation failures for signed snapcraft snaps. Backports the solution from #5058, resolving issues #4683 and #4927. Approved by two reviewers with all CI checks passing. | |
| #4088 providers: allow parallel installations when not running as a snap | snapcraft | merged | 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. | |
| #161 snap: do not refresh already installed snaps | craft-parts | merged | Merged a port of snapcraft#3617 to craft-parts. The change prevents refreshing already installed snaps by removing two lines of code. Approved by one reviewer and passed CI checks. | |
| #4662 fix(test): do not create 2 snapcraft.yaml and confuse location | snapcraft | merged | Merged a test fix preventing duplicate snapcraft.yaml creation in conflicting root and snap directories. Resolves silent overrides and ensures hooks are detected by clarifying fixture paths. Approved and merged. | |
| #417 fix: allow using a snap from the snap store | craft-application | merged | Merged a fix defaulting to the stable channel for snap store snaps, enabling channel overrides via the CRAFT_SNAP_CHANNEL environment variable. Approved by reviewers, passed CI, and modified two files. | |
| #1950 fix: make charmcraft work with snap parallel installs | charmcraft | merged | Merged to resolve issue #1770, enabling charmcraft to operate with snap parallel installs. Approved by two reviewers, the update modifies five files with 55 additions and 15 deletions. | |
| #3505 snaps: don't validate snaps before `SnapPackage.download()` | snapcraft | merged | Merged a fix that skips snap validation before SnapPackage.download(), resolving branch staging issues (LP#1901733). Approved by two reviewers, passed CI, and changed three files (+16/-11 lines). | |
| #49 packages: add snap package support | craft-parts | merged | Merged after implementing snap download and installation for stage and build snaps. Approved by two reviewers, passed CI, and added 1,072 lines across five files. Follows snapcraft conventions with future refactoring planned. | |
| #228 packages: run snap operations silently | craft-parts | merged | Merged to fix #218 by running snap subprocesses silently, preventing stdout leakage that conflicts with craft-cli during lifecycle execution. Approved by one reviewer, passed CI, and modified two files (+32/-15 lines). | |
| #411 Support snap parallel installs | craft-application | closed | Closed. The initial fix was merged into craft-application, but full parallel snap support requires a follow-up update in craft-providers to correctly parse snap names with underscores. |