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

Make Timeout configurable via a flag #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ahurtaud
Copy link

This PR enables the possibility to override the collectd settings Timeout.
It will close #88

@brian-brazil
Copy link
Contributor

brian-brazil commented Nov 23, 2019 via email

@ahurtaud
Copy link
Author

I'm not sure about this, does having data with multiple different scrape intervals on a single target make sense for either collectd or Prometheus?

I think it is technically feasible.
However my question is, if this data is generated every 10 minutes. Will prometheus handle correctly the staleness if scrape_interval is 10m?

@brian-brazil
Copy link
Contributor

Scrape intervals above 2m are not recommended. What exactly is this data? It sounds like it may be a case for the textfile collector.

@ahurtaud
Copy link
Author

Scrape intervals above 2m are not recommended. What exactly is this data? It sounds like it may be a case for the textfile collector.

The metrics are MapR Volumes metrics.

It works for every metrics except the metrics for volumes (because from what MapR told us, they are "heavy to compute").

Metrics are collected from each node in the cluster so that administrators can use the data to monitor the cluster. In general, the collectd service collects metrics every 10 seconds. The exception is volume metrics which are collected every 10 minutes.

And we cannot configure collectd on MapR nodes... This is why we would need such features.

@stale stale bot added the stale label Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request - Make Timeout configurable via a flag
2 participants