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
Add custom datasource possibilities for postgres_exporter #289
Conversation
manifests/postgres_exporter.pp
Outdated
String[1] $arch = $prometheus::real_arch, | ||
String[1] $bin_dir = $prometheus::bin_dir, | ||
Enum['custom', 'env', 'file'] $postgres_auth_method, | ||
Hash[String,String] $data_source_custom = {}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please enforce the minimal string length?
Hash[String,String] $data_source_custom = {}, | |
Hash[String[1],String[1]] $data_source_custom = {}, |
Hi @romdav00, thanks for the PR! Can you have a look at the used email address in the commits? It isn't associated with your github account. |
String[1] $arch = $prometheus::real_arch, | ||
String[1] $bin_dir = $prometheus::bin_dir, | ||
Enum['custom', 'env', 'file'] $postgres_auth_method, | ||
Hash[String[1],String[1]] $data_source_custom = {}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please align all the =
.
Thanks @romdav00. Can you take a look at the first commits in this branch? The used email address isn't associated with your github account. |
Hi @bastelfreak is it ok for you ? |
ah sorry, this PR slipped through. Thanks for pinging! |
Add custom datasource possibilities for postgres_exporter
Add custom datasource possibilities for postgres_exporter
Hi @bastelfreak , it seems better now :-)