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

displacement on round_interval for improved remote monitoring #10406

Closed
poulhs opened this issue Jan 8, 2022 · 3 comments · Fixed by #10545
Closed

displacement on round_interval for improved remote monitoring #10406

poulhs opened this issue Jan 8, 2022 · 3 comments · Fixed by #10545
Assignees
Labels
feature request Requests for new plugin and for new features to existing plugins

Comments

@poulhs
Copy link

poulhs commented Jan 8, 2022

Feature Request

Make it possible to start checking on intervals that are NOT round_interval - most useful for measuring remote response.

Proposal:

Implement a displacement that will be added to round_interval.

Current behavior:

interval = "10s"
collection_jitter = "0s"
round_interval = true

causes collection to start at :00, :10, :20 etc

Desired behavior:

interval = "10s"
displacement = "5s"
collection_jitter = "0s"
round_interval = true

causes collection to start at :05, :15, :25 etc

Use case:

To provide the measurements for SLA reporting, we have several servers used for remote monitoring.

To simplify, assume a customers web-service. To monitor it we configure telegraf on two different servers.
Both of these telegraf polls the web-service. The reason for having two servers is that we always have monitoring of the service - even when after patching the telegraf-servers (obviously assuming that we do not patch them at the same time), or having any other downtime on one of the telegraf-servers.

In normal situations (i.e. when both telegraf-servers are up), we would like to poll the web-service (almost) equidistantly. This cannot be achieve with todays configuration. With a displacement that would be easy and this would also reduce the time to detect that a service is down.

@poulhs poulhs added the feature request Requests for new plugin and for new features to existing plugins label Jan 8, 2022
@unk1nd
Copy link

unk1nd commented Jan 9, 2022

Second this

@poulhs
Copy link
Author

poulhs commented Jan 27, 2022

same as #1319 - Add time offset option to input plugins
but have not duplicated info given here.

@srebhan
Copy link
Member

srebhan commented Jan 28, 2022

Hey everyone, can you please test my PR #10545 implementing the requested feature and provide feedback in the PR? You can use the artifacts built by CI... Looking forward to your comments!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants