Changeset 56407 for trunk/.github/workflows/slack-notifications.yml

Timestamp:

08/17/2023 05:56:06 PM (13 months ago)

Author:

desrosj

Message:

Build/Test Tools: Revert Slack messaging changes.

This reverts [56404], [56405], [56406].

More testing is needed to ensure multiple messages are not sent unintentionally.

Unprops desrosj.
See #58867.

File:

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

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

@@ -72 +72 @@
             });
 
-            // All workflows will restart once when a failure is encountered. Don't notify Slack on the first failure.
-            if ( '${{ inputs.calling_status }}' == 'failure' && workflow_run.data.run_attempt == 1 ) {
-              return 'first-failure';
-            }
-
             // When a workflow has been restarted to fix a failure, check the previous run attempt.
             if ( workflow_run.data.run_attempt > 1 ) {
@@ -163 +158 @@
     timeout-minutes: 5
     needs: [ prepare ]
-    if: ${{ ( contains( fromJson( '["failure", "startup_failure"]' ), needs.prepare.outputs.previous_conclusion ) || failure() ) && needs.prepare.outputs.previous_conclusion != 'first-failure' }}
+    if: ${{ inputs.calling_status == 'failure' || failure() }}
 
     steps: