tests: print rockcraft log files on spread failure
Metadata
Current evaluation
Merged in commit 3a19387b: Spread failures now print the latest Rockcraft log for improved diagnostics.
Resolution: closed resolved
Reason: PR canonical/rockcraft#100 was approved and merged as commit 3a19387b. Its CI output demonstrated that failed tests printed the relevant Rockcraft log contents.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Print rockcraft log files when spread fails, to capture more details in scenarios like [this](https://github.com/canonical/rockcraft/actions/runs/3143892833/jobs/5109299391)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in commit 3a19387b: Spread failures now print the latest Rockcraft log for improved diagnostics. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #100 (commit 3a19387b), adding 11 lines to print Rockcraft log files after Spread failures and reveal underlying errors. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged: adds 11 lines to print rockcraft log files during spread test failures, improving debugging visibility. Approved by one reviewer and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a pull request that prints rockcraft log files on spread test failures to improve debugging visibility. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #174 tests: check before handling rockcraft instances | rockcraft | merged | Merged PR #174 added an existence check before Rockcraft LXC instance cleanup, so spread tests that create no instances no longer fail during teardown. | |
| #519 spread: reorganize spread tests | rockcraft | merged | Merged PR #519 as commit 90d85343, consolidating most spread tests in the renamed rockcraft suite on Jammy VMs and adding a foreign suite for cross-system tests. | |
| #173 tests: check before handling rockcraft instances | rockcraft | closed | Merged PR #173 added a presence check before Rockcraft LXC cleanup, preventing spread tests that create no Rockcraft instance from trying to stop or delete one. | |
| #1304 test(spread): fix cross-test contamination | rockcraft | merged | Merged PR #1304: spread variants shared the python-from-chisel-bare dir, causing cross-test contamination; each test now copies a clean rockcraft.yaml template in first (+6 lines, 2 files). |