cli: capture logs in logfile
Metadata
Current evaluation
Merged implementation capturing rockcraft logs to a file when running in managed mode. Approved by reviewer, passed CI, and added six lines to one file.
Suggested action: —
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
When `rockcraft` is running in managed mode (i.e. inside an instance), then save the log file.
Referenced from [snapcraft](https://github.com/snapcore/snapcraft/blob/9e10fd1b658df80c35e1996cd92a077e831a8d51/snapcraft/cli.py#L166).
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged implementation capturing rockcraft logs to a file when running in managed mode. Approved by reviewer, passed CI, and added six lines to one file. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes that capture logs to a file when rockcraft runs in managed mode, mirroring snapcraft's implementation. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged implementation that captures logs to a file when rockcraft runs in managed mode, mirroring snapcraft's approach. Accepted without comments. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #100 tests: print rockcraft log files on spread failure | rockcraft | merged | Merged: adds 11 lines to print rockcraft log files during spread test failures, improving debugging visibility. Approved by one reviewer and passed CI checks. | |
| #77 providers: capture logs from instance | rockcraft | merged | Merged a refactor moving capture_logs_from_instance() to rockcraft/providers/providers.py and updating it to use craft-providers temporarily_pull_file(). Approved, resolving CRAFT-1351 with a 63-line reduction across six files. | |
| #3937 tests: add unit test for `capture_logs_from_instance()` | snapcraft | merged | Merged. Adds unit tests for capture_logs_from_instance(), adapted from rockcraft to cover gaps in PR 3935. Approved by one reviewer and passed CI checks before integration. |