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.


I originally tested this only against the sudo and cockpit rules, both of which produce relatively small test logs. But it's already unwieldy on firewall, and on storage it's completely unusable.

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.
@martinpitt
Copy link
Contributor Author

Applied to firewall in linux-system-roles/firewall#260 for a real-life test run.

@martinpitt martinpitt requested a review from Copilot April 14, 2025 07:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

@martinpitt martinpitt requested a review from richm April 14, 2025 07:16
@martinpitt
Copy link
Contributor Author

Also pushed to linux-system-roles/storage#518

@richm richm merged commit 41785e3 into linux-system-roles:main Apr 15, 2025
@martinpitt martinpitt deleted the test-logs branch April 15, 2025 17:06
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