spread: disable journal debug dump unless configured
Metadata
Current evaluation
Merged PR #2913 (commit d444bb248): journal debug dumps now require SPREAD_DEBUG_JOURNAL=true; Travis CI enables them by default.
Resolution: closed resolved
Reason: PR #2913 was approved and merged as commit d444bb248, preventing unnecessary journal output by making it opt-in for Spread tests.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Most failures do not require the journal to debug. To prevent flooding
the user with a lot of unnecessary information, only dump the journal
if SPREAD_DEBUG_JOURNAL=true. Set this default for Travis CI.
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2913 (commit d444bb248): journal debug dumps now require SPREAD_DEBUG_JOURNAL=true; Travis CI enables them by default. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2913: journal debug dumps now require SPREAD_DEBUG_JOURNAL=true, with the default enabled for Travis CI to prevent unnecessary log flooding. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and passing CI. Disables journal debug dumps by default unless SPREAD_DEBUG_JOURNAL=true is set to prevent log flooding. Modified two files with a +9/-2 line change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes disabling journal debug dumps in spread tests unless SPREAD_DEBUG_JOURNAL=true is set, preventing log flooding and configuring the Travis CI default. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. |