Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Add substrate-rpc-subscription to exceptions in alert #9172

Merged
1 commit merged into from
Jun 22, 2021

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented Jun 22, 2021

This task is obviously not continuous, so it's not worth alerting about.

Normally the alert rule is loose enough that this kind of task doesn't trigger the alert, but we apparently have some validators that trigger it, most likely because of a script querying information from the node.

@tomaka tomaka added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jun 22, 2021
@tomaka tomaka requested a review from a team as a code owner June 22, 2021 11:41
@TriplEight TriplEight self-requested a review June 22, 2021 14:31
@tomaka
Copy link
Contributor Author

tomaka commented Jun 22, 2021

bot merge

@ghost
Copy link

ghost commented Jun 22, 2021

Trying merge.

@ghost ghost merged commit 088464e into paritytech:master Jun 22, 2021
@tomaka tomaka deleted the fix-alertt branch June 22, 2021 16:12
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants