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

Add additional metrics to the internal plugin #5900

Closed
zoonage opened this issue May 22, 2019 · 4 comments
Closed

Add additional metrics to the internal plugin #5900

zoonage opened this issue May 22, 2019 · 4 comments
Labels
feature request Requests for new plugin and for new features to existing plugins

Comments

@zoonage
Copy link

zoonage commented May 22, 2019

Feature Request

Whilst troubleshooting a Telegraf issue earlier I realised it would have been nice to know how many goroutines were running at a given point in time. This can be found by runtime.NumGoroutine(). I wanted to kickstart a discussion about other metrics that people might want to add to the internal plugin.

Proposal:

Add a metric from runtime.NumGoroutine() to the internal plugin.

Current behavior:

Telegraf does not currently support this metric

Desired behavior:

To be able to see the number of goroutines that Telegraf is using

Use case:

Helps troubleshoot issues with Telegraf

@danielnelson
Copy link
Contributor

Seems like a good idea, can you open a pull request?

@danielnelson danielnelson added the feature request Requests for new plugin and for new features to existing plugins label May 24, 2019
@russorat
Copy link
Contributor

russorat commented Jul 31, 2019

Which measurement would you put numgoroutine in? it doesn't seem to fit well into any of the existing ones. maybe adding an internal_runtime measurement?

Other things that might be useful to have (from: https://github.com/influxdata/influxdb/blob/master/prometheus/influx.go#L27):

  • "version": build.Version,
  • "commit": build.Commit,
  • "build_date": build.Date,
  • "os": runtime.GOOS,
  • "arch": runtime.GOARCH,
  • "cpus": strconv.Itoa(runtime.NumCPU()),

connect: #5958

@zoonage
Copy link
Author

zoonage commented Aug 5, 2019

The number of CPUs is already reported via the n_cpus field on the system measurement

@danielnelson
Copy link
Contributor

We can add num_goroutine to internal_agent. Let's skip build date and commit for now, they impose requirements on the build. Things like number os and arch would make more sense to me to add them to the system plugin potentially with additional platform information.

@zoonage zoonage closed this as completed Jun 10, 2021
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

No branches or pull requests

3 participants