-
Notifications
You must be signed in to change notification settings - Fork 769
Convert pg_stat_database to new collector model #685
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
Convert pg_stat_database to new collector model #685
Conversation
Signed-off-by: Joe Adams <[email protected]>
Ping @SuperQ |
CC @Sticksman |
BREAKING CHANGES: Please note, the following features are deprecated and may be removed in a future release: - `auto-discover-databases` - `extend.query-path` - `constantLabels` - `exclude-databases` - `include-databases` This exporter is meant to monitor PostgresSQL servers, not the user data/databases. If you need a generic SQL report exporter https://github.com./burningalchemist/sql_exporter is recommended. * [CHANGE] Adjust log level for collector startup #784 * [CHANGE] Move queries from queries.yaml to collectors #801 * [CHANGE] Deprecate extend queries feature #811 * [CHANGE] Deprecate additional database features #815 * [CHANGE] Convert pg_stat_database to new collector #685 * [ENHANCEMENT] Supports alternate postgres:// prefix in URLs #787 * [BUGFIX] Fix pg_setting different help values #771 * [BUGFIX] Fix column type for pg_replication_slots #777 * [BUGFIX] Fix pg_stat_database collector #809 Signed-off-by: SuperQ <[email protected]>
It seems that all "pg_stat_database_tup_*" (as shown here) are not exported anymore in v0.13.0 while they are exported in v0.12.1 (tested with postgres14). Sorry, I just found this post reporting the same issue. |
That's odd, they should be getting returned: https://github.com./prometheus-community/postgres_exporter/blob/master/collector/pg_stat_database.go#L389 I also see it in the dump you provided:
|
I guess, I could have expressed that better. The "as shown here" link was meant to show the expected output. I attached two outputs I took from my tests using quay.io/prometheuscommunity/postgres-exporter:v0.13.0 both with postgres 14. metrics_13_0.txt One example is "pg_stat_database_tup_returned", which exists in the 12.1 but not in 13.0. |
Can you check 0.13.2? I don't expect there to be a difference, but just so we're sure. |
I ran version Here's everything I'm getting for
|
Ok, interesting. We're only getting metrics from this:
This means that pg_stat_database metrics from this collector aren't being emitted at all. |
It feels like something super dumb, like somehow a label name and a namespace are colliding |
I have a feeling that it has something to do with a NULL column. Can someone try with log level turned up to debug and post the logs? |
Here's what I'm getting with the log level turned up to
|
Hm, I would have expected it to be registered at least. |
The debug logs do show the problem. I don't know why your logs don't include it. I spun up postgres locally with the exporter log level set to debug and I have these logs:
The first skip is expected because the datname is NULL. The labels won't come out right with that field NULL. This was caused by 2 PRs together. First, pg_stat_database collector was changed to always use NULL-able data types. The stats_reset field is expected to be NULL if the stats have never been reset. See the change here: https://github.com./prometheus-community/postgres_exporter/pull/823/files#diff-19ef1ff88df9efd6fea2fe321a720b0a44edc1daab43571ac07460666efc3697L412 In a later PR the metric line was skipped entirely if the field is NULL. This is the change that would have stopped reporting the metrics entirely. In this scenario, only the labels being NULL should skip the whole row. Any other fields being NULL (should only be stats_reset) should only skip that single metric. |
Just ran a docker container with postgres 14 and a built v0.13.2. I'm seeing stat_database metrics get emitted: # TYPE pg_stat_database_tup_deleted counter
pg_stat_database_tup_deleted{datid="0",datname="unknown"} 0
pg_stat_database_tup_deleted{datid="1",datname="template1"} 0
pg_stat_database_tup_deleted{datid="13779",datname="template0"} 0
pg_stat_database_tup_deleted{datid="13780",datname="postgres"} 0
pg_stat_database_tup_deleted{datid="16384",datname="circle_test"} 0
# HELP pg_stat_database_tup_fetched Number of rows fetched by queries in this database
# TYPE pg_stat_database_tup_fetched counter
pg_stat_database_tup_fetched{datid="0",datname="unknown"} 17
pg_stat_database_tup_fetched{datid="1",datname="template1"} 0
pg_stat_database_tup_fetched{datid="13779",datname="template0"} 0
pg_stat_database_tup_fetched{datid="13780",datname="postgres"} 816
pg_stat_database_tup_fetched{datid="16384",datname="circle_test"} 1198
# HELP pg_stat_database_tup_inserted Number of rows inserted by queries in this database
# TYPE pg_stat_database_tup_inserted counter
pg_stat_database_tup_inserted{datid="0",datname="unknown"} 1
pg_stat_database_tup_inserted{datid="1",datname="template1"} 0
pg_stat_database_tup_inserted{datid="13779",datname="template0"} 0
pg_stat_database_tup_inserted{datid="13780",datname="postgres"} 0
pg_stat_database_tup_inserted{datid="16384",datname="circle_test"} 0
# HELP pg_stat_database_tup_returned Number of rows returned by queries in this database
# TYPE pg_stat_database_tup_returned counter
pg_stat_database_tup_returned{datid="0",datname="unknown"} 32
pg_stat_database_tup_returned{datid="1",datname="template1"} 0
pg_stat_database_tup_returned{datid="13779",datname="template0"} 0
pg_stat_database_tup_returned{datid="13780",datname="postgres"} 897
pg_stat_database_tup_returned{datid="16384",datname="circle_test"} 1684
# HELP pg_stat_database_tup_updated Number of rows updated by queries in this database
# TYPE pg_stat_database_tup_updated counter
pg_stat_database_tup_updated{datid="0",datname="unknown"} 0
pg_stat_database_tup_updated{datid="1",datname="template1"} 0
pg_stat_database_tup_updated{datid="13779",datname="template0"} 0
pg_stat_database_tup_updated{datid="13780",datname="postgres"} 0
pg_stat_database_tup_updated{datid="16384",datname="circle_test"} 0 Edit: Tested with pg15 as well, seems to be ok. |
However I do see that in the newest build we might skip some metrics due to the stats_reset column being null which we should account for. |
This is only in the latest build, not 0.13.2 which is what our @algchoo is running. I'm unclear what the issue with 0.13.2 is since it doesn't yet have the code skipping the metrics yet. |
The testing I've done has been against containers in k8s, rather than a local docker setup. Using I'm running Docker Desktop 4.20.1 on MacOS 13.0.1 with an M1 chip and below is what I'm deploying. Is there anything wrong with this, perhaps? version: "3.9"
services:
postgres:
restart: always
image: postgres:14
ports:
- "5432:5432"
environment:
- POSTGRES_USER=root
- POSTGRES_PASSWORD=password
- POSTGRES_DB=dev
exporter:
restart: always
image: quay.io/prometheuscommunity/postgres-exporter:v0.13.0
ports:
- 9187:9187
environment:
- DATA_SOURCE_NAME=postgresql://root:password@postgres:5432/dev?sslmode=disable
command: --collector.stat_statements --log.level=debug |
that all looks right. Let me run your docker-compose file and see if I can repro |
@algchoo try with |
PostgreSQL 14 with exporter version
PostgreSQL 15.3 with exporter
Unfortunately, the results are the same for me with any version of the exporter |
Did you try hitting the /metrics endpoint? Here's the docker-compose file I used (basically the same except for 0.13.2)
Here's a sample of the stats emitted
Here's a logging sample from the container
One thing about the pg_stat_database collector in v0.13.x is that so far it was created without a logger, so that might be contributing to the lack of seeing it in the logs. There's a patch in master with a logger attached (and I should make sure all the other collectors have loggers) |
@Sticksman Yes, I hit the metrics endpoint and while I see output, I don't see the expected |
I think it'd probably be a decent next step. I'm going to try blowing away all my containers and seeing if there was a cache on my end that was causing me to show the metrics. |
I've run into a rather strange situation: I've run your docker-compose file after pruning my containers and I can indeed reproduce the issue. However after going to run the two commands individually:
I'm finding no issues. Will try using the docker-compose file with the latest build to see if that changes anything. |
@algchoo I added a
This fixed the issue and began allowing stat_database elements to correctly . This leads me to believe there's a race condition where starting the exporter before the db can cause collectors not to load. I'll file an issue. |
@algchoo actually here's the error line in the logs
|
* Dashboard linting improvements for mixin Signed-off-by: Ryan J. Geyer <[email protected]> * WIP: Add prelim multi-target support - Remove multi server support from new collector package - Add http handler for multi-target support Signed-off-by: Joe Adams <[email protected]> * Add config module The config module supports adding configuration to the exporter via a config file. This supports adding authentication details in a config file so that /probe requests can specify authentication for endpoints Signed-off-by: Joe Adams <[email protected]> * cleanup and README Signed-off-by: Joe Adams <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Update cmd/postgres_exporter/main.go Signed-off-by: Joe Adams <[email protected]> * fix for exporter issue 633 fix for exporter issue 633: prometheus-community#633 "Scan error on column index 2, name \"checkpoint_write_time\": converting driver.Value type float64 (\"6.594096e+06\") to a int: invalid syntax prometheus-community#633" Signed-off-by: bravosierrasierra <[email protected]> * Fix checkpoint_sync_time value type Error: sql: Scan error on column index 3, name \"checkpoint_sync_time\": converting driver.Value type float64 (\"1.876469e+06\") to a int: invalid syntax See also: prometheus-community#633 prometheus-community#666 Signed-off-by: Nicolas Rodriguez <[email protected]> * Bump github.com./prometheus/common from 0.35.0 to 0.37.0 Bumps [github.com./prometheus/common](https://github.com./prometheus/common) from 0.35.0 to 0.37.0. - [Release notes](https://github.com./prometheus/common/releases) - [Commits](prometheus/common@v0.35.0...v0.37.0) --- updated-dependencies: - dependency-name: github.com./prometheus/common dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Correct minor typos in README.md Signed-off-by: Luckz <[email protected]> * Release 0.11.1 * [BUGFIX] Fix checkpoint_write_time value type prometheus-community#666 * [BUGFIX] Fix checkpoint_sync_time value type prometheus-community#667 Signed-off-by: SuperQ <[email protected]> * Add dsn type for handling datasources dsn is designed to replace the other uses of dsn as a string in the long term. dsn is designed to be safe to log, properly redacting passwords. The goal is eventually always parse datasource information into a dsn type object which can safely be passed around and logged without worrying about wrapping calls in a redaction function (today this function is loggableDSN(). This should solve the root issue in prometheus-community#648, prometheus-community#677, and prometheus-community#643, although the full fix will require more changes to update all code references over to use the dsn type. Signed-off-by: Joe Adams <[email protected]> * Release 0.12.0-rc.0 BREAKING CHANGES: This release changes support for multiple postgres servers to use the multi-target exporter pattern. This makes it much easier to monitor multiple PostgreSQL servers from a single exporter by passing the target via URL params. See the Multi-Target Support section of the README. * [CHANGE] Add multi-target support prometheus-community#618 * [BUGFIX] Add dsn type for handling datasources prometheus-community#678 Signed-off-by: SuperQ <[email protected]> * fix: typo Signed-off-by: Yoan Blanc <[email protected]> * Update multi-target handler to use new DSN type - Moves new dsn type to config.DSN. This will prevent circular dependencies. - Change DSN.query to be url.Values. This allows the multi-target functionality to merge values without re-parsing the query string - Change NewProbeCollector to use the new config.DSN type - Add DSN.GetConnectionString to return a string formatted for the sql driver to use during connection Signed-off-by: Joe Adams <[email protected]> * Add missing license header Signed-off-by: Joe Adams <[email protected]> * Convert pg_stat_database to new collector model Signed-off-by: Joe Adams <[email protected]> * extended /probe path metrics Signed-off-by: Ildar Valiullin <[email protected]> * Bump github.com./lib/pq from 1.10.6 to 1.10.7 Bumps [github.com./lib/pq](https://github.com./lib/pq) from 1.10.6 to 1.10.7. - [Release notes](https://github.com./lib/pq/releases) - [Commits](lib/pq@v1.10.6...v1.10.7) --- updated-dependencies: - dependency-name: github.com./lib/pq dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Capture usename and application_name for pg_stat_activity It is necessary to be able to exclude backups from long-running transaction alerts, as they are to be expected. With the current pg_stat_activity metric there is no ability to filter out specific users or application names. Resolves prometheus-community#668 Signed-off-by: cezmunsta <[email protected]> * Fixed formatting Signed-off-by: cezmunsta <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * 4kB size added for postgres with 4kB block_size Signed-off-by: Sergey Morozov <[email protected]> * Correct additional typo in README.md Signed-off-by: Luckz <[email protected]> * Set gauge to 1 when collector is successful Signed-off-by: Julien Pivotto <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * probe: clean-up database connection after probe to prevent connection leak Signed-off-by: Kurtis Bass <[email protected]> * Set gauge to 1 when collector is successful Signed-off-by: Julien Pivotto <[email protected]> Signed-off-by: Khiem Doan <[email protected]> * Add postgres 15 for CI test Signed-off-by: Khiem Doan <[email protected]> * Add postgres 15 for CI test Signed-off-by: Khiem Doan <[email protected]> * New unit value 64kB Signed-off-by: Oleksandr Mysyura <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Update exporter-toolkit Update to the latest exporter-toolkit * Enables multi-listener and systemd socket activation. * Bump Go to 1.19. * Remove `PG_EXPORTER_WEB_LISTEN_ADDRESS` env var because this is now a repeatable flag. Signed-off-by: SuperQ <[email protected]> * go fmt Signed-off-by: SuperQ <[email protected]> * adding codified functionality for logical replication metrics Signed-off-by: Zachary Caldarola <[email protected]> * Bump github.com./prometheus/client_golang from 1.13.0 to 1.14.0 Bumps [github.com./prometheus/client_golang](https://github.com./prometheus/client_golang) from 1.13.0 to 1.14.0. - [Release notes](https://github.com./prometheus/client_golang/releases) - [Changelog](https://github.com./prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.13.0...v1.14.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Bump github.com./prometheus/common from 0.37.0 to 0.39.0 Bumps [github.com./prometheus/common](https://github.com./prometheus/common) from 0.37.0 to 0.39.0. - [Release notes](https://github.com./prometheus/common/releases) - [Commits](prometheus/common@v0.37.0...v0.39.0) --- updated-dependencies: - dependency-name: github.com./prometheus/common dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * addressing comments Signed-off-by: Zachary Caldarola <[email protected]> * more comments Signed-off-by: Zachary Caldarola <[email protected]> * fmt Signed-off-by: Zachary Caldarola <[email protected]> * typing Signed-off-by: Zachary Caldarola <[email protected]> * fmt Signed-off-by: Zachary Caldarola <[email protected]> * send stdout/stderr to syslog Signed-off-by: Mike <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Fix exclude-databases for collector package The pg_database collector was not respecting the --exclude-databases flag and causing problems where databases were not accessible. This now respects the list of databases to exclude. - Adjusts the Collector create func to take a config struct instead of a logger. This allows more changes like this in the future. I figured we would need to do this at some point but I wasn't sure if we could hold off. - Split the database size collection to a separate query when database is not excluded. - Comment some probe code that was not useful/accurate Signed-off-by: Joe Adams <[email protected]> * Remove commented code Signed-off-by: Joe Adams <[email protected]> * Remove more dead code Signed-off-by: Joe Adams <[email protected]> * Update build * Update Go to 1.20. * Update golanci-lint. * Bump modules. * Update CI orb. * Fix up use of deprecated ioutil. Signed-off-by: SuperQ <[email protected]> * Reduce cardinality of pg_stat_statements Make the example queries.yaml `pg_stat_statements` query safer. * Select the top 10% of queries by total query time. * Only expose the top 100 queries by total query time. * Keep only the most useful metrics. * Comment out the example by default. Fixes: prometheus-community#549 Signed-off-by: SuperQ <[email protected]> * Update changelog and version for v0.12.0 release Signed-off-by: Joe Adams <[email protected]> * Update exporter-toolkit Updates the exporter-toolkit to the latest version * Adds new landing page feature. * Allow metrics path to be on `/`. Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Fix column type for pg_replication_slots Change the data type of `active` from int64 to bool. The documentation confirms that this is a boolean field. https://www.postgresql.org/docs/current/view-pg-replication-slots.html fixes prometheus-community#769 Signed-off-by: Joe Adams <[email protected]> * Update versions listed in the README Update the supported versions based on what we actually test in CI. Signed-off-by: SuperQ <[email protected]> * Update README cli flags These have not been kept up to date. Signed-off-by: Joe Adams <[email protected]> * Adjust log level for collector startup Since we support both multi-target and typical direct scrapes, either of these can fail and it is no longer an error. Signed-off-by: Joe Adams <[email protected]> * Fix pg_setting different help values Signed-off-by: GitHub <[email protected]> * Supports alternate postgres:// prefix in URLs Adds support for the alternate postgres:// prefix in URLs. It's maybe not the cleanest approach, but works. Hoping I can either get some pointers on a more appropriate patch, or that we could use this in the interim to unblock this use-case. Signed-off-by: Jack Wink <[email protected]> * Bump github.com./lib/pq from 1.10.7 to 1.10.9 Bumps [github.com./lib/pq](https://github.com./lib/pq) from 1.10.7 to 1.10.9. - [Release notes](https://github.com./lib/pq/releases) - [Commits](lib/pq@v1.10.7...v1.10.9) --- updated-dependencies: - dependency-name: github.com./lib/pq dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Refactor collector descriptors Use individual collector metric descriptor vars to help avoid miss-mapped or unused metrics. Signed-off-by: SuperQ <[email protected]> * Bump github.com./prometheus/common from 0.42.0 to 0.44.0 Bumps [github.com./prometheus/common](https://github.com./prometheus/common) from 0.42.0 to 0.44.0. - [Release notes](https://github.com./prometheus/common/releases) - [Commits](prometheus/common@v0.42.0...v0.44.0) --- updated-dependencies: - dependency-name: github.com./prometheus/common dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Update linting * Move errcheck exclude list to config file. * Enable revive linter * Fix up revive linting issues. Signed-off-by: SuperQ <[email protected]> * Bump github.com./prometheus/exporter-toolkit from 0.9.1 to 0.10.0 Bumps [github.com./prometheus/exporter-toolkit](https://github.com./prometheus/exporter-toolkit) from 0.9.1 to 0.10.0. - [Release notes](https://github.com./prometheus/exporter-toolkit/releases) - [Changelog](https://github.com./prometheus/exporter-toolkit/blob/master/CHANGELOG.md) - [Commits](prometheus/exporter-toolkit@v0.9.1...v0.10.0) --- updated-dependencies: - dependency-name: github.com./prometheus/exporter-toolkit dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Move queries from queries.yaml to collectors (prometheus-community#801) Signed-off-by: Ben Kochie <[email protected]> * Fix pg_stat_database collector The signature for creating a collector changed and CI didn't retrigger. Move metrics out of map and into individual vars. Signed-off-by: Joe Adams <[email protected]> * Fix up collector registration (prometheus-community#812) Use const definitions to make collector registration consistent. * Use collector subsystem name consistently. * Fix up replication metric name unit. Signed-off-by: SuperQ <[email protected]> * Update release info for v0.12.1 Signed-off-by: Joe Adams <[email protected]> * Deprecate extend queries feature (prometheus-community#811) Mark the extend queries feature as deprecated in favor of recommending the sql_exporter. Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Deprecate additional database features Now that we have deprecated extended queries we can deprecate related database features. * Deprecate flags/functions around auto discover databases. * Deprecate flags/functions for additional constant labels. Signed-off-by: SuperQ <[email protected]> * Release v0.13.0 BREAKING CHANGES: Please note, the following features are deprecated and may be removed in a future release: - `auto-discover-databases` - `extend.query-path` - `constantLabels` - `exclude-databases` - `include-databases` This exporter is meant to monitor PostgresSQL servers, not the user data/databases. If you need a generic SQL report exporter https://github.com./burningalchemist/sql_exporter is recommended. * [CHANGE] Adjust log level for collector startup prometheus-community#784 * [CHANGE] Move queries from queries.yaml to collectors prometheus-community#801 * [CHANGE] Deprecate extend queries feature prometheus-community#811 * [CHANGE] Deprecate additional database features prometheus-community#815 * [CHANGE] Convert pg_stat_database to new collector prometheus-community#685 * [ENHANCEMENT] Supports alternate postgres:// prefix in URLs prometheus-community#787 * [BUGFIX] Fix pg_setting different help values prometheus-community#771 * [BUGFIX] Fix column type for pg_replication_slots prometheus-community#777 * [BUGFIX] Fix pg_stat_database collector prometheus-community#809 Signed-off-by: SuperQ <[email protected]> * Add the instance struct to handle connections The intent is to use the instance struct to hold the connection to the database as well as metadata about the instance. Currently this metadata only includes the version of postgres for the instance which can be used in the collectors to decide what query to run. In the future this could hold more metadata but for now it keeps the Collector interface arguments to a reasonable number. Signed-off-by: Joe Adams <[email protected]> * chore: fix a few typos Signed-off-by: Alex Tymchuk <[email protected]> * Bug fix: Make collector not fail on null values (prometheus-community#823) * Make all values nullable --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Release 0.13.1 (prometheus-community#824) * [BUGFIX] Make collectors not fail on null values prometheus-community#823 Signed-off-by: SuperQ <[email protected]> * Fixed replication pgReplicationSlotQuery - now it's working correctly for replica and primary (prometheus-community#825) Signed-off-by: Vadim Voitenko <[email protected]> Co-authored-by: Vadim Voitenko <[email protected]> * Migrate pg_locks to collector package (prometheus-community#817) Migrate the `pg_locks_count` query from `main` to the `collector` package. Signed-off-by: SuperQ <[email protected]> * Cleanup collectors (prometheus-community#826) Fix up `replication` and `process_idle` Update input params to match the rest of the collectors. Signed-off-by: SuperQ <[email protected]> * Bug Fix: Fix lingering type issues (prometheus-community#828) * Fix postmaster type issue * Disable postmaster collector by default --------- Signed-off-by: Felix Yuan <[email protected]> * Update common Prometheus files (prometheus-community#829) Signed-off-by: prombot <[email protected]> * Fix replication collector Signed-off-by: Tom Hughes <[email protected]> * Add some more escapes to the query sanitizer Signed-off-by: Tom Hughes <[email protected]> * Add a collector to gather metrics on WAL size Signed-off-by: Tom Hughes <[email protected]> * Bump github.com./prometheus/client_golang from 1.15.1 to 1.16.0 (prometheus-community#853) Bumps [github.com./prometheus/client_golang](https://github.com./prometheus/client_golang) from 1.15.1 to 1.16.0. - [Release notes](https://github.com./prometheus/client_golang/releases) - [Changelog](https://github.com./prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.15.1...v1.16.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Fix untyped integer overflows on 32-bit archs (prometheus-community#857) go-sqlmock's Rows.AddRow() takes values which have a type alias of "any", and appear to default to untyped ints if not explicitly cast. When large values are passed which would overflow int32, tests fail. Signed-off-by: Daniel Swarbrick <[email protected]> * Bump github.com./smartystreets/goconvey from 1.8.0 to 1.8.1 (prometheus-community#852) Bumps [github.com./smartystreets/goconvey](https://github.com./smartystreets/goconvey) from 1.8.0 to 1.8.1. - [Release notes](https://github.com./smartystreets/goconvey/releases) - [Commits](smartystreets/goconvey@v1.8.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com./smartystreets/goconvey dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Unpack postgres arrays for process idle times correctly (prometheus-community#855) Signed-off-by: Ben Kochie <[email protected]> * Include all idle processes in the process idle metrics Signed-off-by: Tom Hughes <[email protected]> * Improve linting (prometheus-community#861) * Disable unused-parameter check due to false positives on Collect() calls. * Enable misspell. * Simplify error returns. Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files (prometheus-community#860) Signed-off-by: prombot <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Gitlab collector: Database wraparound collector and test (prometheus-community#834) * Database wraparound collector and test --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Joe Adams <[email protected]> * Add a logger to stat_database collector to get better handle on error (also clean up some metric validity checks) Signed-off-by: Felix Yuan <[email protected]> * Update changelog for release 0.13.2 (prometheus-community#872) Signed-off-by: Joe Adams <[email protected]> * Gitlab Collector: Autovacuum collector and test (prometheus-community#840) * Autovacuum collector and test Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_activity_autovacuum.go Co-authored-by: Joe Adams <[email protected]> Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_activity_autovacuum.go Co-authored-by: Joe Adams <[email protected]> Signed-off-by: Felix Yuan <[email protected]> * Use timestamp seconds Signed-off-by: Felix Yuan <[email protected]> * query formating Signed-off-by: Felix Yuan <[email protected]> * SQL format Signed-off-by: Felix Yuan <[email protected]> * Loosen autovacuum query Signed-off-by: Felix Yuan <[email protected]> --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Joe Adams <[email protected]> * Gitlab Collector: Wal Receiver Collector and Test (prometheus-community#844) * Wal Receiver Collector and Test Signed-off-by: Felix Yuan <[email protected]> * Add more escapes Signed-off-by: Felix Yuan <[email protected]> * Corrections to wal_receiver Signed-off-by: Felix Yuan <[email protected]> * Continue on null labels Signed-off-by: Felix Yuan <[email protected]> * Skip nulls and log a message Signed-off-by: Felix Yuan <[email protected]> * Redundant breaks Signed-off-by: Felix Yuan <[email protected]> * Fix up walreceiver Signed-off-by: Felix Yuan <[email protected]> * Remove extra label Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_walreceiver.go Co-authored-by: Ben Kochie <[email protected]> Signed-off-by: Felix Yuan <[email protected]> * Clean up the extra assignments Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_walreceiver.go Co-authored-by: Joe Adams <[email protected]> Signed-off-by: Felix Yuan <[email protected]> --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> Co-authored-by: Joe Adams <[email protected]> * Gitlab collector: Xlog location collector and test (prometheus-community#849) * Xlog location collector and test Signed-off-by: Felix Yuan <[email protected]> * Add more escapes Signed-off-by: Felix Yuan <[email protected]> * Change to Gauge Signed-off-by: Felix Yuan <[email protected]> --------- Signed-off-by: Felix Yuan <[email protected]> * Handle new pg_stat_statements column names (prometheus-community#874) Update pg_stat_statements collector to handle the new column names in PostgreSQL 13. Fixes: prometheus-community#502 Signed-off-by: SuperQ <[email protected]> * Fixup new pg_stats_statements query (prometheus-community#876) Fix all renames of `total_time` to `total_exec_time`. Fixes: prometheus-community#502 Signed-off-by: SuperQ <[email protected]> * Add a multi-target example config (prometheus-community#890) Add an example Prometheus scrape config, similar to the blackbox_exporter's example config. Fixes: prometheus-community#888 Signed-off-by: SuperQ <[email protected]> * Delay database connection until scrape (prometheus-community#882) This no longer returns an error when creating a collector.instance when the database cannot be reached for the version query. This will resolve the entire postgresCollector not being registered for metrics collection when a database is not available. If the version query fails, the scrape will fail. Resolves prometheus-community#880 Signed-off-by: Joe Adams <[email protected]> * Bugfix: Make statsreset nullable (prometheus-community#877) * Stats_reset as null seems to actually be legitimate for new databases, so don't fail for it --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Gitlab Collector: User Index io stats collector and test (prometheus-community#845) * User Index io stats collector and test --------- Signed-off-by: Felix Yuan <[email protected]> * Update README to reflect changes made in prometheus-community#828 (prometheus-community#894) Signed-off-by: Mathis Raguin <[email protected]> * Gitlab Collector: Long running transactions collector and test (prometheus-community#836) * Long running transactions collector and test --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Update common Prometheus files (prometheus-community#900) Signed-off-by: prombot <[email protected]> * Fix a connection leak (prometheus-community#902) The leak was introduced in PR#882 Signed-off-by: Christian Albrecht <[email protected]> Co-authored-by: Christian Albrecht <[email protected]> * Fix cross-compilation command in README.md (prometheus-community#903) Signed-off-by: David Cook <[email protected]> * fix pg_replication_lag_seconds (prometheus-community#895) Signed-off-by: Vladimir Luksha <[email protected]> Co-authored-by: Vladimir Luksha <[email protected]> * stat_user_tables: Add total size metric (prometheus-community#904) Signed-off-by: David Cook <[email protected]> * Fix bugs mentioned in prometheus-community#908 (prometheus-community#910) * Fix bugs mentioned in prometheus-community#908 These collectors are disabled by default, so unless enabled, they are not tested regularly. Signed-off-by: Joe Adams <[email protected]> --------- Signed-off-by: Joe Adams <[email protected]> * Update common Prometheus files (prometheus-community#913) Signed-off-by: prombot <[email protected]> * Add changelog for v0.14 (prometheus-community#906) * Add changelog for v0.14 - Add changelog entries since v0.13.2 - Update README with new options - Bump version file Signed-off-by: Joe Adams <[email protected]> * Add changelog entry for prometheus-community#904 Signed-off-by: Joe Adams <[email protected]> --------- Signed-off-by: Joe Adams <[email protected]> * PMM-12154 pull upstream changes. * PMM-12154 Fix go mod. * PMM-12154 Remove some built-in queries, they were moved to collectors. * PMM-12154 compatibility improvements. * PMM-12154 compatibility improvements. * PMM-12154 performance improvement. * revert pg_lock_conflicts --------- Signed-off-by: Ryan J. Geyer <[email protected]> Signed-off-by: Joe Adams <[email protected]> Signed-off-by: bravosierrasierra <[email protected]> Signed-off-by: Nicolas Rodriguez <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Luckz <[email protected]> Signed-off-by: SuperQ <[email protected]> Signed-off-by: Yoan Blanc <[email protected]> Signed-off-by: Ildar Valiullin <[email protected]> Signed-off-by: cezmunsta <[email protected]> Signed-off-by: prombot <[email protected]> Signed-off-by: Sergey Morozov <[email protected]> Signed-off-by: Julien Pivotto <[email protected]> Signed-off-by: Kurtis Bass <[email protected]> Signed-off-by: Khiem Doan <[email protected]> Signed-off-by: Oleksandr Mysyura <[email protected]> Signed-off-by: Zachary Caldarola <[email protected]> Signed-off-by: Zachary Caldarola <[email protected]> Signed-off-by: Mike <[email protected]> Signed-off-by: GitHub <[email protected]> Signed-off-by: Jack Wink <[email protected]> Signed-off-by: Ben Kochie <[email protected]> Signed-off-by: Alex Tymchuk <[email protected]> Signed-off-by: Felix Yuan <[email protected]> Signed-off-by: Vadim Voitenko <[email protected]> Signed-off-by: Tom Hughes <[email protected]> Signed-off-by: Daniel Swarbrick <[email protected]> Signed-off-by: Mathis Raguin <[email protected]> Signed-off-by: Christian Albrecht <[email protected]> Signed-off-by: David Cook <[email protected]> Signed-off-by: Vladimir Luksha <[email protected]> Co-authored-by: Ryan J. Geyer <[email protected]> Co-authored-by: Joe Adams <[email protected]> Co-authored-by: Ben Kochie <[email protected]> Co-authored-by: Joe Adams <[email protected]> Co-authored-by: bravosierrasierra <[email protected]> Co-authored-by: Nicolas Rodriguez <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> Co-authored-by: Luckz <[email protected]> Co-authored-by: Yoan Blanc <[email protected]> Co-authored-by: Ildar Valiullin <[email protected]> Co-authored-by: cezmunsta <[email protected]> Co-authored-by: prombot <[email protected]> Co-authored-by: Sergey Morozov <[email protected]> Co-authored-by: Julien Pivotto <[email protected]> Co-authored-by: Kurtis Bass <[email protected]> Co-authored-by: Khiem Doan <[email protected]> Co-authored-by: Oleksandr Mysyura <[email protected]> Co-authored-by: Zachary Caldarola <[email protected]> Co-authored-by: Zachary Caldarola <[email protected]> Co-authored-by: Mike <[email protected]> Co-authored-by: Khaled Khalifa <[email protected]> Co-authored-by: Jack Wink <[email protected]> Co-authored-by: Felix Yuan <[email protected]> Co-authored-by: Alex Tymchuk <[email protected]> Co-authored-by: Vadim Voitenko <[email protected]> Co-authored-by: Vadim Voitenko <[email protected]> Co-authored-by: Tom Hughes <[email protected]> Co-authored-by: Daniel Swarbrick <[email protected]> Co-authored-by: Mathis Raguin <[email protected]> Co-authored-by: Christian Albrecht <[email protected]> Co-authored-by: Christian Albrecht <[email protected]> Co-authored-by: David Cook <[email protected]> Co-authored-by: Vladimir Luksha <[email protected]> Co-authored-by: Vladimir Luksha <[email protected]> Co-authored-by: David Cook <[email protected]>
Hello, it appears that as part of this change the Currently working on upgrading from an older version and noted that the labels for |
* Dashboard linting improvements for mixin Signed-off-by: Ryan J. Geyer <[email protected]> * Convert pg_stat_database to new collector model Signed-off-by: Joe Adams <[email protected]> * Capture usename and application_name for pg_stat_activity It is necessary to be able to exclude backups from long-running transaction alerts, as they are to be expected. With the current pg_stat_activity metric there is no ability to filter out specific users or application names. Resolves prometheus-community#668 Signed-off-by: cezmunsta <[email protected]> * Fixed formatting Signed-off-by: cezmunsta <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * probe: clean-up database connection after probe to prevent connection leak Signed-off-by: Kurtis Bass <[email protected]> * Set gauge to 1 when collector is successful Signed-off-by: Julien Pivotto <[email protected]> Signed-off-by: Khiem Doan <[email protected]> * Add postgres 15 for CI test Signed-off-by: Khiem Doan <[email protected]> * Add postgres 15 for CI test Signed-off-by: Khiem Doan <[email protected]> * New unit value 64kB Signed-off-by: Oleksandr Mysyura <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Update exporter-toolkit Update to the latest exporter-toolkit * Enables multi-listener and systemd socket activation. * Bump Go to 1.19. * Remove `PG_EXPORTER_WEB_LISTEN_ADDRESS` env var because this is now a repeatable flag. Signed-off-by: SuperQ <[email protected]> * go fmt Signed-off-by: SuperQ <[email protected]> * adding codified functionality for logical replication metrics Signed-off-by: Zachary Caldarola <[email protected]> * Bump github.com./prometheus/client_golang from 1.13.0 to 1.14.0 Bumps [github.com./prometheus/client_golang](https://github.com./prometheus/client_golang) from 1.13.0 to 1.14.0. - [Release notes](https://github.com./prometheus/client_golang/releases) - [Changelog](https://github.com./prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.13.0...v1.14.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Bump github.com./prometheus/common from 0.37.0 to 0.39.0 Bumps [github.com./prometheus/common](https://github.com./prometheus/common) from 0.37.0 to 0.39.0. - [Release notes](https://github.com./prometheus/common/releases) - [Commits](prometheus/common@v0.37.0...v0.39.0) --- updated-dependencies: - dependency-name: github.com./prometheus/common dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * addressing comments Signed-off-by: Zachary Caldarola <[email protected]> * more comments Signed-off-by: Zachary Caldarola <[email protected]> * fmt Signed-off-by: Zachary Caldarola <[email protected]> * typing Signed-off-by: Zachary Caldarola <[email protected]> * fmt Signed-off-by: Zachary Caldarola <[email protected]> * send stdout/stderr to syslog Signed-off-by: Mike <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Fix exclude-databases for collector package The pg_database collector was not respecting the --exclude-databases flag and causing problems where databases were not accessible. This now respects the list of databases to exclude. - Adjusts the Collector create func to take a config struct instead of a logger. This allows more changes like this in the future. I figured we would need to do this at some point but I wasn't sure if we could hold off. - Split the database size collection to a separate query when database is not excluded. - Comment some probe code that was not useful/accurate Signed-off-by: Joe Adams <[email protected]> * Remove commented code Signed-off-by: Joe Adams <[email protected]> * Remove more dead code Signed-off-by: Joe Adams <[email protected]> * Update build * Update Go to 1.20. * Update golanci-lint. * Bump modules. * Update CI orb. * Fix up use of deprecated ioutil. Signed-off-by: SuperQ <[email protected]> * Reduce cardinality of pg_stat_statements Make the example queries.yaml `pg_stat_statements` query safer. * Select the top 10% of queries by total query time. * Only expose the top 100 queries by total query time. * Keep only the most useful metrics. * Comment out the example by default. Fixes: prometheus-community#549 Signed-off-by: SuperQ <[email protected]> * Update changelog and version for v0.12.0 release Signed-off-by: Joe Adams <[email protected]> * Update exporter-toolkit Updates the exporter-toolkit to the latest version * Adds new landing page feature. * Allow metrics path to be on `/`. Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Fix column type for pg_replication_slots Change the data type of `active` from int64 to bool. The documentation confirms that this is a boolean field. https://www.postgresql.org/docs/current/view-pg-replication-slots.html fixes prometheus-community#769 Signed-off-by: Joe Adams <[email protected]> * Update versions listed in the README Update the supported versions based on what we actually test in CI. Signed-off-by: SuperQ <[email protected]> * Update README cli flags These have not been kept up to date. Signed-off-by: Joe Adams <[email protected]> * Adjust log level for collector startup Since we support both multi-target and typical direct scrapes, either of these can fail and it is no longer an error. Signed-off-by: Joe Adams <[email protected]> * Fix pg_setting different help values Signed-off-by: GitHub <[email protected]> * Supports alternate postgres:// prefix in URLs Adds support for the alternate postgres:// prefix in URLs. It's maybe not the cleanest approach, but works. Hoping I can either get some pointers on a more appropriate patch, or that we could use this in the interim to unblock this use-case. Signed-off-by: Jack Wink <[email protected]> * Bump github.com./lib/pq from 1.10.7 to 1.10.9 Bumps [github.com./lib/pq](https://github.com./lib/pq) from 1.10.7 to 1.10.9. - [Release notes](https://github.com./lib/pq/releases) - [Commits](lib/pq@v1.10.7...v1.10.9) --- updated-dependencies: - dependency-name: github.com./lib/pq dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Refactor collector descriptors Use individual collector metric descriptor vars to help avoid miss-mapped or unused metrics. Signed-off-by: SuperQ <[email protected]> * Bump github.com./prometheus/common from 0.42.0 to 0.44.0 Bumps [github.com./prometheus/common](https://github.com./prometheus/common) from 0.42.0 to 0.44.0. - [Release notes](https://github.com./prometheus/common/releases) - [Commits](prometheus/common@v0.42.0...v0.44.0) --- updated-dependencies: - dependency-name: github.com./prometheus/common dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Update linting * Move errcheck exclude list to config file. * Enable revive linter * Fix up revive linting issues. Signed-off-by: SuperQ <[email protected]> * Bump github.com./prometheus/exporter-toolkit from 0.9.1 to 0.10.0 Bumps [github.com./prometheus/exporter-toolkit](https://github.com./prometheus/exporter-toolkit) from 0.9.1 to 0.10.0. - [Release notes](https://github.com./prometheus/exporter-toolkit/releases) - [Changelog](https://github.com./prometheus/exporter-toolkit/blob/master/CHANGELOG.md) - [Commits](prometheus/exporter-toolkit@v0.9.1...v0.10.0) --- updated-dependencies: - dependency-name: github.com./prometheus/exporter-toolkit dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Move queries from queries.yaml to collectors (prometheus-community#801) Signed-off-by: Ben Kochie <[email protected]> * Fix pg_stat_database collector The signature for creating a collector changed and CI didn't retrigger. Move metrics out of map and into individual vars. Signed-off-by: Joe Adams <[email protected]> * Fix up collector registration (prometheus-community#812) Use const definitions to make collector registration consistent. * Use collector subsystem name consistently. * Fix up replication metric name unit. Signed-off-by: SuperQ <[email protected]> * Update release info for v0.12.1 Signed-off-by: Joe Adams <[email protected]> * Deprecate extend queries feature (prometheus-community#811) Mark the extend queries feature as deprecated in favor of recommending the sql_exporter. Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Deprecate additional database features Now that we have deprecated extended queries we can deprecate related database features. * Deprecate flags/functions around auto discover databases. * Deprecate flags/functions for additional constant labels. Signed-off-by: SuperQ <[email protected]> * Release v0.13.0 BREAKING CHANGES: Please note, the following features are deprecated and may be removed in a future release: - `auto-discover-databases` - `extend.query-path` - `constantLabels` - `exclude-databases` - `include-databases` This exporter is meant to monitor PostgresSQL servers, not the user data/databases. If you need a generic SQL report exporter https://github.com./burningalchemist/sql_exporter is recommended. * [CHANGE] Adjust log level for collector startup prometheus-community#784 * [CHANGE] Move queries from queries.yaml to collectors prometheus-community#801 * [CHANGE] Deprecate extend queries feature prometheus-community#811 * [CHANGE] Deprecate additional database features prometheus-community#815 * [CHANGE] Convert pg_stat_database to new collector prometheus-community#685 * [ENHANCEMENT] Supports alternate postgres:// prefix in URLs prometheus-community#787 * [BUGFIX] Fix pg_setting different help values prometheus-community#771 * [BUGFIX] Fix column type for pg_replication_slots prometheus-community#777 * [BUGFIX] Fix pg_stat_database collector prometheus-community#809 Signed-off-by: SuperQ <[email protected]> * Add the instance struct to handle connections The intent is to use the instance struct to hold the connection to the database as well as metadata about the instance. Currently this metadata only includes the version of postgres for the instance which can be used in the collectors to decide what query to run. In the future this could hold more metadata but for now it keeps the Collector interface arguments to a reasonable number. Signed-off-by: Joe Adams <[email protected]> * chore: fix a few typos Signed-off-by: Alex Tymchuk <[email protected]> * Bug fix: Make collector not fail on null values (prometheus-community#823) * Make all values nullable --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Release 0.13.1 (prometheus-community#824) * [BUGFIX] Make collectors not fail on null values prometheus-community#823 Signed-off-by: SuperQ <[email protected]> * Fixed replication pgReplicationSlotQuery - now it's working correctly for replica and primary (prometheus-community#825) Signed-off-by: Vadim Voitenko <[email protected]> Co-authored-by: Vadim Voitenko <[email protected]> * Migrate pg_locks to collector package (prometheus-community#817) Migrate the `pg_locks_count` query from `main` to the `collector` package. Signed-off-by: SuperQ <[email protected]> * Cleanup collectors (prometheus-community#826) Fix up `replication` and `process_idle` Update input params to match the rest of the collectors. Signed-off-by: SuperQ <[email protected]> * Bug Fix: Fix lingering type issues (prometheus-community#828) * Fix postmaster type issue * Disable postmaster collector by default --------- Signed-off-by: Felix Yuan <[email protected]> * Update common Prometheus files (prometheus-community#829) Signed-off-by: prombot <[email protected]> * Fix replication collector Signed-off-by: Tom Hughes <[email protected]> * Add some more escapes to the query sanitizer Signed-off-by: Tom Hughes <[email protected]> * Add a collector to gather metrics on WAL size Signed-off-by: Tom Hughes <[email protected]> * Bump github.com./prometheus/client_golang from 1.15.1 to 1.16.0 (prometheus-community#853) Bumps [github.com./prometheus/client_golang](https://github.com./prometheus/client_golang) from 1.15.1 to 1.16.0. - [Release notes](https://github.com./prometheus/client_golang/releases) - [Changelog](https://github.com./prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.15.1...v1.16.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Fix untyped integer overflows on 32-bit archs (prometheus-community#857) go-sqlmock's Rows.AddRow() takes values which have a type alias of "any", and appear to default to untyped ints if not explicitly cast. When large values are passed which would overflow int32, tests fail. Signed-off-by: Daniel Swarbrick <[email protected]> * Bump github.com./smartystreets/goconvey from 1.8.0 to 1.8.1 (prometheus-community#852) Bumps [github.com./smartystreets/goconvey](https://github.com./smartystreets/goconvey) from 1.8.0 to 1.8.1. - [Release notes](https://github.com./smartystreets/goconvey/releases) - [Commits](smartystreets/goconvey@v1.8.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com./smartystreets/goconvey dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Unpack postgres arrays for process idle times correctly (prometheus-community#855) Signed-off-by: Ben Kochie <[email protected]> * Include all idle processes in the process idle metrics Signed-off-by: Tom Hughes <[email protected]> * Improve linting (prometheus-community#861) * Disable unused-parameter check due to false positives on Collect() calls. * Enable misspell. * Simplify error returns. Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files (prometheus-community#860) Signed-off-by: prombot <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Update common Prometheus files Signed-off-by: prombot <[email protected]> * Gitlab collector: Database wraparound collector and test (prometheus-community#834) * Database wraparound collector and test --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Joe Adams <[email protected]> * Add a logger to stat_database collector to get better handle on error (also clean up some metric validity checks) Signed-off-by: Felix Yuan <[email protected]> * Update changelog for release 0.13.2 (prometheus-community#872) Signed-off-by: Joe Adams <[email protected]> * Gitlab Collector: Autovacuum collector and test (prometheus-community#840) * Autovacuum collector and test Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_activity_autovacuum.go Co-authored-by: Joe Adams <[email protected]> Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_activity_autovacuum.go Co-authored-by: Joe Adams <[email protected]> Signed-off-by: Felix Yuan <[email protected]> * Use timestamp seconds Signed-off-by: Felix Yuan <[email protected]> * query formating Signed-off-by: Felix Yuan <[email protected]> * SQL format Signed-off-by: Felix Yuan <[email protected]> * Loosen autovacuum query Signed-off-by: Felix Yuan <[email protected]> --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Joe Adams <[email protected]> * Gitlab Collector: Wal Receiver Collector and Test (prometheus-community#844) * Wal Receiver Collector and Test Signed-off-by: Felix Yuan <[email protected]> * Add more escapes Signed-off-by: Felix Yuan <[email protected]> * Corrections to wal_receiver Signed-off-by: Felix Yuan <[email protected]> * Continue on null labels Signed-off-by: Felix Yuan <[email protected]> * Skip nulls and log a message Signed-off-by: Felix Yuan <[email protected]> * Redundant breaks Signed-off-by: Felix Yuan <[email protected]> * Fix up walreceiver Signed-off-by: Felix Yuan <[email protected]> * Remove extra label Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_walreceiver.go Co-authored-by: Ben Kochie <[email protected]> Signed-off-by: Felix Yuan <[email protected]> * Clean up the extra assignments Signed-off-by: Felix Yuan <[email protected]> * Update collector/pg_stat_walreceiver.go Co-authored-by: Joe Adams <[email protected]> Signed-off-by: Felix Yuan <[email protected]> --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> Co-authored-by: Joe Adams <[email protected]> * Gitlab collector: Xlog location collector and test (prometheus-community#849) * Xlog location collector and test Signed-off-by: Felix Yuan <[email protected]> * Add more escapes Signed-off-by: Felix Yuan <[email protected]> * Change to Gauge Signed-off-by: Felix Yuan <[email protected]> --------- Signed-off-by: Felix Yuan <[email protected]> * Handle new pg_stat_statements column names (prometheus-community#874) Update pg_stat_statements collector to handle the new column names in PostgreSQL 13. Fixes: prometheus-community#502 Signed-off-by: SuperQ <[email protected]> * Fixup new pg_stats_statements query (prometheus-community#876) Fix all renames of `total_time` to `total_exec_time`. Fixes: prometheus-community#502 Signed-off-by: SuperQ <[email protected]> * Add a multi-target example config (prometheus-community#890) Add an example Prometheus scrape config, similar to the blackbox_exporter's example config. Fixes: prometheus-community#888 Signed-off-by: SuperQ <[email protected]> * Delay database connection until scrape (prometheus-community#882) This no longer returns an error when creating a collector.instance when the database cannot be reached for the version query. This will resolve the entire postgresCollector not being registered for metrics collection when a database is not available. If the version query fails, the scrape will fail. Resolves prometheus-community#880 Signed-off-by: Joe Adams <[email protected]> * Bugfix: Make statsreset nullable (prometheus-community#877) * Stats_reset as null seems to actually be legitimate for new databases, so don't fail for it --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Gitlab Collector: User Index io stats collector and test (prometheus-community#845) * User Index io stats collector and test --------- Signed-off-by: Felix Yuan <[email protected]> * Update README to reflect changes made in prometheus-community#828 (prometheus-community#894) Signed-off-by: Mathis Raguin <[email protected]> * Gitlab Collector: Long running transactions collector and test (prometheus-community#836) * Long running transactions collector and test --------- Signed-off-by: Felix Yuan <[email protected]> Co-authored-by: Ben Kochie <[email protected]> * Update common Prometheus files (prometheus-community#900) Signed-off-by: prombot <[email protected]> * Fix a connection leak (prometheus-community#902) The leak was introduced in PR#882 Signed-off-by: Christian Albrecht <[email protected]> Co-authored-by: Christian Albrecht <[email protected]> * Fix cross-compilation command in README.md (prometheus-community#903) Signed-off-by: David Cook <[email protected]> * fix pg_replication_lag_seconds (prometheus-community#895) Signed-off-by: Vladimir Luksha <[email protected]> Co-authored-by: Vladimir Luksha <[email protected]> * stat_user_tables: Add total size metric (prometheus-community#904) Signed-off-by: David Cook <[email protected]> * Fix bugs mentioned in prometheus-community#908 (prometheus-community#910) * Fix bugs mentioned in prometheus-community#908 These collectors are disabled by default, so unless enabled, they are not tested regularly. Signed-off-by: Joe Adams <[email protected]> --------- Signed-off-by: Joe Adams <[email protected]> * Update common Prometheus files (prometheus-community#913) Signed-off-by: prombot <[email protected]> * Add changelog for v0.14 (prometheus-community#906) * Add changelog for v0.14 - Add changelog entries since v0.13.2 - Update README with new options - Bump version file Signed-off-by: Joe Adams <[email protected]> * Add changelog entry for prometheus-community#904 Signed-off-by: Joe Adams <[email protected]> --------- Signed-off-by: Joe Adams <[email protected]> * Adds 1kB and 2kB units (prometheus-community#915) Signed-off-by: Eric tyrrell <[email protected]> * Add error log when probe collector creation fails (prometheus-community#918) Signed-off-by: Joe Adams <[email protected]> * Fix test build failures on 32-bit arch again (prometheus-community#919) Another case of untyped integer overflows on 32-bit arch. Signed-off-by: Daniel Swarbrick <[email protected]> * Add 32-bit testing to CI (prometheus-community#920) Run Go tests with 32-bit to validate value overflow. Signed-off-by: SuperQ <[email protected]> * Bump github.com./prometheus/client_golang from 1.16.0 to 1.17.0 (prometheus-community#925) * Bump github.com./prometheus/client_golang from 1.16.0 to 1.17.0 Bumps [github.com./prometheus/client_golang](https://github.com./prometheus/client_golang) from 1.16.0 to 1.17.0. - [Release notes](https://github.com./prometheus/client_golang/releases) - [Changelog](https://github.com./prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.16.0...v1.17.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Update tests for latest client_golang. Signed-off-by: SuperQ <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: SuperQ <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> Co-authored-by: SuperQ <[email protected]> * Update common Prometheus files (prometheus-community#926) Signed-off-by: prombot <[email protected]> * Adjust collector to use separate connection per scrape (prometheus-community#931) Fixes prometheus-community#921 Signed-off-by: Joe Adams <[email protected]> * Bump golang.org/x/net from 0.10.0 to 0.17.0 (prometheus-community#936) Bumps [golang.org/x/net](https://github.com./golang/net) from 0.10.0 to 0.17.0. - [Commits](golang/net@v0.10.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Release v0.15.0 (prometheus-community#944) * [ENHANCEMENT] Add 1kB and 2kB units prometheus-community#915 * [BUGFIX] Add error log when probe collector creation fails prometheus-community#918 * [BUGFIX] Fix test build failures on 32-bit arch prometheus-community#919 * [BUGFIX] Adjust collector to use separate connection per scrape prometheus-community#936 Signed-off-by: SuperQ <[email protected]> * Update common Prometheus files (prometheus-community#951) Signed-off-by: prombot <[email protected]> * Update common Prometheus files (prometheus-community#963) Signed-off-by: prombot <[email protected]> * pg_replication_slot: add slot type label (prometheus-community#960) Signed-off-by: Alex Simenduev <[email protected]> * Bump github.com./prometheus/common from 0.44.0 to 0.45.0 (prometheus-community#948) Bumps [github.com./prometheus/common](https://github.com./prometheus/common) from 0.44.0 to 0.45.0. - [Release notes](https://github.com./prometheus/common/releases) - [Commits](prometheus/common@v0.44.0...v0.45.0) --- updated-dependencies: - dependency-name: github.com./prometheus/common dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Bump github.com./prometheus/client_model (prometheus-community#949) Bumps [github.com./prometheus/client_model](https://github.com./prometheus/client_model) from 0.4.1-0.20230718164431-9a2bf3000d16 to 0.5.0. - [Release notes](https://github.com./prometheus/client_model/releases) - [Commits](https://github.com./prometheus/client_model/commits/v0.5.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_model dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * pg_stat_database: added support for `active_time` counter (prometheus-community#961) * feat(pg_stat_database): active time metric --------- Signed-off-by: Jiri Sveceny <[email protected]> * Bump golang.org/x/crypto from 0.14.0 to 0.17.0 (prometheus-community#988) Bumps [golang.org/x/crypto](https://github.com./golang/crypto) from 0.14.0 to 0.17.0. - [Commits](golang/crypto@v0.14.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * Bump github.com./prometheus/client_golang from 1.17.0 to 1.18.0 (prometheus-community#993) Bumps [github.com./prometheus/client_golang](https://github.com./prometheus/client_golang) from 1.17.0 to 1.18.0. - [Release notes](https://github.com./prometheus/client_golang/releases) - [Changelog](https://github.com./prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.17.0...v1.18.0) --- updated-dependencies: - dependency-name: github.com./prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> * use Info level for excluded databases log message (prometheus-community#1003) This is the only log message which didn't specify a level in the postgres_exporter. I am unsure if this log message should be info or debug, but leaning towards the more important since previously it would just always log. The way I validated this was the only non-leveled logger was via grep. Both of these only returned this callsite previously: git grep 'logger\.Log' git grep '\.Log(' | grep -v level Signed-off-by: Keegan Carruthers-Smith <[email protected]> --------- Signed-off-by: Ryan J. Geyer <[email protected]> Signed-off-by: Joe Adams <[email protected]> Signed-off-by: cezmunsta <[email protected]> Signed-off-by: prombot <[email protected]> Signed-off-by: Kurtis Bass <[email protected]> Signed-off-by: Julien Pivotto <[email protected]> Signed-off-by: Khiem Doan <[email protected]> Signed-off-by: Oleksandr Mysyura <[email protected]> Signed-off-by: SuperQ <[email protected]> Signed-off-by: Zachary Caldarola <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Zachary Caldarola <[email protected]> Signed-off-by: Mike <[email protected]> Signed-off-by: GitHub <[email protected]> Signed-off-by: Jack Wink <[email protected]> Signed-off-by: Ben Kochie <[email protected]> Signed-off-by: Alex Tymchuk <[email protected]> Signed-off-by: Felix Yuan <[email protected]> Signed-off-by: Vadim Voitenko <[email protected]> Signed-off-by: Tom Hughes <[email protected]> Signed-off-by: Daniel Swarbrick <[email protected]> Signed-off-by: Mathis Raguin <[email protected]> Signed-off-by: Christian Albrecht <[email protected]> Signed-off-by: David Cook <[email protected]> Signed-off-by: Vladimir Luksha <[email protected]> Signed-off-by: Eric tyrrell <[email protected]> Signed-off-by: Alex Simenduev <[email protected]> Signed-off-by: Jiri Sveceny <[email protected]> Signed-off-by: Keegan Carruthers-Smith <[email protected]> Co-authored-by: Ryan J. Geyer <[email protected]> Co-authored-by: Joe Adams <[email protected]> Co-authored-by: cezmunsta <[email protected]> Co-authored-by: prombot <[email protected]> Co-authored-by: Kurtis Bass <[email protected]> Co-authored-by: Julien Pivotto <[email protected]> Co-authored-by: Khiem Doan <[email protected]> Co-authored-by: Oleksandr Mysyura <[email protected]> Co-authored-by: Ben Kochie <[email protected]> Co-authored-by: Zachary Caldarola <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com.> Co-authored-by: Zachary Caldarola <[email protected]> Co-authored-by: Mike <[email protected]> Co-authored-by: Khaled Khalifa <[email protected]> Co-authored-by: Jack Wink <[email protected]> Co-authored-by: Felix Yuan <[email protected]> Co-authored-by: Alex Tymchuk <[email protected]> Co-authored-by: Vadim Voitenko <[email protected]> Co-authored-by: Vadim Voitenko <[email protected]> Co-authored-by: Tom Hughes <[email protected]> Co-authored-by: Daniel Swarbrick <[email protected]> Co-authored-by: Mathis Raguin <[email protected]> Co-authored-by: Christian Albrecht <[email protected]> Co-authored-by: Christian Albrecht <[email protected]> Co-authored-by: David Cook <[email protected]> Co-authored-by: Vladimir Luksha <[email protected]> Co-authored-by: Vladimir Luksha <[email protected]> Co-authored-by: David Cook <[email protected]> Co-authored-by: Eric Tyrrell <[email protected]> Co-authored-by: Alex Simenduev <[email protected]> Co-authored-by: Jiri Sveceny <[email protected]> Co-authored-by: Keegan Carruthers-Smith <[email protected]>
Signed-off-by: Joe Adams [email protected]