Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace test-summary action with custom action code #3119

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

ScottDugas
Copy link
Collaborator

@ScottDugas ScottDugas commented Feb 7, 2025

The test-summary action takes the xml files produced by the tests, but our xml files in the nightly action get too large for the action to load them as a string (~540mb), due to all the logging.
This replaces the test-summary action with built-in steps that print to GITHUB_STEP_SUMMARY. They use the listener to gather the test results and exceptions.

You can see two examples of PRBs with the resulting output. Both of these disable many of the tests so that I could iterate more quickly:

I think there's two followups that could be nice, but decided not to do them right now:

  1. It would probably make sense to remove all the destructiveTest runs from the summary that have 0 tests in them, which would make that table much smaller.
  2. It would probably make sense to trim the stack traces to remove a whole slew of junit code.

@ScottDugas ScottDugas changed the title Action test output Replace test-summary action with custom action code Feb 10, 2025
@ScottDugas ScottDugas marked this pull request as ready for review February 10, 2025 18:14
@ScottDugas ScottDugas requested review from MMcM and removed request for alecgrieser February 10, 2025 18:14
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.

1 participant