Skip to content

Commit

Permalink
feat: add discord e2e status reporting
Browse files Browse the repository at this point in the history
Since we have recently added Discord as another chat platform, we should
also report the status of E2E jobs there in addition to Slack.
This commit introduces this capability by mirroring a PR that was made in
instructlab/eval: instructlab/eval#200

Signed-off-by: Oleg S <[email protected]>
  • Loading branch information
RobotSail committed Dec 18, 2024
1 parent 8e6be0d commit ee03bdd
Showing 1 changed file with 24 additions and 0 deletions.
24 changes: 24 additions & 0 deletions .github/workflows/e2e-nvidia-l40s-x4.yml
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,30 @@ jobs:
env:
SLACK_BOT_TOKEN: ${{ secrets.SON_OF_JEEVES_TOKEN }}

- name: Send Discord notification for failure
if: failure() && steps.check_pr.outputs.is_pr == 'false'
uses: sarisia/actions-status-discord@65843b6a7d18626c252a055e247ccad1f41b4004 # v1.15.1
with:
webhook: ${{ secrets.SON_OF_JEEVES_DISCORD_WEBHOOK }}
status: ${{ job.status }}
title: "e2e-nvidia-l40s-x4"
description: |
Job in **${{ github.repository }}** running on branch `${{ steps.check_pr.outputs.pr_or_branch }}` completed **with failures** ❌
Click [here](${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}) to view details.
color: 0xCB2431 # Red color for failure

- name: Send Discord notification for success
if: success() && steps.check_pr.outputs.is_pr == 'false'
uses: sarisia/actions-status-discord@65843b6a7d18626c252a055e247ccad1f41b4004 # v1.15.1
with:
webhook: ${{ secrets.SON_OF_JEEVES_DISCORD_WEBHOOK }}
status: ${{ job.status }}
title: "e2e-nvidia-l40s-x4"
description: |
Job in **${{ github.repository }}** running on branch `${{ steps.check_pr.outputs.pr_or_branch }}` completed **successfully** ✅
Click [here](${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}) to view details.
color: 0x28A745 # Green color for success

stop-large-ec2-runner:
needs:
- start-large-ec2-runner
Expand Down

0 comments on commit ee03bdd

Please sign in to comment.