Skip to content

Conversation

@josemallma-dd
Copy link

What does this PR do?

Updates the Prometheus integration README to document the V2 webhook endpoint for Alertmanager and improves the overall setup documentation with tabs, enhanced explanations, and better formatting.

Motivation

The Prometheus integration now supports a V2 webhook endpoint (event-management-intake.datadoghq.com/api/v2/events/webhook) that offers additional features compared to V1, including:

  • Support for multiple query parameters (e.g., oncall_team for Datadog On-Call integration)
  • Better integration with Datadog's event management system

This PR documents both webhook versions using tabs, with V2 as the preferred option. It also enhances the setup instructions by:

  • Adding information about using Alertmanager matchers to route alerts to different receivers
  • Including helpful tips about send_resolved configuration and its importance for On-Call page resolution
  • Highlighting relevant configuration lines in code blocks
  • Adding informational notes about group_by behavior
  • Providing a link to the Prometheus Alertmanager section from the Events section for easier navigation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

@josemallma-dd josemallma-dd added the qa/skip-qa Automatically skip this PR for the next QA label Dec 4, 2025
@josemallma-dd josemallma-dd force-pushed the jmallma/prometheus-v2-webhook branch from d7ac751 to f064cb0 Compare December 4, 2025 17:41
@cswatt cswatt self-assigned this Dec 4, 2025
@cswatt
Copy link
Contributor

cswatt commented Dec 4, 2025

Thanks for this PR! You're using the correct syntax for tabs in integration docs, but it's unexpectedly causing some bugs in rendering the code with the appropriate highlighted lines and indentations:
Screenshot 2025-12-04 at 2 38 38 PM

I've created DOCS-12839 to help fix this issue and for a general editorial review from the Docs team.

@cswatt cswatt added the editorial review Waiting on a more in-depth review from a docs team editor label Dec 4, 2025
@cswatt cswatt removed their assignment Dec 4, 2025
drichards-87
drichards-87 previously approved these changes Dec 8, 2025
Copy link
Contributor

@drichards-87 drichards-87 left a comment

Choose a reason for hiding this comment

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

Looks like the issue with the tabs was fixed. I've left some additional (minor) feedback on the PR.

Co-authored-by: DeForest Richards <[email protected]>
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed drichards-87’s stale review December 10, 2025 11:47

Review from drichards-87 is dismissed. Related teams and files:

  • documentation
    • prometheus/README.md
@josemallma-dd
Copy link
Author

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants