Joe Adams
Joe Adams
We don't have tests for version 8 but unless there was something big changed in version 8, the exporter should already work with version 8. Is there a metric that...
I don't know that adding in direct kubernetes integration is a great direction from a maintainability perspecitve. What about referencing the secrets that are mounted to the pod as a...
Using a file for username/password is not yet supported but seems like a generally useful feature so I think we would be happy to add it. How secrets get mounted...
What does your config file look like? The errors seem to indicate a problem with the config file. What happens when you try to start the exporter from the CLI...
We don't support specifying the host in the config file. You may not get metrics because you don't have a DSN specified. The logs still show that the config file...
The README has an example of the config file here: https://github.com/prometheus-community/postgres_exporter#auth_modules The queries.yaml file in the repo is now empty because the feature is deprecated. This is the old example:...
The plan is to rework this whole dashboard via #911
Unless this PR is updated to include log messages when relevant, I don't think we need this PR. @Sticksman I will leave it up to you to decide if you...
I think there's a decision to be made on whether or not the exporter is going to support multiple DSNs long term. If it does support multiple DSNs, we would...
These changes are out of sync with the repo. This PR needs a rebase.