recording: record the snaps installed on the machine
Metadata
Current evaluation
Merged host-installed snap recording in commit 45131df2a, building on #1566; fake-snapd test refactoring continued in #1569.
Resolution: closed resolved
Reason: PR #1567 was approved and merged as commit 45131df2a, adding recording of snaps installed on the build machine. The follow-up fake-snapd test work was merged in #1569.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
This requires #1566
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged host-installed snap recording in commit 45131df2a, building on #1566; fake-snapd test refactoring continued in #1569. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1567 (commit 45131df2a) records snaps installed on the machine, building on merged #1566; #1569 was the linked fake-snapd test follow-up. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after passing CI and receiving approval from two reviewers. The change records installed snaps on the machine and depends on PR #1566. Updates span 8 files with a net addition of 126 lines. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged implementation to record locally installed snaps. Requires PR #1566 and aligns with snapd fake merge #1569. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#1566
(confidence 98%)
Required prerequisite that added recording of build-snaps installed during the pull.
-
Related To:
snapcraft#1569
(confidence 95%)
Merged follow-up refactoring the fake snapd used by recording tests.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1566 recording: record build-snaps installed during the pull | snapcraft | merged | Merged as e1d74020b, recording build-snaps installed during the pull and implementing canonical/snapcraft#1556. | |
| #1295 recording: record build packages installed in the snap | snapcraft | merged | Merged PR #1295 as commit 9a475ba5d, adding recording of build packages installed in the snap. | |
| #1293 recording: record stage packages installed in the snap | snapcraft | merged | Merged PR #1293 as commit 9b3f7a05b, adding recording of stage packages installed in the snap after prerequisite #1292. | |
| #1556 build-snaps recording | snapcraft | closed | Build-snaps recording was implemented and merged in canonical/snapcraft#1566 via commit e1d74020b. | |
| #1598 recording: do not crash when snapd is not installed | snapcraft | merged | Recording no longer crashes when snapd is missing; the fix was merged in commit 73565f0d5, with related armhf test corrections in #1595. | |
| #1529 recording: record the packages installed in the machine | snapcraft | merged | Merged #1529 added recording of packages installed on the host, covering the package-recording portion of #1452 alongside prerequisite #1528 (958024822). |