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 (7.5) #5087

Merged
merged 2 commits into from
Oct 3, 2024

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.

Same as #5058, but for hotfix/7.5.

I recommend reviewing by commit as the refactor commit is noisy.

Builds on top of #5088
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 force-pushed the work/CRAFT-3259/snap-alias-7.5 branch from 9d6fa2b to 7905258 Compare October 2, 2024 21:31
@mr-cal mr-cal changed the title [draft] build(deps): point craft-providers to work/CRAFT-3259/snap-alias-1.20 [draft] build(deps): point craft-providers to 1.20.4 Oct 3, 2024
@mr-cal mr-cal changed the title [draft] build(deps): point craft-providers to 1.20.4 fix: allow parallel installations of signed snapcraft snaps Oct 3, 2024
@mr-cal mr-cal changed the title fix: allow parallel installations of signed snapcraft snaps fix: allow parallel installations of signed snapcraft snaps (7.5) Oct 3, 2024
@mr-cal mr-cal added the rebase label Oct 3, 2024
@mr-cal mr-cal force-pushed the work/CRAFT-3259/snap-alias-7.5 branch 2 times, most recently from 08a7f78 to cadd63d Compare October 3, 2024 15:10
@mr-cal mr-cal marked this pull request as ready for review October 3, 2024 16:46
@mr-cal mr-cal force-pushed the work/CRAFT-3259/snap-alias-7.5 branch from 984d89d to 6953cbf Compare October 3, 2024 17:02
Copy link
Contributor

@lengau lengau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gg

@mr-cal mr-cal requested a review from a team October 3, 2024 19:32
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 <callahan.kovacs@canonical.com>
Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
@mr-cal mr-cal force-pushed the work/CRAFT-3259/snap-alias-7.5 branch from 6953cbf to 819f926 Compare October 3, 2024 20:13
@mr-cal mr-cal merged commit 119b92a into hotfix/7.5 Oct 3, 2024
12 of 13 checks passed
@mr-cal mr-cal deleted the work/CRAFT-3259/snap-alias-7.5 branch October 3, 2024 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants