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-lua + prometheus-node-exporter-lua plugin #24101

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Kevinjil
Copy link

@Kevinjil Kevinjil commented May 8, 2024

The ethtool-lua library is a partial re-implementation of the ethtool. The goal is to provide the CLI queries and configuration options as a Lua API.

The reason for starting this library, was the desire for a nice and efficient way to query DSA switch statistics in the prometheus-node-exporter-lua on OpenWRT devices. Existing suggestions around the internet focussed mainly on calling the ethtool CLI program and parsing the output. This is neither elegant nor efficient, as the collection time for this implementation was 300% higher on my rtl838x based switch running OpenWRT.

The ethtool-lua library is a partial re-implementation of the ethtool.
The goal is to provide the CLI queries and configuration options as a
Lua API.

The reason for staring this library, was the desire for a nice and
efficient way to query DSA switch statistics in the
prometheus-node-exporter-lua on OpenWRT devices. Existing suggestions
around the internet focussed mainly on calling the ethtool CLI program
and parsing the output. This is neither elegant nor efficient, as the
collection time for this implementation was 300% higher on my rtl838x
based switch running OpenWRT.

This package can be easily compiled and packaged for OpenWRT, as it was
initially created as an OpenWRT package.

Signed-off-by: Kevin Jilissen <info@kevinjilissen.nl>
Based on ethtool-lua library, add interface statistics to the
prometheus-node-exporter.

Signed-off-by: Kevin Jilissen <info@kevinjilissen.nl>
The package was defined, but was never actually built.

Signed-off-by: Kevin Jilissen <info@kevinjilissen.nl>
@Kevinjil
Copy link
Author

@champtar Would you mind reviewing these changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant