Skip to content

Conversation

SuperQ
Copy link
Contributor

@SuperQ SuperQ commented Nov 1, 2019

Add a standard Prometheus build info metric to make monitoring rollouts
easier.

@coveralls
Copy link

coveralls commented Nov 1, 2019

Coverage Status

Coverage decreased (-0.4%) to 64.7% when pulling 934592d on SuperQ:version_metric into 03013de on wrouesnel:master.

@SuperQ
Copy link
Contributor Author

SuperQ commented Nov 1, 2019

I'm not sure the best way to add test coverage for the additional code.

@wrouesnel
Copy link
Contributor

I'm content with the test coverage, but could you rebase the vendoring changes into one commit?

@SuperQ
Copy link
Contributor Author

SuperQ commented Nov 20, 2019

Sure, no problem

@SuperQ SuperQ force-pushed the version_metric branch 2 times, most recently from 90782ff to 547f01c Compare November 20, 2019 01:37
@SuperQ
Copy link
Contributor Author

SuperQ commented Nov 20, 2019

Fast-forward rebased and squashed the vendor change.

Fix missing bool in builtinMetricMaps. Signed-off-by: Ben Kochie <superq@gmail.com>
Add a standard Prometheus build info metric to make monitoring rollouts easier.
@wrouesnel wrouesnel merged commit aeec47e into prometheus-community:master Nov 25, 2019
english pushed a commit to form3tech-oss/postgres_exporter that referenced this pull request Jan 31, 2020
* Add a build info metric Add a standard Prometheus build info metric to make monitoring rollouts easier. * Update prometheus vendoring. * Fix build error Fix missing bool in builtinMetricMaps. Signed-off-by: Ben Kochie <superq@gmail.com> (cherry picked from commit aeec47e)
english pushed a commit to form3tech-oss/postgres_exporter that referenced this pull request Jan 31, 2020
* Add a build info metric Add a standard Prometheus build info metric to make monitoring rollouts easier. * Update prometheus vendoring. * Fix build error Fix missing bool in builtinMetricMaps. Signed-off-by: Ben Kochie <superq@gmail.com> (cherry picked from commit aeec47e)
ritbl referenced this pull request in heniek/postgres_exporter Mar 19, 2023
* Add a build info metric Add a standard Prometheus build info metric to make monitoring rollouts easier. * Update prometheus vendoring. * Fix build error Fix missing bool in builtinMetricMaps. Signed-off-by: Ben Kochie <superq@gmail.com>
alessio-form3 pushed a commit to form3tech-oss/postgres_exporter that referenced this pull request Oct 10, 2023
* Add a build info metric Add a standard Prometheus build info metric to make monitoring rollouts easier. * Update prometheus vendoring. * Fix build error Fix missing bool in builtinMetricMaps. Signed-off-by: Ben Kochie <superq@gmail.com> (cherry picked from commit aeec47e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants