[PagerDuty MCP] Add write actions via REST Incidents API by erikcurrin-elastic · Pull Request #283501 · elastic/kibana

erikcurrin-elastic · 2026-08-06T18:21:41Z

Summary

Adds triggerIncident, acknowledgeIncident, resolveIncident, updateIncident, listServices, addResponders, and runResponsePlay actions to the PagerDuty MCP connector, backed by the PagerDuty v2 REST API rather than MCP. Includes input schemas with bounds, encodeURIComponent on all URL path segments, paramsSerializer for array query params, and tests covering the full incident lifecycle.

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
Documentation was added for features that require explanation or tutorials
Unit or functional tests were updated or added to match the most common scenarios
If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
Flaky Test Runner was used on any tests changed
The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
Review the backport guidelines and apply applicable backport:* labels.

Testing

Adds triggerIncident, acknowledgeIncident, resolveIncident, updateIncident, listServices, addResponders, and runResponsePlay actions to the PagerDuty MCP connector, backed by the PagerDuty v2 REST API rather than MCP. Includes input schemas with bounds, encodeURIComponent on all URL path segments, paramsSerializer for array query params, and tests covering the full incident lifecycle. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…descriptions Strip "via the REST Incidents API", "MCP server", and "REST API" from action descriptions, test handler messages, and docs — users and agents don't need to know how actions are backed. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

github-actions · 2026-08-07T22:26:44Z

🔍 Preview links for changed docs

docs/reference/connectors-kibana/pagerduty-mcp-action-type.md

Removes redundant `enabled: true` that appeared twice in the `test` object, causing a TS1117 type error. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

florent-leborgne

Docs changes LGTM but wondering if the page should now provide the detail of the available actions?

…s-connectors-list.md Co-authored-by: Florent LB <florent.leborgne@elastic.co>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

florent-leborgne

Second pass on docs after the addition of the list of actions 👍

florent-leborgne · 2026-08-11T08:02:50Z

+`getUserData`
+:   Return the current PagerDuty user — the account that owns the API key. Returns id, name, email, summary, role, and teams. Use this to confirm which user the connector is authenticated as and to obtain your user ID and email for write actions that require the `from` parameter.
+
+`triggerIncident` {applies_to}`serverless:` {applies_to}`stack: ga 9.6+`


👍 thanks @erikcurrin-elastic

erikcurrin-elastic · 2026-08-11T14:14:40Z

/ci

Co-authored-by: Florent LB <florent.leborgne@elastic.co>

github-actions

Reviewed the new PagerDuty write actions. Implementation is solid — path segments are encodeURIComponent-encoded, auth flows through the configured axios instance, and validation refinements are well tested. One minor input-validation gap noted inline.

Generated by Claude Reviewer for #283501 · 151.4 AIC · ⌖ 16.2 AIC · ⊞ 4.6K

kibanamachine · 2026-08-11T17:44:27Z

Saved Objects CI check failed

1 issue(s) across 1 type(s).

General

[generic] Feature 'Connector: PagerDuty' has already been registered with another license type. (current: gold, new: enterprise) (docs)

Run locally

node scripts/check_saved_objects --baseline 02f294b83b35af67973dcc8844af9d864f1c6638

See the Saved Objects troubleshooting guide and the model versions documentation for details.

… registration conflict The classic PagerDuty connector registers under the feature name 'Connector: PagerDuty' (gold). Renaming the spec-based connector's displayName to plain 'PagerDuty' produced the same feature key at enterprise level, causing a startup crash. Reverting to 'PagerDuty (MCP)' restores a unique feature name. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

github-actions

Reviewed the PagerDuty REST write actions. Implementation is solid — path segments are consistently URL-encoded, inputs are bounded, payload shapes match the v2 API, and tests cover the incident lifecycle. One minor validation inconsistency noted inline.

Generated by Claude Reviewer for #283501 · 179.4 AIC · ⌖ 40.7 AIC · ⊞ 4.6K

erikcurrin-elastic · 2026-08-13T23:24:49Z

/ci

tara-elastic

some minor issues flagged by the bot but otherwise looks ok to me

dennis-tismenko

Code changes LGTM, but I did want to confirm about the response_plays endpoint since I can't find any live documentation about it.

kibanamachine · 2026-08-14T19:54:13Z

💛 Build succeeded, but was flaky

Buildkite Build
Commit: 497c4a2
Build duration: 59 mins

Failed CI Steps

Jest Tests #1

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id	before	after	diff
`shared-packages`	4.3MB	4.3MB	+11.6KB

Unknown metric groups

total optimizer output size

id	before	after	diff
`all`	62.0MB	62.0MB	+11.6KB

Test Failures

[job] [logs] Jest Tests #1 / ActionPolicyFormPage create mode submits create payload on save

History

