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

Max ttl for gauges in statsd input plugin #8348

Closed
davidfaulkner12 opened this issue Nov 2, 2020 · 1 comment · Fixed by #8509
Closed

Max ttl for gauges in statsd input plugin #8348

davidfaulkner12 opened this issue Nov 2, 2020 · 1 comment · Fixed by #8509
Assignees
Labels
area/statsd feature request Requests for new plugin and for new features to existing plugins good first issue This is a smaller issue suited for getting started in Telegraf, Golang, and contributing to OSS. help wanted Request for community participation, code, contribution

Comments

@davidfaulkner12
Copy link

Feature Request

Proposal:

Allow for setting "gaugesMaxTTL" option in the statsd input plugin. Essentially, this from the core statsd repo:

https://github.com/statsd/statsd/pull/599/files

Current behavior:

If deleteGauges is set to "False" then once a gauge metric is reported it is reported for the life of the telegraf server. There is no way to set a TTL value where if the gauge metric is not received at the ingress endpoint after a certain time it is reset to 0.

Desired behavior:

An optional gaugesMaxTTL option is available for the statsd input -- this has exactly the same effect as the deleteGauges option, except that it only executes that logic after no input has been received for the value specified in gaugesMaxTTL.

Use case:

Telegraf server A on host X is taking input from service B and service C. Both of these services are reporting metrics via a gauge with "deleteGauges" set to False -- let's assume this is required w/o getting distracted by details. :-)

Now, service C is turned down by some load-managing automation. However, telegraf A will still report the last gauge metrics to Influx until restarted, with no way of ever reflecting the fact that service C is permanently gone.

@davidfaulkner12 davidfaulkner12 added the feature request Requests for new plugin and for new features to existing plugins label Nov 2, 2020
@ssoroka
Copy link
Contributor

ssoroka commented Nov 2, 2020

This sounds like a great addition

@ssoroka ssoroka added good first issue This is a smaller issue suited for getting started in Telegraf, Golang, and contributing to OSS. help wanted Request for community participation, code, contribution labels Nov 2, 2020
@ivorybilled ivorybilled self-assigned this Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/statsd feature request Requests for new plugin and for new features to existing plugins good first issue This is a smaller issue suited for getting started in Telegraf, Golang, and contributing to OSS. help wanted Request for community participation, code, contribution
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants