-
Notifications
You must be signed in to change notification settings - Fork 188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add CI slack notifications. #1961
Conversation
🟥 CI finished in 2m 13s: Pass: 0%/1 | Total: 2m 13s | Avg: 2m 13s | Max: 2m 13s
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 1)
# | Runner |
---|---|
1 | linux-amd64-cpu16 |
🟥 CI finished in 3m 07s: Pass: 0%/1 | Total: 3m 07s | Avg: 3m 07s | Max: 3m 07s
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 1)
# | Runner |
---|---|
1 | linux-amd64-cpu16 |
36caf34
to
b4693f0
Compare
🟥 CI finished in 1m 54s: Pass: 0%/1 | Total: 1m 54s | Avg: 1m 54s | Max: 1m 54s
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 1)
# | Runner |
---|---|
1 | linux-amd64-cpu16 |
🟥 CI finished in 1m 53s: Pass: 0%/1 | Total: 1m 53s | Avg: 1m 53s | Max: 1m 53s
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 1)
# | Runner |
---|---|
1 | linux-amd64-cpu16 |
8122258
to
90cc3e2
Compare
🟥 CI finished in 2m 00s: Pass: 0%/1 | Total: 2m 00s | Avg: 2m 00s | Max: 2m 00s
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 1)
# | Runner |
---|---|
1 | linux-amd64-cpu16 |
🟥 CI finished in 2m 33s: Pass: 0%/1 | Total: 2m 33s | Avg: 2m 33s | Max: 2m 33s
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 1)
# | Runner |
---|---|
1 | linux-amd64-cpu16 |
🟩 CI finished in 4h 14m: Pass: 100%/420 | Total: 2d 04h | Avg: 7m 29s | Max: 1h 05m | Hits: 91%/523096
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
🏃 Runner counts (total jobs: 420)
# | Runner |
---|---|
305 | linux-amd64-cpu16 |
64 | linux-amd64-gpu-v100-latest-1 |
28 | linux-arm64-cpu16 |
23 | windows-amd64-cpu16 |
@alliepiper could you update the PR description to summarize when notifications will be pushed to various slack channels? Like @bernhardmgruber I want to make sure we're starting conservatively such that we have a meaningful signal-to-noise ratio. |
@jrhemstad done |
slack_token: | ||
description: "The Slack token to use for notifications. No notifications will be sent if not provided." | ||
required: false | ||
slack_log: | ||
description: "Slack channel ID for verbose notifications." | ||
required: false | ||
slack_alert: | ||
description: "Slack channel ID for alert notifications." | ||
required: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jrhemstad The slack notifications are only sent when these are defined.
slack_token: ${{ secrets.SLACK_NOTIFIER_BOT_TOKEN }} | ||
slack_log: ${{ secrets.SLACK_CHANNEL_CI_LOG }} | ||
slack_alert: ${{ secrets.SLACK_CHANNEL_CI_ALERT }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jrhemstad We only define these here for the nightly workflow.
* Fix divide by zero in `parse-job-times.py` * Add CI slack notifications.
* Fix divide by zero in `parse-job-times.py` * Add CI slack notifications.
The notifications are just for nightlies/weeklies/etc.
There will be two slack channels -- one that only reports failures, and one that reports everything. The first is meant to be monitored by the team so we know when things break, the other is more of a log for looking at historical results, runtimes, detecting startup issues, etc.