Skip to content

feat: disable default database collector for postgres_exporter #1709

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 17, 2025

Conversation

darora
Copy link
Contributor

@darora darora commented Jul 17, 2025

The database collector emits a metric for the size of each database by
default. We emit a metric that represents the sum of all database,
which suffices for our purposes. This change removes the redundant
per-database metric.

@darora darora requested review from a team as code owners July 17, 2025 10:29
Copy link
Contributor

@tgallacher tgallacher left a comment

Choose a reason for hiding this comment

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

Will this impact v2 negatively?

Assuming the "database collector" is not a v3 specific component?

The database collector emits a metric for the size of each database by
default. We emit a metric that represents the sum of all database,
which suffices for our purposes. This change removes the redundant
per-database metric.

In order to avoid a change in the AMI metrics, the change is only made
for the QEMU artifact.
@darora darora force-pushed the da/upgrade-image branch from e649029 to 319e3b2 Compare July 17, 2025 11:06
@darora
Copy link
Contributor Author

darora commented Jul 17, 2025

I'm pretty sure this was inadvertently left out in #788 ("all the collectors are disabled with the additions to the CLI switches in the systemd unit file").
While it does represent a change, the metrics returned do not represent a stable API, and as such, imo it's fine to disable it.

However, out of an abundance of caution, I've updated the PR to only disable it in the QEMU artifact.

@darora darora merged commit 5547a4f into develop Jul 17, 2025
13 of 14 checks passed
@darora darora deleted the da/upgrade-image branch July 17, 2025 12:37
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.

3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy