[wp-trac] [WordPress Trac] #61701: Rate limiting can occur when sending Slack notifications out through GitHub Actions
WordPress Trac
noreply at wordpress.org
Thu Jul 18 19:03:40 UTC 2024
#61701: Rate limiting can occur when sending Slack notifications out through GitHub
Actions
------------------------------+--------------------
Reporter: desrosj | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 6.7
Component: Build/Test Tools | Version:
Severity: normal | Keywords:
Focuses: |
------------------------------+--------------------
In some scenarios, Slack notifications being sent through GitHub Actions
are being rate limited. This is mainly happening when the scheduled
testing of old branches able to receive security updates kicks off, but it
could happen any time there is enough of a volume of events that send
notifications.
This seems to happen because of how GHA jobs are queued and picked up.
Since the Slack notification step is a separate job, many are queued while
waiting for the completion of more time intensive ones, such as PHPUnit or
E2E testing. While Slack does not publish exact parameters for rate
limiting, they are [https://api.slack.com/apis/rate-limits#overview
documented for webhooks as 1 per second with short bursts >1 allowed].
There's currently [https://github.com/slackapi/slack-github-
action/issues/245 no built in way for the Slack action to handle this],
though [https://github.com/slackapi/slack-github-action/issues/312 retries
for failed calls is being discussed as a possible feature for V2] of the
action.
It would be great to find a way to retry or space out these attempts until
then to prevent workflow failures from occurring solely because the
notification is unsuccessful.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/61701>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list