Skip to content

[snap] GUI-less snap Launchpad integration#4822

Merged
sharder996 merged 8 commits intomainfrom
gui-less-snap
Apr 21, 2026
Merged

[snap] GUI-less snap Launchpad integration#4822
sharder996 merged 8 commits intomainfrom
gui-less-snap

Conversation

@sharder996
Copy link
Copy Markdown
Collaborator

@sharder996 sharder996 commented Apr 14, 2026

This PR makes the following changes in order to automate the building of a gui-less Multipass snap in Launchpad for the s390x and ppc64 architectures:

  1. Additional supported architectures added to snap/snapcraft.yaml. They have been added here in order to minimum the number of edits made to the gui-less snapcraft file.
  2. Mirrored version of snap/snapcraft.yaml without GUI located at packaging/gui-less/snap/snapcraft.yaml. Because pathing implications and assumptions made by snapcraft, the source of some parts are now the GitHub remote instead of your local development clone.
  3. Python script to programmatically generate the mirrored snapcraft file. This is where edits to the gui-less snapcraft build should be defined.
  4. CI step added to the Lint workflow which verifies that any edits made to the original snapcraft file are properly manifested in the gui-less version.
  5. Optional pre-commit hook that regenerates the mirrored snapcraft file if snap/snapcraft.yaml has been edited.

The Launchpad side can be observed here. Once this is merged in, the snapcraft recipe in Launchpad will be modified to track the main branch, and additional recipes added to mirror the candidate and beta recipes, except without the GUI and on the s390x/ppc64 architectures.


MULTI-2379

@sharder996 sharder996 force-pushed the gui-less-snap branch 3 times, most recently from 48416b3 to 8a9ca1e Compare April 14, 2026 21:29
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.45%. Comparing base (c702588) to head (4491870).
⚠️ Report is 65 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4822      +/-   ##
==========================================
- Coverage   87.54%   87.45%   -0.09%     
==========================================
  Files         258      258              
  Lines       14155    14094      -61     
==========================================
- Hits        12391    12324      -67     
- Misses       1764     1770       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sharder996 sharder996 marked this pull request as ready for review April 15, 2026 04:39
@sharder996 sharder996 requested review from a team and xmkg and removed request for a team April 15, 2026 04:39
@sharder996 sharder996 changed the title Gui less snap [snap] GUI-less snap Launchpad integration Apr 15, 2026
Comment thread snap/snapcraft.yaml Outdated
Comment thread packaging/gui-less/generate-snapcraft.py Outdated
Comment thread packaging/gui-less/generate-snapcraft.py
Comment thread packaging/gui-less/generate-snapcraft.py Outdated
@sharder996 sharder996 force-pushed the gui-less-snap branch 2 times, most recently from 0cd3c0b to 41655dd Compare April 17, 2026 21:41
@sharder996 sharder996 requested a review from xmkg April 18, 2026 14:26
Comment thread packaging/gui-less/generate-snapcraft.py
Copy link
Copy Markdown
Member

@xmkg xmkg left a comment

Choose a reason for hiding this comment

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

Alright, this is as good as it gets. Thanks, @sharder996!

@sharder996 sharder996 added this pull request to the merge queue Apr 21, 2026
Merged via the queue into main with commit f1e2ebe Apr 21, 2026
53 of 56 checks passed
@sharder996 sharder996 deleted the gui-less-snap branch April 21, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants