Skip to content

ci: dump sccache logs on integration failures - #2851

Merged
sylvestre merged 1 commit into
mozilla:mainfrom
JalenBuildsHub:ci/dump-integration-logs-1685
Sep 12, 2026
Merged

ci: dump sccache logs on integration failures#2851
sylvestre merged 1 commit into
mozilla:mainfrom
JalenBuildsHub:ci/dump-integration-logs-1685

Conversation

@JalenBuildsHub

Copy link
Copy Markdown
Contributor

The integration matrix currently loses the sccache server diagnostics when a test container fails. This enables a debug log for every integration test container, persists it through the existing coverage bind mount, clears the append-only log before each subtest, and prints the failing subtest's log from the job on failure.

Fixes #1685

Validated with Compose expansion across all 18 test services, Actionlint, both branches of the failure-log shell block, and cargo fmt -- --check. Native Clippy/tests were not completed locally because this Windows machine does not have the MSVC link.exe; the PR's Linux CI will run them.

@codecov-commenter

codecov-commenter commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.35%. Comparing base (6089d04) to head (db13a7a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2851      +/-   ##
==========================================
- Coverage   76.22%   75.35%   -0.87%     
==========================================
  Files          72       72              
  Lines       39890    39890              
==========================================
- Hits        30406    30059     -347     
- Misses       9484     9831     +347     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sylvestre
sylvestre merged commit 0ebbbf2 into mozilla:main Sep 12, 2026
54 of 58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: when an integration test fail, dump the log

3 participants