Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: allow parallel installations of signed snapcraft snaps #5085

Open
wants to merge 3 commits into
base: hotfix/8.4
Choose a base branch
from

Conversation

mr-cal
Copy link
Collaborator

@mr-cal mr-cal commented Oct 2, 2024

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • 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)

@mr-cal mr-cal marked this pull request as draft October 2, 2024 21:16
@mr-cal mr-cal changed the base branch from main to hotfix/8.4 October 2, 2024 23:41
@mr-cal mr-cal changed the title [draft] build(deps): point craft-providers to work/CRAFT-3259/snap-alias [draft] build(deps): point craft-providers to 2.0.3 Oct 3, 2024
@mr-cal mr-cal changed the title [draft] build(deps): point craft-providers to 2.0.3 fix: allow parallel installations of signed snapcraft snaps Oct 3, 2024
@mr-cal mr-cal added the rebase label Oct 3, 2024
@mr-cal mr-cal marked this pull request as ready for review October 3, 2024 15:53
Rename legacy variables to match snapd's definition:
snap_name->snap_instance_name (possibly aliased)
snap_store_name->snap_name (unaliased)

Signed-off-by: Callahan Kovacs <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant