Upstream snap hooks
Metadata
Current evaluation
Centralized upstream snap hooks in craft-providers via merged PR #710 (commit 5bbaf39), shipping in 2.2.0 as reusable LXD cleanup utilities.
Resolution: closed resolved
Reason: Merged PR #710 in commit 5bbaf39 added craft_providers/hookutil.py. The 2.2.0 changelog confirms it provides dependent projects reusable utilities for cleaning up LXD instances.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Every *craft application has its own slightly different implementation of snap hooks that clean stale lxd instances, but they all essentially do the same thing. Some are slightly broken or incorrect or incomplete. These could be centralized and corrected in starbase.
Possibly this ticket should be moved to one of the craft-* libraries instead, not sure which one.
I'll mention this ticket from the private repo where I recently reworked the snap hooks, so github links it here for those who have access to that repo.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Centralized upstream snap hooks in craft-providers via merged PR #710 (commit 5bbaf39), shipping in 2.2.0 as reusable LXD cleanup utilities. | |
| qwen/qwen3.6-35b-a3b | — | closed stale | Proposed centralizing and correcting snap hook implementations across craft applications in starbase or a craft-* library. Closed due to inactivity after migration to Jira CRAFT-3821. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/craft-providers#710
(confidence 99%)
The merged upstream-snap-hooks PR added the centralized hook utilities requested by #709.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #710 feat: upstream snap hooks | craft-providers | merged | Merged as 5bbaf39, adding reusable configure and remove snap hooks to prune obsolete LXD base instances, derived instances, images, and projects. | |
| #3660 providers: integrate craft-providers support | snapcraft | merged | PR #3660 by cmatsuoka integrated craft-providers into Snapcraft, running lifecycle commands in LXD/Multipass instances. Merged into the 7.0 line via merge commit 7c2ac0027 (squash commit 8a06cdbbd). | |
| #4014 snap: add remove hook to delete base instances and base images | snapcraft | merged | Merged #4014 in commit 4e41bf0c9, adding an uninstall hook that deletes Snapcraft's LXD base instances and images (CRAFT-1488). | |
| #3962 Setup snapd for snapcraft_legacy providers | snapcraft | merged | Merged #3962 as 6122e30fd, adding craft-providers-style snapd setup for snapcraft_legacy LXD containers so /run/snapd.socket is available during core18 spread tests. |