-
Notifications
You must be signed in to change notification settings - Fork 89
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
alerting: Slack #lamprey-builds
Channel
#1524
Conversation
#lamprey-builds
Channel#lamprey-builds
Channel
#lamprey-builds
Channel#lamprey-builds
Channel
#lamprey-builds
Channel#lamprey-builds
Channel
@@ -54,7 +54,7 @@ jobs: | |||
- install_linter | |||
- run_tests | |||
# Change this to run_e2e_tests once we have stable algod binaries containing delta APIs | |||
- run_e2e_tests_nightly | |||
# - run_e2e_tests_nightly |
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.
Chicken and egg problem here. I can't re-route the slack messaging until I fix the test. But the test is broken and getting slacked to the wrong channel. I've added a TODO to #1521 :
Codecov Report
@@ Coverage Diff @@
## develop #1524 +/- ##
========================================
Coverage 70.35% 70.35%
========================================
Files 40 40
Lines 8135 8136 +1
========================================
+ Hits 5723 5724 +1
Misses 1982 1982
Partials 430 430
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Summary
Indexer is owned by Team Lamprey. Therefore, alerts of indexer build failures ought to be routed to a Lamprey-specific slack channel instead of
#dev-alerts
.Issues
#1521
Test Plan
CI
You can see that the configuration works by looking at recent notifications inside of
#lamprey-builds
.