-
-
Notifications
You must be signed in to change notification settings - Fork 449
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
Don't make Prometheus monitor prometheus itself #450
Comments
I also noticed a previous PR broke the spacing for the pihole-exporter config, so I'm going to fix that here. |
martinbrose
pushed a commit
to martinbrose/internet-pi
that referenced
this issue
Dec 28, 2022
…prometheus_monitor_prometheus.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
By default, this configuration monitors Prometheus, and scrapes every 5 seconds. There are like 1,000 metrics Prometheus exports, so that's a lot of data to be scraping 720 times per hour, or over 17,000 times per day, especially since none of it is exposed in the dashboards we ship with this project.
So, I propose adding a new
prometheus_monitor_prometheus
variable, and defaulting it tofalse
.Not only do I propose it, I'm about to push the variable to the project, and then this will be done, heh.
The text was updated successfully, but these errors were encountered: