← Back to issue list

spread: capture developer debug information

View original Github issue

Metadata

Project
snapcraft
Number
#2943
Type
pull request
State
merged
Author
cjp256
Labels
Created
Updated
Closed

Current evaluation

Merged changes setting SNAPCRAFT_ENABLE_DEVELOPER_DEBUG=yes in the spread environment to capture debug logs on failure. Approved by one reviewer, passed CI, and updated four files with twelve added lines.

Suggested action:

No scores available.

Issue body

This information can often be useful to diagnose what is wrong. Since spread only spits out the output on failure, it shouldn't clutter up typical logs. Sets SNAPCRAFT_ENABLE_DEVELOPER_DEBUG=yes in the spread env. 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
qwen/qwen3.6-35b-a3b Merged changes setting SNAPCRAFT_ENABLE_DEVELOPER_DEBUG=yes in the spread environment to capture debug logs on failure. Approved by one reviewer, passed CI, and updated four files with twelve added lines.
qwen3.6-35b-a3b-mtp-q6 Merged changes that set SNAPCRAFT_ENABLE_DEVELOPER_DEBUG=yes in the spread environment. This captures debug information on test failures without cluttering standard logs.
qwen3.6-35b-a3b-mtp-q6 Merged changes to set SNAPCRAFT_ENABLE_DEVELOPER_DEBUG=yes in the spread environment. This captures developer debug information on test failures without cluttering standard logs.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2793 tests: enable SNAPCRAFT_BUILD_INFO for spread snapcraft merged Merged following approval and passing CI. Enabled SNAPCRAFT_BUILD_INFO for spread tests to cover manifest changes. Included a fix for the nil-basic spread test. Minimal diff across three files.
78%
#5389 test(spread): trace logs in spread tests snapcraft merged 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.
72%
#1996 tests: add SNAPCRAFT_KEEP_DATA to debug snapcraft merged Merged test updates adding the SNAPCRAFT_KEEP_DATA environment variable for debugging. Approved by two reviewers, the change modifies two files with a net addition of 14 lines. Submitted as a farewell gift to @sergiusens.
71%
#141 providers: capture logs with debug-level logging rockcraft merged Merged a fix changing captured log emission from trace to debug level, resolving a bug where logs were omitted in quiet and brief modes. Approved by two reviewers, passed CI, and aligns with a snapcraft fix. Resolves #140.
70%