Skip to content

amtool: command to test receivers #4807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
SoloJacobs wants to merge 18 commits into
base: main
Choose a base branch
from
Open

amtool: command to test receivers #4807

SoloJacobs wants to merge 18 commits into from

Conversation

@SoloJacobs
Copy link
Contributor

@SoloJacobs SoloJacobs commented Dec 12, 2025 •
edited
Loading

This PR is an update of #3491 that's updated to work against main, which in turn is an update of #3139 .
So far, I have held off on making some changes myself, except for what was necessary to get CI green again. I also have a small laundry list of things I would fix here, but I would like a second pair of eyes on this request first.

This PR is an extension of @OktarianTB's 9-month old PR #3139 that's updated to work against the latest code in main, to hopefully increase its likelihood of being merged.

Credit for this work goes to @OktarianTB - I believe I've marked the commit authorship correctly, please let me know if I made any mistakes here.

This PR aims to make progress towards #2845 by adding a new command to amtool to send test notifications to every receiver in an alertmanager config file. The command takes in an alertmanager config file and optionally an alert file to mock values for labels and annotations. This is the MVP for the command, potentially a few other options can be added later if deemed of use. This largely re-uses the equivalent implementation in grafana/grafana.

I have only tested this with Slack so far.

OktarianTB and others added 18 commits December 12, 2025 08:44
@OktarianTB @SoloJacobs
Test receivers command setup
90e4220 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
First iteration to test receivers
84e277e 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
Improvements to test receivers
1614b04 
Co-authored-by: Solomon Jacobs <[email protected]>
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
Refactor code
a047f44 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
Add alert.file option to mock alert
27aa009 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
Add tests for test receivers command
de85f29 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
update mock url
2b81b41 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@OktarianTB @SoloJacobs
fix linting
9bc100c 
Signed-off-by: Oktarian TB <[email protected]>
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@alexweav @SoloJacobs
Fix updated FromGlobs signature
af0bbca 
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@alexweav @SoloJacobs
Update to latest config definition, use latest package path for kingpin
ec90837 
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@alexweav @SoloJacobs
Lint imports
8f0a520 
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@alexweav @SoloJacobs
Lint again
5e11f46 
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@alexweav @SoloJacobs
One more time...
97ed856 
Signed-off-by: Alex Weaver <[email protected]>
Signed-off-by: Solomon Jacobs <[email protected]>
@alexweav @SoloJacobs
Add two missing receiver types
27106b8 
Signed-off-by: Solomon Jacobs <[email protected]>
@SoloJacobs
fix compilation
9f63a48 
* Update logging
* Add argument to notify.NewIntegration

Signed-off-by: Solomon Jacobs <[email protected]>
@SoloJacobs
golangci-lint run --enable modernize --fix
139c67e 
Signed-off-by: Solomon Jacobs <[email protected]>
@SoloJacobs
go mod tidy
715f078 
Signed-off-by: Solomon Jacobs <[email protected]>
@SoloJacobs
repair yamllint issues
d32d742 
Signed-off-by: Solomon Jacobs <[email protected]>
@SoloJacobs SoloJacobs changed the title (WIP) amtool: command to test receivers amtool: command to test receivers Dec 12, 2025
@SoloJacobs SoloJacobs marked this pull request as ready for review December 12, 2025 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SoloJacobs @OktarianTB @alexweav