💛 Build #485681 was flaky df669f6
💛 Build #485513 was flaky df669f6
💔 Build #484303 failed 780976d
💔 Build #484120 failed 2497a69
💔 Build #483945 failed 87a2302
💔 Build #483543 failed 34732ff

) ## Summary Adds triggerIncident, acknowledgeIncident, resolveIncident, updateIncident, listServices, addResponders, and runResponsePlay actions to the PagerDuty MCP connector, backed by the PagerDuty v2 REST API rather than MCP. Includes input schemas with bounds, encodeURIComponent on all URL path segments, paramsSerializer for array query params, and tests covering the full incident lifecycle. ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [ ] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md) - [x] [Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html) was added for features that require explanation or tutorials - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [ ] If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the [docker list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker) - [ ] This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The `release_note:breaking` label should be applied in these situations. - [ ] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/docs/extend/kibana/contributing/workflow/how-we-use-github#release-notes) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. ### Testing <img width="696" height="233" alt="image" src="https://github.com/user-attachments/assets/15dc5fdb-8a0a-4f16-b83f-3d2026c0ca19" /> --------- Co-authored-by: Florent LB <florent.leborgne@elastic.co>

erikcurrin-elastic and others added 3 commits August 6, 2026 14:19

Change the name to remove MCP

e3ab5a6

erikcurrin-elastic added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:agent-builder labels Aug 7, 2026

erikcurrin-elastic marked this pull request as ready for review August 7, 2026 22:11

erikcurrin-elastic requested review from a team as code owners August 7, 2026 22:11

Merge branch 'main' into pagerduty-incident-write

4f1b7cb

botelastic Bot added the connectors-v2 label Aug 7, 2026

[PagerDuty] Fix duplicate enabled property in test handler

b3b4897

Removes redundant `enabled: true` that appeared twice in the `test` object, causing a TS1117 type error. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

florent-leborgne approved these changes Aug 10, 2026

View reviewed changes

Comment thread docs/reference/connectors-kibana/_snippets/data-context-sources-connectors-list.md Outdated

Comment thread docs/reference/connectors-kibana/pagerduty-mcp-action-type.md

erikcurrin-elastic and others added 4 commits August 10, 2026 13:32

Update docs/reference/connectors-kibana/_snippets/data-context-source…

f817719

…s-connectors-list.md Co-authored-by: Florent LB <florent.leborgne@elastic.co>

Merge branch 'main' into pagerduty-incident-write

26e73e0

Add actions in documentation

4841949

[PagerDuty] Add applies_to tags for new write actions in connector doc

34732ff

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

florent-leborgne approved these changes Aug 11, 2026

View reviewed changes

Update docs/reference/connectors-kibana/pagerduty-mcp-action-type.md

87a2302

Co-authored-by: Florent LB <florent.leborgne@elastic.co>

github-actions Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread src/platform/packages/shared/kbn-connector-specs/src/specs/pagerduty/types.ts

Merge branch 'main' into pagerduty-incident-write

2497a69

github-actions Bot reviewed Aug 12, 2026

View reviewed changes

Comment thread src/platform/packages/shared/kbn-connector-specs/src/specs/pagerduty/types.ts

Merge branch 'main' into pagerduty-incident-write

df669f6

tara-elastic approved these changes Aug 14, 2026

View reviewed changes

dennis-tismenko approved these changes Aug 14, 2026

View reviewed changes

Comment thread src/platform/packages/shared/kbn-connector-specs/src/specs/pagerduty/pagerduty.ts

Comment thread src/platform/packages/shared/kbn-connector-specs/src/specs/pagerduty/pagerduty.ts

Comment thread src/platform/packages/shared/kbn-connector-specs/src/specs/pagerduty/types.ts

Add offset

497c4a2

erikcurrin-elastic enabled auto-merge (squash) August 14, 2026 18:54

erikcurrin-elastic disabled auto-merge August 14, 2026 19:40

erikcurrin-elastic merged commit 0a90e74 into elastic:main Aug 14, 2026
40 checks passed

kibanamachine added the v9.6.0 label Aug 14, 2026

Uh oh!

Conversation

erikcurrin-elastic commented Aug 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Checklist

Testing

Uh oh!

github-actions Bot commented Aug 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔍 Preview links for changed docs

Uh oh!

florent-leborgne left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

florent-leborgne left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

florent-leborgne Aug 11, 2026

Choose a reason for hiding this comment

Uh oh!

erikcurrin-elastic commented Aug 11, 2026

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

kibanamachine commented Aug 11, 2026

Saved Objects CI check failed

General

Run locally

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

erikcurrin-elastic commented Aug 13, 2026

Uh oh!

tara-elastic left a comment

Choose a reason for hiding this comment

Uh oh!

dennis-tismenko left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kibanamachine commented Aug 14, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Page load bundle

total optimizer output size

Test Failures

History

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

erikcurrin-elastic commented Aug 6, 2026 •

edited

Loading

github-actions Bot commented Aug 7, 2026 •

edited

Loading