overlay: pass error message from chrooted process (CRAFT-580)
Metadata
Current evaluation
Merged updates to chroot error handling that pass error message strings and raise ChrootExecutionError instead of original exceptions, avoiding custom unpickling. Approved by one reviewer, passed CI, and modified four files.
Suggested action: —
No scores available.
Issue body
Pass error message string and raise `ChrootExecutionError` instead
of passing the original exception. This avoids implementing `__reduce__`
for unpickling different possible errors.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged updates to chroot error handling that pass error message strings and raise ChrootExecutionError instead of original exceptions, avoiding custom unpickling. Approved by one reviewer, passed CI, and modified four files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updated chroot error handling to pass error message strings and raise ChrootExecutionError instead of original exceptions, eliminating the need for custom __reduce__ unpickling logic. Resolves CRAFT-580. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #139 overlays: replace pychroot with non-magic chroot (CRAFT-558) | craft-parts | merged | Merged changes replacing the hackish pychroot library with a simpler, non-magic chroot implementation. Resolves interpreter and platform restrictions, fixes CI failures, and passed reviewer approval. | |
| #535 packages: don't write info logs in overlays | craft-parts | merged | Merged a fix that downgrades overlay package logging from info to debug to prevent terminal deadlocks during chroot subprocess operations. A main-process info message replaces it. Approved by two reviewers and passed CI. | |
| #134 overlay: add overlay package install (CRAFT-81) | craft-parts | merged | Merged. Adds overlay package installation by executing maintainer scripts in a chrooted environment to correctly update the package database. Approved by two reviewers, with clarifications added on pychroot usage and empty file safety. |