Skip to content

Commit 6a055dc

Browse files
authored
[CICD-428] Remove notify step from e2e deploy (#117)
Removes the notify step of our e2e workflow. The failures in this workflow have consistently been unrelated to the action itself and are therefore not actionable. Instead, they indicate an issue with some other service we depend on (GitHub, Docker Hub, SSH Gateway, WPE User Portal). These services are either out of our control or are already monitored in other ways. Bonus: this gets rid of a dependency on voxmedia/github-action-slack-notify-build, which has been deprecated.
1 parent 7f66122 commit 6a055dc

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

.github/workflows/e2e-deploy.yml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -41,18 +41,3 @@ jobs:
4141
- name: Validate deploy results
4242
run: |
4343
[ ${{ fromJson(steps.fetchResult.outputs.response).status }} = "success" ] || exit 1
44-
notify:
45-
runs-on: ubuntu-latest
46-
if: ${{ !cancelled() }}
47-
needs: run_action
48-
steps:
49-
- name: Notify slack on failure
50-
if: needs.run_action.result == 'failure' && github.ref == 'refs/heads/main'
51-
env:
52-
SLACK_BOT_TOKEN: ${{ secrets.SLACK_NOTIFICATIONS_BOT_TOKEN }}
53-
uses: voxmedia/github-action-slack-notify-build@v1
54-
with:
55-
# Channel: status-github-action
56-
channel_id: C03QDL9U0TW
57-
status: FAILED
58-
color: danger

0 commit comments

Comments
 (0)