[wp-trac] [WordPress Trac] #52644: Configure Slack notifications for GitHub Action workflows
WordPress Trac
noreply at wordpress.org
Tue Aug 3 16:25:55 UTC 2021
#52644: Configure Slack notifications for GitHub Action workflows
------------------------------+---------------------
Reporter: desrosj | Owner: (none)
Type: task (blessed) | Status: new
Priority: normal | Milestone: 5.9
Component: Build/Test Tools | Version: 5.6
Severity: normal | Resolution:
Keywords: has-patch | Focuses:
------------------------------+---------------------
Comment (by desrosj):
Some revised notes after [51535-51537]:
- Slack notifications are now controlled by a separate workflow that runs
on the `workflow_run` event when a listed workflow is `completed`. I took
this route to prevent repeating large amounts of code across all workflows
where Slack notifications are desired. But ideally, this workflow would be
published as an action that could be defined in the `uses:` for a step.
- The first successful job following a failure will now be posted as a
"fixed" notice to #core. There are some potential edge cases with the
current implementation where re-running workflows could result in the
wrong workflow run being used for comparison.
- All workflow results regardless of outcome are posted to to the #core-
firehose.
- All workflow results regardless of outcome are posted to the new #core-
ci-firehose channel.
- Since these notifications are controlled in a separate workflow
initiated on the `workflow_run` event, this does not technically need to
be backported (all `workflow_run` events are created against the primary
branch). But, [51511-51512,51535-51537] could be backported just to have
consistent contents of the `.github/workflows` directory.
- The messages themselves are configured within Slack Workflows.
I'm going to leave this open in case any adjustments are needed.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/52644#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list