Skip to content
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

Closed
geerlingguy opened this issue Oct 14, 2022 · 1 comment
Closed

Don't make Prometheus monitor prometheus itself #450

geerlingguy opened this issue Oct 14, 2022 · 1 comment

Comments

@geerlingguy
Copy link
Owner

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 to false.

Not only do I propose it, I'm about to push the variable to the project, and then this will be done, heh.

@geerlingguy
Copy link
Owner Author

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant