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

Telegraf Influxdb input: Add missing data from debug vars #15437

Closed
cpinflux opened this issue May 31, 2024 · 2 comments · Fixed by #15438
Closed

Telegraf Influxdb input: Add missing data from debug vars #15437

cpinflux opened this issue May 31, 2024 · 2 comments · Fixed by #15438
Assignees
Labels
feature request Requests for new plugin and for new features to existing plugins

Comments

@cpinflux
Copy link

cpinflux commented May 31, 2024

Use Case

/debug/vars provides this information:

"build": {"Branch":"unknown","Build Time":"","Commit":"d97e21d4473164bb3ba0055ecd1971f2529b6d82","Version":"v1.11.5"},
"cmdline": ["influxd"],
"crypto": {"FIPS":false,"ensureFIPS":false,"implementation":"Go","passwordHash":"bcrypt;cost=10"},

But as there is no explicit struct defined for these, unlike system and memstats, and they are not preformatted with tags and fields like the rest of the data, they are ignored by telegraf.

It would be very handy to know what version of InfluxDB is running over a large estate

Expected behavior

These three lines should be either converted into three different measurements or added into the influxdb base measurement that currently contains only a custom "n_shards" field.

I prefer the influxdb measurement route to keep n_shards company. If this were the path taken I would probably also suggest duplicating the influxdb_system output into there as well for neatness.

Actual behavior

Data is not present in output at all

Additional info

telegraf_influxdb.txt
debug_vars.txt

@cpinflux cpinflux added the feature request Requests for new plugin and for new features to existing plugins label May 31, 2024
@srebhan srebhan self-assigned this May 31, 2024
@srebhan
Copy link
Member

srebhan commented May 31, 2024

@cpinflux please test the binary in PR #15438, available as soon as CI finished the tests, and let me know if this fixes the issue!

@srebhan
Copy link
Member

srebhan commented May 31, 2024

Successfully tested as stated in the PR...

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.

2 participants