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

config.sample.yaml: alertmanager: update example (text|html)_template #368

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

herbetom
Copy link

@herbetom herbetom commented Jan 3, 2022

I've updated the (text|html)_template within in the sample config file and made the following changes:

  • better readability: This helps other's since it allows custom modifications more easily.
  • introduced the severity info: it's blue and probably quite common (hoewer other's might use none but that should be a easy fix for them)
  • dropped the FIRING - prefix: With limited space in Pop-ups i think the first few characters are too important as to waste them with kinda redundant information. [RESOLVED] and everthing else should be enough to distinguish between a firing alert and a resolved one.

Since it's only an example config i don't really see an issue with changing it. As long as others agree that my version is better. ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant