← Back to issue list

snap: add remove hook to delete base instances and base images

View original Github issue

Metadata

Project
snapcraft
Number
#4014
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged into main. Adds a remove hook to delete LXD base instances and images when snapcraft is uninstalled. Approved by one reviewer, passed CI checks, and caused no coverage changes. Resolves CRAFT-1488.

Suggested action:

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] Have you successfully run `make lint`? - [X] Have you successfully run `pytest tests/unit`? ----- When snapcraft is uninstalled, the base instances and images for LXD are removed. (CRAFT-1488)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged into main. Adds a remove hook to delete LXD base instances and images when snapcraft is uninstalled. Approved by one reviewer, passed CI checks, and caused no coverage changes. Resolves CRAFT-1488.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Introduces a remove hook to automatically clean up LXD base instances and images upon snapcraft uninstallation, resolving CRAFT-1488. Codecov confirms no coverage impact.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Adds a remove hook to automatically delete LXD base instances and images when snapcraft is uninstalled, resolving CRAFT-1488. Code coverage unchanged.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#199 snap: add remove hook to delete rockcraft data from lxd rockcraft merged Merged implementation of a snap remove hook that automatically deletes rockcraft's LXD images, instances, and project data upon uninstallation. Approved by two reviewers, passed CI checks, and updated four files.
75%
#207 snap: add configure hook to delete outdated instances from lxd rockcraft merged Merged. Adds a snap configure hook to remove outdated LXD images and instances during Rockcraft installation or refresh. Approved by two reviewers and merged after addressing test failures from spread resources and runner deprecation.
74%
#1984 fix(snap): check if LXD is actually installed charmcraft merged Merged fix updating configure and remove hooks to verify LXD installation before proceeding. Prevents unwanted LXD snap installations on Ubuntu that cause delays. Approved by two reviewers, passed CI, and resolves #1982.
71%