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

snapcraft.yaml: fix missing primed packages in manifest #15029

Conversation

valentindavid
Copy link
Contributor

@valentindavid valentindavid commented Feb 5, 2025

This change was initiated by a conversation started by Iseoluwa Titiloye https://chat.canonical.com/canonical/pl/qpo9iy53z3fudkbdj4akdnteth

Related to: https://bugs.launchpad.net/snapcraft/+bug/2097440

Copy link

codecov bot commented Feb 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (master@365a0c0). Learn more about missing BASE report.
Report is 287 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master   #15029   +/-   ##
=========================================
  Coverage          ?   78.27%           
=========================================
  Files             ?     1156           
  Lines             ?   154329           
  Branches          ?        0           
=========================================
  Hits              ?   120806           
  Misses            ?    26095           
  Partials          ?     7428           
Flag Coverage Δ
unittests 78.27% <ø> (?)

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.

@ernestl ernestl self-requested a review February 5, 2025 13:23
@valentindavid
Copy link
Contributor Author

The attributes are there, but the manifest is still not correct. I will have to debug a bit deeper into this.

@valentindavid
Copy link
Contributor Author

@valentindavid valentindavid force-pushed the valentindavid/missing-manifest-stage-packages branch from 7910805 to a4b4942 Compare February 5, 2025 14:38
Copy link

github-actions bot commented Feb 5, 2025

Mon Feb 17 12:18:08 UTC 2025
The following results are from: https://github.com/canonical/snapd/actions/runs/13179140672

Failures:

Preparing:

  • openstack:opensuse-tumbleweed-64
  • openstack:opensuse-tumbleweed-64
  • openstack:opensuse-tumbleweed-64
  • openstack:opensuse-tumbleweed-64
  • openstack:opensuse-tumbleweed-64
  • openstack:opensuse-tumbleweed-64
  • openstack:opensuse-15.6-64
  • openstack:opensuse-15.6-64
  • openstack:opensuse-15.6-64
  • openstack:opensuse-15.6-64
  • openstack:opensuse-15.6-64
  • openstack:opensuse-15.6-64

Executing:

  • openstack:debian-sid-64:tests/unit/go:gcc
  • openstack:debian-sid-64:tests/unit/go:clang
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups:kmsg
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups-strict-enforced
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups-self-manage
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups-serial-port
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups-required-or-optional
  • google:ubuntu-25.04-64:tests/main/cgroup-devices-v2
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups-helper
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups:uinput

Restoring:

  • openstack:debian-12-64:tests/unit/c-unit-tests-clang
  • openstack:debian-12-64:tests/unit/
  • openstack:debian-12-64:tests/unit/
  • openstack:debian-12-64:tests/unit/
  • google:ubuntu-25.04-64:tests/main/security-device-cgroups-strict-enforced

@valentindavid valentindavid force-pushed the valentindavid/missing-manifest-stage-packages branch from a4b4942 to 501bd34 Compare February 5, 2025 16:45
@valentindavid
Copy link
Contributor Author

I have done a work-around for https://bugs.launchpad.net/snapcraft/+bug/2097440

Copy link
Contributor

@bboozzoo bboozzoo left a comment

Choose a reason for hiding this comment

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

LGTM

@valentindavid valentindavid force-pushed the valentindavid/missing-manifest-stage-packages branch 3 times, most recently from 9bce881 to da06a76 Compare February 6, 2025 12:38
@valentindavid valentindavid force-pushed the valentindavid/missing-manifest-stage-packages branch from da06a76 to b1661dd Compare February 6, 2025 12:48
Copy link
Collaborator

@ernestl ernestl left a comment

Choose a reason for hiding this comment

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

Thanks

@ernestl ernestl added this to the 2.68.1 milestone Feb 17, 2025
@ernestl
Copy link
Collaborator

ernestl commented Feb 17, 2025

Adding this to 2.68.1, there is a customer waiting for this.

@valentindavid valentindavid merged commit a843c82 into canonical:master Feb 21, 2025
63 of 67 checks passed
@ernestl ernestl modified the milestones: 2.68.1, 2.68.2 Feb 24, 2025
@ernestl ernestl mentioned this pull request Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants