Skip to content

Conversation

@sbadia
Copy link
Contributor

@sbadia sbadia commented Feb 2, 2018

Hi here,

This is just a trivial PR in order to update alerting rules from the 1.x to 2.x format.

This PR is maybe linked to prometheus/alertmanager#1062 and #857 (extract alertmanager documentation).

}
annotations:
description: '{{ $labels.instance }} of job {{ $labels.job }} has been down for more than 5 minutes.'
summary: Instance {{ $labels.instance }} down
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd keep the single quotes here for clarity

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, you're right!

@sbadia sbadia force-pushed the rules_prometheus_2 branch from e3c5aef to aacd432 Compare February 2, 2018 14:51
@brian-brazil brian-brazil merged commit 0066de1 into prometheus:master Feb 2, 2018
@brian-brazil
Copy link
Contributor

Thanks!

@sbadia sbadia deleted the rules_prometheus_2 branch February 2, 2018 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants