search

influxdata / telegraf

Agent for collecting, processing, aggregating, and writing metrics, logs, and other arbitrary data.
https://influxdata.com/telegraf
MIT License
14.65k stars 5.59k forks source link

Input: Bind - strconv.ParseInt : value out of range #15866

Closed rumanmaharjan closed 6 days ago

rumanmaharjan commented 1 month ago

telegraf configuration::

[[inputs.bind]]
urls = ["http://localhost:8053/xml/v3"]

error message:

[inputs.bind] Error in plugin: unable to decode XML document: strconv.ParseInt: parsing "18446744073709546041": value out of range

telegraf version v1.31.2

related to #6059

srebhan commented 1 month ago

@rumanmaharjan would you please care to create a sensible issue description pasting your actual current error and a configuration to reproduce the issue!?

rumanmaharjan commented 1 month ago

telegraf configuration::

[[inputs.bind]] urls = ["http://localhost:8053/xml/v3"]

log: [inputs.bind] Error in plugin: unable to decode XML document: strconv.ParseInt: parsing "18446744073709546041": value out of range

srebhan commented 1 month ago

@rumanmaharjan we do have issue-templates for a reason... No I need to spend the time in formatting the issue instead of spending the time fixing the bug...

Anyway, which version of Bind do you use? Is this happening reliably? If not, how can we provoke the issue?

rumanmaharjan commented 1 month ago

bind version : BIND 9.11.4 telegraf version : Telegraf 1.31.2

you can provoke the issue by increasing the value like : counter name="RecursClients"18446744073709546851

rumanmaharjan commented 1 month ago

@srebhan , are you able to recreate the issue ? let me know if any extra details required.

rumanmaharjan commented 1 month ago

Hello , we can try with downgrade or upgrade version.

srebhan commented 3 weeks ago

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

Be aware that you need to set report_counters_as_int to false for the large numbers you show above which will change the output field-type to unsigned integers! This might lead to type conflicts on the output side, so handle with care!

srebhan commented 2 weeks ago

@rumanmaharjan any chance you test this?

rumanmaharjan commented 2 weeks ago

Hi @srebhan thank you for update. We have planned for next week. I will update you.

rumanmaharjan commented 1 week ago

Hi @srebhan , i am not able to download package from PR https://github.com/influxdata/telegraf/pull/16015 .

srebhan commented 1 week ago

@rumanmaharjan I retriggered building of the packages. Please note, those packages will be deleted after 3 days IIRC so it would be good if you could at least download the package before it is removed again...

rumanmaharjan commented 1 week ago

@srebhan , thank you for packages. I have installed on one production server and it looks good.