Skip to content

SIGNL4 Alert Notification#20911

Merged
ilyam8 merged 8 commits intonetdata:masterfrom
rons4:master
Sep 2, 2025
Merged

SIGNL4 Alert Notification#20911
ilyam8 merged 8 commits intonetdata:masterfrom
rons4:master

Conversation

@rons4
Copy link
Contributor

@rons4 rons4 commented Sep 2, 2025

Summary

This PR is to add SIGNL4 mobile alerting as an alert notification integration to Netdata.

I followed the recommendations for contributing alert notifications as mentioned here:
https://github.com/netdata/.github/blob/main/CONTRIBUTING.md

Test Plan

I tested the notifications as described here under "Test Notification":
https://learn.netdata.cloud/docs/alerts-&-notifications/notifications/agent-dispatched-notifications/discord

This was helpful for debugging and it runs fine now (OK states).

Additional Information

SIGNL4 (https://www.signl4.com/) is a mobile alerting service integrating with various tools and Netdata surly belongs here: https://docs.signl4.com/integrations/integrations.html

The idea is to alert responsible users or teams via app push, SMS, or voice call if a critical incident is detected by Netdata. SIGNL4 also includes escalations, on-call scheduling and in-alert collaboration.

For users: How does this change affect me?

@CLAassistant
Copy link

CLAassistant commented Sep 2, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ilyam8
Copy link
Member

ilyam8 commented Sep 2, 2025

Hi, @rons4. The .md files will be created from metadata. You don't need to add them.

@github-actions github-actions bot removed the area/docs label Sep 2, 2025
@rons4
Copy link
Contributor Author

rons4 commented Sep 2, 2025

Thanks, @ilyam8. I saw it but since the .md files were there for the other notification channels I thought I just create the. I have removed them now.

@ilyam8 ilyam8 requested a review from Copilot September 2, 2025 14:46

This comment was marked as outdated.

@ilyam8 ilyam8 requested a review from Copilot September 2, 2025 18:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds SIGNL4 mobile alerting service as a new notification integration to Netdata. SIGNL4 provides critical alerting via app push, SMS, voice calls, and email with features like tracking, escalation, and on-call scheduling.

  • Adds SIGNL4 as a new alert notification method alongside existing integrations
  • Implements webhook-based notification delivery to SIGNL4 service
  • Provides configuration options for enabling and customizing SIGNL4 notifications

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/health/notifications/signl4/metadata.yaml Configuration metadata and documentation for SIGNL4 integration
src/health/notifications/health_alarm_notify.conf Configuration file updates to include SIGNL4 settings and role mappings
src/health/notifications/alarm-notify.sh.in Core implementation of SIGNL4 notification sender function

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

ilyam8 and others added 2 commits September 2, 2025 21:30
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ilyam8 ilyam8 merged commit c321fbe into netdata:master Sep 2, 2025
105 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/health area/metadata Integrations metadata

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants