alertmanager-github-receiver icon indicating copy to clipboard operation
alertmanager-github-receiver copied to clipboard

Feature/template improvements

Open aorfanos opened this issue 4 years ago β€’ 2 comments

Allows use of a custom alert message template file.

Example invocation would be: -alert-template-file=alertfile.md .

I have commented out Test_formatIssueBody because I'm still working my way to make it work.


This change is Reviewable

aorfanos avatar Sep 09 '21 23:09 aorfanos

Pull Request Test Coverage Report for Build 171

  • 17 of 19 (89.47%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.5%) to 99.516%

Changes Missing Coverage Covered Lines Changed/Added Lines %
alerts/handler.go 7 9 77.78%
<!-- Total: 17 19
Totals Coverage Status
Change from base Build 162: -0.5%
Covered Lines: 411
Relevant Lines: 413

πŸ’› - Coveralls

coveralls avatar Sep 09 '21 23:09 coveralls

We just started looking at using this project, this feature is a must have though - it'd be great to see this merged and released so we don't have to build and maintain a fork.

Limess avatar Jun 01 '22 11:06 Limess

Thank you for this change @aorfanos - I'm taking a look now. I'm not sure how we missed it this long.

stephen-soltesz avatar Dec 22 '22 20:12 stephen-soltesz

I've forgotten about it as well - better late than never πŸ˜„

aorfanos avatar Dec 22 '22 20:12 aorfanos

@aorfanos thank you for this change - the logic here was the basis of https://github.com/m-lab/alertmanager-github-receiver/pull/59 now merged (your contribution is acknowledged in the commit). We'll prepare a new release this week. I'll close this PR since the functionality is now merged to the main branch.

stephen-soltesz avatar Jan 04 '23 15:01 stephen-soltesz