Add alertmanager config #169
No reviewers
Labels
No labels
Bug
Docs
Feature request
Good for newcomers
Infra
Privacy
Refactoring or migration
Security
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: pub-solar/infra#169
Loading…
Reference in a new issue
No description provided.
Delete branch "pinpox/infra:main"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
There are a few TODO's open, I marked them with comments accordingly so they can be easily grepped
e504392cf8
toa66c6ada59
@ -68,0 +80,4 @@
alertmanager = {
enable = true;
# port = 9093; # Default
webExternalUrl = "https://alerts.pub.solar"; # TODO use a proper url?
You're missing the nginx config in front of the alertmanager
fixed by
d58209ef93
teutat3s referenced this pull request2024-04-27 13:52:42 +00:00
Thanks a lot for this contribution! Would you be fine with us taking this to a branch and adding our fixes on top?
@ -40,0 +41,4 @@
logFormat = lib.mkForce ''
output discard
'';
extraConfig = ''
Here we'd want to use the
bind
directive to only listen on the WireGuard IP, exposing alertmanager only internally.and then
To have valid Let's Encrypt certificates, we'd need to configure DNS challenge, but that could be a future task IMO.
Sure feel free to reuse as you wish. I might be able to join tomorrow if you are still hakking on it
Closing in favor of #189
Pull request closed