Skip to content

[DBM][Postgres] Adds logical database breakdown for connection metrics#22999

Open
aldrickdev wants to merge 16 commits intomasterfrom
aldrick.castro/SDBM-2452
Open

[DBM][Postgres] Adds logical database breakdown for connection metrics#22999
aldrickdev wants to merge 16 commits intomasterfrom
aldrick.castro/SDBM-2452

Conversation

@aldrickdev
Copy link
Contributor

@aldrickdev aldrickdev commented Mar 20, 2026

What does this PR do?

This will add 2 new connection metrics that have a logical database breakdown.

  • postgresql.database_connections
  • postgresql.percent_database_usage_connections

Example of both metrics below:

image

Motivation

Customer Request -> SDBM-2452

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

…ion metrics

Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
@aldrickdev aldrickdev changed the title Added support for logical database break down on the postgres connect… [DBM][Postgres] Adds logical database breakdown for connection metrics Mar 20, 2026
@datadog-prod-us1-5
Copy link

datadog-prod-us1-5 bot commented Mar 20, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: b7998b2 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
Signed-off-by: aldrick.castro <aldrick.castro@datadoghq.com>
@aldrickdev aldrickdev force-pushed the aldrick.castro/SDBM-2452 branch from d82cf60 to f426d52 Compare March 20, 2026 19:29
@aldrickdev aldrickdev added the qa/skip-qa Automatically skip this PR for the next QA label Mar 20, 2026
@aldrickdev aldrickdev marked this pull request as ready for review March 20, 2026 19:59
@aldrickdev aldrickdev requested review from a team as code owners March 20, 2026 19:59
@codecov
Copy link

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.34%. Comparing base (31eda08) to head (b7998b2).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f426d528b3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@github-actions
Copy link
Contributor

github-actions bot commented Mar 22, 2026

⚠️ The qa/skip-qa label has been added with shippable changes

The following files, which will be shipped with the agent, were modified in this PR and
the qa/skip-qa label has been added.

You can ignore this if you are sure the changes in this PR do not require QA. Otherwise, consider removing the label.

List of modified files that will be shipped with the agent
postgres/changelog.d/22999.added
postgres/datadog_checks/postgres/postgres.py
postgres/datadog_checks/postgres/util.py

@aldrickdev aldrickdev marked this pull request as draft March 23, 2026 02:18
@aldrickdev aldrickdev marked this pull request as ready for review March 23, 2026 19:28
@aldrickdev aldrickdev added the assets/no-deploy Prevents APW from deploying this PR in staging label Mar 24, 2026
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.

2 participants