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

Substrate alerts rules update #10642

Merged
merged 3 commits into from
Feb 2, 2022

Conversation

PierreBesson
Copy link
Contributor

Following #9543, metrics are now prefixed with polkadot_ instead of substrate.

This PR update grafana dashboards and prometheus alert rules accordingly.

* match the `substrate_` metrics prefix in alerts instead of `polkadot_`, following changes in paritytech#9543
* remove the filtering on polkadot|kusama domain for NumberOfFileDescriptorsHigh alert
* match the `substrate_` metrics prefix instead of `polkadot_` in dashboards, following changes in paritytech#9543
@PierreBesson PierreBesson requested a review from a team as a code owner January 12, 2022 11:01
@bkchr bkchr added A0-please_review Pull request needs code review. 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 Jan 12, 2022
@bkchr bkchr merged commit e570d2e into paritytech:master Feb 2, 2022
agryaznov pushed a commit to agryaznov/substrate that referenced this pull request Feb 4, 2022
* .maintain/monitoring: Update substrate prometheus alert rules

* match the `substrate_` metrics prefix in alerts instead of `polkadot_`, following changes in paritytech#9543
* remove the filtering on polkadot|kusama domain for NumberOfFileDescriptorsHigh alert

* .maintain/monitoring: Update substrate Grafana dashboards

* match the `substrate_` metrics prefix instead of `polkadot_` in dashboards, following changes in paritytech#9543

* .maintain/monitoring:  make the NumberOfFileDescriptorsHigh alert only apply for metrics tagged with 'chain'
Wizdave97 pushed a commit to ComposableFi/substrate that referenced this pull request Feb 4, 2022
* .maintain/monitoring: Update substrate prometheus alert rules

* match the `substrate_` metrics prefix in alerts instead of `polkadot_`, following changes in paritytech#9543
* remove the filtering on polkadot|kusama domain for NumberOfFileDescriptorsHigh alert

* .maintain/monitoring: Update substrate Grafana dashboards

* match the `substrate_` metrics prefix instead of `polkadot_` in dashboards, following changes in paritytech#9543

* .maintain/monitoring:  make the NumberOfFileDescriptorsHigh alert only apply for metrics tagged with 'chain'
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
* .maintain/monitoring: Update substrate prometheus alert rules

* match the `substrate_` metrics prefix in alerts instead of `polkadot_`, following changes in paritytech#9543
* remove the filtering on polkadot|kusama domain for NumberOfFileDescriptorsHigh alert

* .maintain/monitoring: Update substrate Grafana dashboards

* match the `substrate_` metrics prefix instead of `polkadot_` in dashboards, following changes in paritytech#9543

* .maintain/monitoring:  make the NumberOfFileDescriptorsHigh alert only apply for metrics tagged with 'chain'
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* .maintain/monitoring: Update substrate prometheus alert rules

* match the `substrate_` metrics prefix in alerts instead of `polkadot_`, following changes in paritytech#9543
* remove the filtering on polkadot|kusama domain for NumberOfFileDescriptorsHigh alert

* .maintain/monitoring: Update substrate Grafana dashboards

* match the `substrate_` metrics prefix instead of `polkadot_` in dashboards, following changes in paritytech#9543

* .maintain/monitoring:  make the NumberOfFileDescriptorsHigh alert only apply for metrics tagged with 'chain'
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. 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.

2 participants