Make WordPress Core

Opened 5 months ago

Last modified 3 weeks ago

#64298 reopened enhancement

Move the `failed-workflow` job logic into a reusable workflow

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

Description

The failed-workflow job is present in all workflows that run on push events in all branches receiving security updates.

While the logic within the job is minimal, it still relies on a 3rd-party action and backports are required periodically to ensure the job continues working as expected.

The logic within this job should be moved to a reusable workflow in trunk that all other branches can reference. This not only eliminates the maintenance burden, but also allows older branches to take advantage of any improvements made to the logic without needing to backport.

Attachments (1)

patch.diff (12.8 KB) - added by bsetiawan88 7 weeks ago.

Download all attachments as: .zip

Change History (9)

#1 @desrosj
5 months ago

  • Keywords good-first-bug added

Marking as a good-first-bug for the first part of this ticket: moving the logic into a reusable workflow.

After this is fixed in trunk, then it can be backported to older branches (which is not so much a good first bug to do).

#2 @atik0130961
7 weeks ago

  • Resolution set to wontfix
  • Status changed from new to closed

This is a very sensible improvement. Moving the failed-workflow job into a reusable workflow in trunk will significantly reduce maintenance overhead and eliminate the need for repeated backports across multiple branches.

#3 @desrosj
7 weeks ago

  • Resolution wontfix deleted
  • Status changed from closed to reopened

Hi @atik0130961,

Please leave tickets open until they’re addressed in a commit.

@bsetiawan88
7 weeks ago

#4 @moisesfonseca
3 weeks ago

Reviewed the ticket and patch.
Moving the failed-workflow job to a reusable workflow makes sense given the duplication across workflows.
This should improve maintainability across branches.

#5 @desrosj
3 weeks ago

  • Keywords has-patch added
  • Milestone changed from Future Release to 7.1

Hi @bsetiawan88,

Could you please open a pull request against WordPress/wordpress-develop on GitHub? That will help validate that the workflow files are valid after applying the patch on this ticket. You'll find some guidance around how to contribute using GitHub in the Core Gandbook.

This ticket was mentioned in PR #11425 on WordPress/wordpress-develop by @kapilpaul.


3 weeks ago
#6

Trac ticket: https://core.trac.wordpress.org/ticket/64298

## Use of AI Tools

This ticket was mentioned in PR #11349 on WordPress/wordpress-develop by @johnbillion.


3 weeks ago
#7

#8 @johnbillion
3 weeks ago

Alternate approach: https://github.com/WordPress/wordpress-develop/pull/11349. Needs testing on a separate repo because it only fires when the workflow is present in the default branch.

Note: See TracTickets for help on using tickets.