Changeset 59209

Timestamp:

10/10/2024 05:56:41 PM (3 months ago)

Author:

desrosj

Message:

Build/Test Tools: Temporarily ignore Slack failures.

When many workflows are initiated at the same time, there are often instances where the requests to Slack providing updates are rate limited. This usually happens when the Test Old Branches workflow runs and initiates testing for all workflows in branches that could potentially receive a security update.

Even though everything was successful in the workflow except the message, the workflow run is marked as failed. The next time the same workflow runs for that branch, a “fixed” message will be sent to #core in Slack. The result is a burst of messages that is quite noisy and unnecessary.

This temporarily adds continue-on-error to the jobs responsible for sending the messages until a better solution can be decided on.

See #61701.

File:

• trunk/.github/workflows/slack-notifications.yml (modified) (4 diffs)

trunk/.github/workflows/slack-notifications.yml

@@ -162 +162 @@
     name: Failure notifications
     runs-on: ubuntu-latest
+    continue-on-error: true
     timeout-minutes: 5
     needs: [ prepare ]
@@ -178 +179 @@
     name: Fixed notifications
     runs-on: ubuntu-latest
+    continue-on-error: true
     timeout-minutes: 5
     needs: [ prepare ]
@@ -194 +196 @@
     name: Success notifications
     runs-on: ubuntu-latest
+    continue-on-error: true
     timeout-minutes: 5
     needs: [ prepare ]
@@ -210 +213 @@
     name: Cancelled notifications
     runs-on: ubuntu-latest
+    continue-on-error: true
     timeout-minutes: 5
     needs: [ prepare ]