Skip to content

Open and resolve PagerDuty incidents - #81

Merged
tomschlick merged 1 commit into
masterfrom
feature/68550ee6
Sep 23, 2026
Merged

tomschlick merged 1 commit into
masterfrom
feature/68550ee6

Conversation

@tomschlick

Copy link
Copy Markdown
Member

Summary

  • Send Events API v2 trigger events when a monitor goes down and resolve events with the same dedup key when it recovers, so PagerDuty opens and closes the incident.
  • Reject integration keys that are not 32 hexadecimal characters, strip credentials from the affected system, and include the failure details plus a link back to the monitor.
  • Channel tests trigger and then resolve, and throttled or 5xx responses are retried.

Test plan

  • Save a PagerDuty channel with a real 32-character Events API v2 integration key, and confirm a short or non-hex key is rejected.
  • Send a test notification and confirm the incident opens and then resolves.
  • Fail a monitor past its threshold and confirm one incident opens; recover it and confirm that incident resolves.
  • Confirm a database monitor alert does not include the connection password.

@tomschlick
tomschlick merged commit 132950d into master Sep 23, 2026
2 checks passed
@tomschlick
tomschlick deleted the feature/68550ee6 branch September 23, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant