fix(executor): clean overlay directories
Metadata
Current evaluation
Merged into main as commit bea8d2dc, fixing #1207/CRAFT-4697 by removing overlay directories during executor cleanup; all CI checks passed.
Resolution: closed resolved
Reason: PR #1208 was approved and merged as commit bea8d2dc. Its executor cleanup now removes configured overlay directories when cleaning through Step.OVERLAY, resolving #1207.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #1207
CRAFT-4697
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged into main as commit bea8d2dc, fixing #1207/CRAFT-4697 by removing overlay directories during executor cleanup; all CI checks passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1208 as bea8d2dc, fixing #1207/CRAFT-4697 by removing executor overlay directories during cleanup; all CI checks passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1208 as bea8d2dc, fixing #1207/CRAFT-4697 by cleaning executor overlay directories. All CI checks passed. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for overlay directory cleanup in the executor, resolving #1207 and CRAFT-4697. Approved by four reviewers, passed all CI checks, and modified three files with 57 additions. No changelog entry was added. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to clean overlay directories in the executor, resolving issue #1207 and CRAFT-4697. The fix was integrated following maintainer approval, with a brief review regarding changelog documentation. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-parts#1207
(confidence 100%)
The linked issue is explicitly identified as fixed by PR #1208 and tracked under CRAFT-4697.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1207 Overlay directory is not removed on a clean | craft-parts | closed | Fixed in merged PR #1208 (commit bea8d2dc), which removes executor overlay directories during cleanup. | |
| #345 executor: fix directory cleaning | craft-parts | merged | Merged PR #345 fixed executor cleanup logic in commit b0086bbe, preventing lifecycle directories from unexpectedly remaining and resolving #329. | |
| #135 executor: add overlay step handling (CRAFT-86) | craft-parts | merged | Merged PR #135 via b7527d5b, adding overlay-step execution handling that mounts the overlay stack during builds. |