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

[ethtool] Error - ethtool currently doesn't support more than 1000 entries, received 2479 #6913

Closed
mkrug1981 opened this issue Jan 16, 2020 · 5 comments · Fixed by #6975
Closed
Labels
bug unexpected problem or unintended behavior feature request Requests for new plugin and for new features to existing plugins upstream bug or issues that rely on dependency fixes
Milestone

Comments

@mkrug1981
Copy link

Relevant telegraf.conf:

[[inputs.ethtool]]
#   ## List of interfaces to pull metrics for
interface_include = ["eno1","ens3"]
#
#   ## List of interfaces to ignore when pulling metrics.
#interface_exclude = ["eth1"]

System info:

  • Telegraf 1.13.1 (git: HEAD 0c17572)
  • CentOS Linux release 7.6.1810 (Core)
  • Network Card "Ethernet controller: Mellanox Technologies MT27800 Family [ConnectX-5]"

Steps to reproduce:

  1. Just enable input plugin "ethtool" on a Mellanox 100Gbps NIC

Expected behavior:

The ethtool input plugin should be able to cope with the amount of values from a Mellanox NIC

Actual behavior:

Telegraf input plugin ethtool is not able to read all values from a Mellanox 100Gbps NIC. It looks like the Cards has 2480 values but plugin has a max value of 1000 entries. See Error below

Additional info:

$ telegraf --config telegraf.conf --input-filter ethtool --test
2020-01-16T10:54:44Z I! Starting Telegraf 1.13.1
2020-01-16T10:54:44Z D! [agent] Initializing plugins
2020-01-16T10:54:44Z E! [inputs.ethtool] Error in plugin: ens3 stats: ethtool currently doesn't support more than 1000 entries, received 2479

Count the NIC values provided from ethtool

ethtool -S ens3 | wc -l
2480
@danielnelson
Copy link
Contributor

I opened safchain/ethtool#29 upstream.

@danielnelson danielnelson added bug unexpected problem or unintended behavior feature request Requests for new plugin and for new features to existing plugins upstream bug or issues that rely on dependency fixes labels Jan 17, 2020
@ykuksenko
Copy link

This was bumped to 4096 in safchain/ethtool#28 - though someone also wanted this to go to 8192.

@danielnelson
Copy link
Contributor

Thanks for the update @ykuksenko, I'm going to schedule this for 1.13.3 and I'll merge the latest ethtool once we get a bit closer to the date, in case we get another update.

@ykuksenko
Copy link

This is now 8192 in safchain/ethtool#30.

@danielnelson
Copy link
Contributor

@ykuksenko Added your update to safchain/ethtool for 1.13.4, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior feature request Requests for new plugin and for new features to existing plugins upstream bug or issues that rely on dependency fixes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants