Skip to content

Conversation

@martinpitt
Copy link
Contributor

Some roles produce ansible/test logs which are too large to sensibly/comfortably view in the main workflow log view. Show only the interesting failures there (the lines around fatal:) and upload the full logs as artifact.


See recent run for example. The primary PR for this is linux-system-roles/.github#99 , this tests it against a real-life project with a failing F42 test.

Some roles produce ansible/test logs which are too large to
sensibly/comfortably view in the main workflow log view. Show only the
interesting failures there (the lines around `fatal:`) and upload the
full logs as artifact.
@codecov
Copy link

codecov bot commented Apr 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.18%. Comparing base (2d7c4ba) to head (eac2499).
Report is 52 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #260      +/-   ##
==========================================
+ Coverage   61.09%   61.18%   +0.08%     
==========================================
  Files           2        2              
  Lines         910      912       +2     
==========================================
+ Hits          556      558       +2     
  Misses        354      354              
Flag Coverage Δ
sanity ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@martinpitt
Copy link
Contributor Author

linux-system-roles/.github#99 landed -- @richm what should we do with this one? Land it, or wait for the next round of "apply github.git changes" to pick it up?

@richm richm merged commit 0b6020f into linux-system-roles:main Apr 15, 2025
20 of 21 checks passed
@richm
Copy link
Contributor

richm commented Apr 15, 2025

linux-system-roles/.github#99 landed -- @richm what should we do with this one? Land it, or wait for the next round of "apply github.git changes" to pick it up?

Merging now because it helps with debugging

@martinpitt martinpitt deleted the test-logs branch April 15, 2025 17:10
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.

2 participants