-
Notifications
You must be signed in to change notification settings - Fork 11
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
record-tester: Test stream health (#118)
* cmd: Create healthtester cmd based on recordtester Basically copied the whole command and then went through it rewriting some code as a way to understand the logic. Also copied some helpers from analyzer code (contextUntilSig) * healthtester: Create helper for running multiple tests * recordtester: Improve PagerDuty formatting/links * recordtester: Allow cancelling in between tests * testers: Create Stream2Options * testers: Create Finite iface and receive on streamer2 * cmd: Remove healthchecker new command Can check any code on history if necessary * testers: Dummy stream health tester for arch review * DEV-REVERT: Use local livepeer-data lib * testers: Simplify additional tests glue * testers: Implement health tester for real * cmd/recordtester: Plug everything together * Revert "DEV-REVERT: Use local livepeer-data lib" This reverts commit b6cb78c. * go.mod: Use beta (unmerged) version of livepeer-data * cmd/recordtester: Simplify analyzer URL flag passing * testers: Improve logs on stream health tester * testers: Fix unhealthy timer * testers: Log all errors after half the wait time * testers: Add insane log * record-tester: moar insane logs * recordtester: Ignore commas in env vars * Revert "record-tester: moar insane logs" This reverts commit f516d3f. * testers: Improve formatting of stream health errors Aggregate regions for each error message and try to make it clearer that we are testing stream health global API, not multistream. * testers: A couple more formatting improvements * record-tester: Improve pagerduty API usage Use a dedup key so we dont spam the oncall engineer and also make sure to resolve incidents automatically once the tests start working. * testers: Improve formatting of regions list Just say all regions when it's all regions * go.mod: Update analyzer client version * recordtester: Add option to only send lopri PagerDuty alerts * recordtester: Include pagerduty component in alert Will differentiate messages on #incident-response based on the component, which we'll configure to be GCP/Datapacket. * health-tester: Make sure to compare health with same time Otherwise absolutely every error message is going to be slightly different. * record-tester: Remove all fatal logs from m3utester2 * record-tester: Use different dedup key when lopri Otherwise it could get confusing that the incident with the [LOPRI] prefix sends a high urgency alert. * record-tester: Make sure to finish on fatal errors * record-tester: Ensure a global test timeout * record-tester: Looser global timeout and proper log * record-tester: Accumulate errors from finite testers * record-tester: Bring back early test of crt.ctx.Err Cleaner discord msgs * stream-tester: Add hostname to start&end logs
- Loading branch information
Showing
11 changed files
with
458 additions
and
178 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.