test(spread): trace logs in spread tests
Metadata
Current evaluation
Merged PR #5389 as ff6f5950e, adding trace logging to spread tests for better failure diagnostics.
Resolution: closed resolved
Reason: PR #5389 was merged into main in commit ff6f5950e. Its six-line change enables trace logging in two files, despite three Google-hosted spread integration jobs still failing.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5389 as ff6f5950e, adding trace logging to spread tests for better failure diagnostics. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5389 in commit ff6f5950e to add trace logging to spread tests. Google-hosted spread jobs remained failing, attributed during review to extra Snapcraft output. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval. Added trace logging to spread tests. Despite CI failures from extra snapcraft output, the change was approved and merged following a related feature branch integration to prevent conflicts. | |
| qwen/qwen3.6-35b-a3b | — | — | Added trace logs to spread tests. Initial CI failures were attributed to extra output. Merging was deferred to coordinate with a concurrent branch merge to main. The PR was ultimately merged after review. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after adding trace logs to spread tests. Review was paused to align with the feature/snapcraft-test branch merge to main, preventing a large conflict. Changes were successfully integrated. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1385 test: show exception tracebacks in spread tests | charmcraft | merged | PR #1385 merged a one-line test change to display exception tracebacks in spread-test failures. | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged #1366 at commit 05b18a03, reducing flakiness in the Spread release test and resolving #1362. | |
| #5406 test: drop verbose flags in spread tests | snapcraft | merged | Merged PR #5406 (c4722d22f) removed verbose flags from six spread test files as a follow-up to #5389, despite three spread integration checks failing. | |
| #723 fix(testing): more intelligent test handling | craft-application | merged | Merged PR #723 via commit 6b319b77, running Spread tests as root and strictly validating only top-level spread.yaml keys. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged PR #534 (commit 4f89bb7) worked around #533 by skipping the flaky snap names-query assertion while retaining the other spread-test checks. | |
| #2711 tests: print journal logs when spread tests fail | snapcraft | merged | Merged PR #2711 (commit 33e759874), adding debug and debug-each helpers to print journal logs when individual or project-wide spread tests fail. | |
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged into master as commit 1cf0f8c9f "tests: minor cleanups on the spread tests (#1287)" — a test-only tidy-up touching 4 spread test files (+16/-6), reviewed and approved by fgimenez, with coverage unchanged at 95.27%. | |
| #16 tests: add spread support | rockcraft | merged | Merged PR #16 added Spread support for CRAFT-865 via commit 3aeba2b3 (merge commit 5bf2680a). | |
| #2943 spread: capture developer debug information | snapcraft | merged | Merged PR #2943 (commit 2fe8456b8) enabled developer debug output in spread, improving failure diagnostics without cluttering normal logs. | |
| #717 fix(test): minor test fixes | craft-application | merged | Merged PR #717 (commit 9f02b30f) switched spread tests to streaming output and adopted Spread's new reroot key, fixing #704 and #715. |