-
Notifications
You must be signed in to change notification settings - Fork 544
input: network_io_metrics: general documentation improvements #2285
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
input: network_io_metrics: general documentation improvements #2285
Conversation
- Add "Metrics reported" section with table of output metrics - Add Linux platform availability note - Add hyperlink to Node Exporter metrics plugin - Add verbose and test_at_init options to YAML and conf examples - Standardize config parameter names to lowercase - Sort configuration parameters alphabetically - Fix grammar: "returns something the following" -> "returns output similar to the following" - Improve consistency with other input plugin docs Fixes fluent#2284. Signed-off-by: Eric D. Schabell <eric@schabell.org>
WalkthroughThe documentation for the network-io-metrics input plugin is updated to standardize configuration parameter naming from CamelCase to snake_case, add a new "Metrics reported" section, include a Linux-only note, and refresh examples and CLI output formatting. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🔇 Additional comments (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
AI review passed, merging. |
Fixes #2284.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.