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

core-initrd: fix some issues with ubuntu-core-initramfs package builds #14909

Conversation

alfonsosanchezbeato
Copy link
Member

  • Fix builds in launchpad
  • Add changelog entries automatically
  • Fix golang dependency
  • Include snapd version and Ubuntu series in package version

Create new changelog entry when creating new ubuntu-core-initramfs
debian source packages. Tweak the version so now it includes snapd
version and Ubuntu series.
It was incorrectly a run dependecy.
Launchpad dos not allow change in the root of the debian package
directory, create the go cache elsewhere.
@alfonsosanchezbeato alfonsosanchezbeato added the Run nested The PR also runs tests inluded in nested suite label Jan 8, 2025
the ubuntu-core-initramfs source packages.
Copy link

codecov bot commented Jan 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.24%. Comparing base (24a0034) to head (1408e64).
Report is 98 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #14909      +/-   ##
==========================================
+ Coverage   78.20%   78.24%   +0.03%     
==========================================
  Files        1151     1157       +6     
  Lines      151396   152828    +1432     
==========================================
+ Hits       118402   119576    +1174     
- Misses      25662    25869     +207     
- Partials     7332     7383      +51     
Flag Coverage Δ
unittests 78.24% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Copy link
Member

@Meulengracht Meulengracht left a comment

Choose a reason for hiding this comment

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

thank you

@alfonsosanchezbeato alfonsosanchezbeato merged commit f11e834 into canonical:master Jan 13, 2025
53 of 60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Run nested The PR also runs tests inluded in nested suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants