Skip to content

Adding integrationKey to alert docs

Administrator requested to merge emchap-patch-1 into main

Created by: emchap

A customer reported an issue configuring PagerDuty alerts; after looking the code they resolved it by including both routingKey and integrationKey in their site config, eg:

"observability.alerts": [
  {
    "level": "critical",
    "notifier": {
      "type": "pagerduty",
      "routingKey": "XXXXXXXX"
      "integrationKey": "XXXXXXXX"
    }
  }

What is unclear to us is whether routingKey actually does anything—should the user actually just pass integrationKey? I see both referenced at https://sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/schema/site.schema.json?utm_source=share#L1158

So, assigning for review to make sure that the alerts really do need both!

Merge request reports