← Back to issue list

providers: support `rockcraft clean` for a rockcraft environment

View original Github issue

Metadata

Project
rockcraft
Number
#108
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged implementation of rockcraft clean to delete LXD instances. Follows snapcraft and charmcraft patterns using the Executor object. Approved by reviewers and passed CI checks.

Suggested action: —

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Support the command `rockcraft clean` to clean a rockcraft environment (i.e. delete a LXD instance). I'm following the same implementation recently done in [snapcraft](https://github.com/snapcore/snapcraft/pull/3875) and [charmcraft](https://github.com/canonical/charmcraft/pull/835), where instances are deleted via the `Executor` object. (CRAFT-1373)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged implementation of rockcraft clean to delete LXD instances. Follows snapcraft and charmcraft patterns using the Executor object. Approved by reviewers and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged code adding rockcraft clean command to delete LXD instances. Implementation follows snapcraft and charmcraft patterns using the Executor object, resolving CRAFT-1373.
qwen3.6-35b-a3b-mtp-q6 — — Merged implementation adding rockcraft clean command to delete LXD instances for rockcraft environments. Follows snapcraft and charmcraft patterns using the Executor object, resolving CRAFT-1373.

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.
78%
#174 tests: check before handling rockcraft instances rockcraft merged Merged a fix wrapping rockcraft LXC instance cleanup in an if condition to prevent errors when spread tests do not create instances. Approved by two reviewers, passed CI, and modified one file.
71%