Make WordPress Core

Opened 2 years ago

Closed 4 months ago

#56150 closed task (blessed) (fixed)

Add GitHub Action job summaries to workflows

Reported by: desrosj's profile desrosj Owned by:
Milestone: 6.4 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: good-first-bug
Focuses: performance Cc:

Description

In May, GitHub released Job Summaries for GitHub Actions, which allows markdown or raw HTML to be displayed on the workflow run summary page for jobs.

This ticket is for exploring ways to use this feature in the WordPress Core workflows to make it easier to understand the data and outcomes buried in each step's log output within a job.

Change History (13)

#1 @desrosj
2 years ago

  • Keywords good-first-bug added

Was chatting with @johnbillion a bit in Slack about how this feature could be utilized in Core. He shared one example from a project of his.

After some discussion, the most apparent place that this could be useful is within the PHPUnit workflow to display a table with data around how many:

  • tests were identified as risky.
  • tests were skipped.
  • tests encountered errors.
  • total tests were found.
  • assertions were made.
  • etc.

I'm marking this as good-first-bug as it's self-contained, and a great way to dig in to learn more about GitHub Actions.

This ticket was mentioned in PR #3013 on WordPress/wordpress-develop by sarahricker.


2 years ago
#2

  • Keywords has-patch added

This ticket was mentioned in Slack in #core by desrosj. View the logs.


2 years ago

#5 @desrosj
21 months ago

I think it would be great to use this feature as a way to surface the results of the performance testing introduced in [55459].

The data is submitted to an external dashboard and is visible within the logs, but having it presented as a job summary would be really neat. cc/@joemcgill @flixos90 @mukesh27

#6 @joemcgill
21 months ago

  • Focuses performance added
  • Keywords needs-patch added; has-patch removed
  • Milestone changed from Future Release to 6.3

I dig this idea. Thanks @desrosj!

#7 @joemcgill
18 months ago

  • Type changed from enhancement to task (blessed)

I still really like the idea of displaying automated performance benchmarks on PRs using this method, but doing so should be preceded by #58358.

Since this ticket is only related to our build tools I'm changing it to a task ticket because it can be worked on outside the normal release cycle. @desrosj feel free to change it back if that's incorrect.

#8 @mukesh27
17 months ago

  • Milestone changed from 6.3 to Future Release

#10 @whyisjake
16 months ago

  • Keywords needs-testing added; needs-patch removed

Reviewing tickets with @mikestraw at WCUS. Has a patch, needs testing.

Last edited 16 months ago by whyisjake (previous) (diff)

This ticket was mentioned in Slack in #core-performance by swissspidy. View the logs.


15 months ago

#12 @swissspidy
15 months ago

In 56506:

Build/Test Tools: Compare results in performance measurement workflow.

This extends the performance test workflow added in [55459] to also run tests against the target branch (if running on a pull request) or the previous commit (if running on trunk).

Those results are then compared with the ones from the current commit, and the difference is displayed as a GitHub Actions workflow summary for convenience.

Props mukesh27, flixos90, desrosj, joemcgill, swissspidy.
Fixes #58358, #58359.
See #56150.

#13 @joemcgill
5 months ago

  • Keywords close added

@desrosj Now that we've implemented GH Action summaries in [56506] and in the GB repo (see a summary from the Performance Test actions), I wonder if we can consider this complete and create follow-up issues for adding summaries to specific workflows?

#14 @desrosj
4 months ago

  • Keywords needs-testing close removed
  • Milestone changed from Future Release to 6.4
  • Resolution set to fixed
  • Status changed from new to closed

@joemcgill That makes sense to me. This was mainly meant as a reminder to see if there were valid uses for us!

Note: See TracTickets for help on using tickets.