in_ebp: add three missing config option descriptions. Fixes #11221.
in_ebp: add three missing config option descriptions for:
ringbuf_map_name - should describe the BPF ring buffer map name
poll_ms - should describe the polling interval in milliseconds
Trace - should describe the trace type to enable (for example, bind, malloc, signal)
Fixes #11221
Testing Before we can approve your change; please submit the following in a comment:
- [N/A ] Example configuration file for the change
- [N/A ] Debug log output from testing the change
- [N/A ] Attached Valgrind output that shows no leaks or memory corruption was found
- [N/A ] Run local packaging test showing all targets (including any new ones) build.
- [N/A ] Set
ok-package-testlabel to test for all targets (requires maintainer to do).
Documentation
- [X] Documentation required for this feature Issue: https://github.com/fluent/fluent-bit-docs/issues/2239 Doc PR: https://github.com/fluent/fluent-bit-docs/pull/2240
Backporting
- [N/A ] Backport to latest stable release.
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.
Summary by CodeRabbit
- Documentation
- Added user-facing descriptions for eBPF plugin configuration options:
- ring buffer map name: clarifies which ring buffer to read events from.
- polling interval: clarifies the interval (ms) used to collect events.
- trace settings: explains how to specify eBPF traces and that the option can be provided multiple times.
- Added user-facing descriptions for eBPF plugin configuration options:
✏️ Tip: You can customize this high-level summary in your review settings.
Walkthrough
Added description strings to each config_map entry in the eBPF input plugin and updated the public config map structure to include a description member so the options ringbuf_map_name, poll_ms, and Trace now have help text.
Changes
| Cohort / File(s) | Summary |
|---|---|
eBPF plugin config plugins/in_ebpf/in_ebpf.c |
Added description strings to three config_map entries: ringbuf_map_name ("Set the name of the eBPF ring buffer map to read events from"), poll_ms ("Set the polling interval in milliseconds for collecting events"), and Trace ("Set the eBPF trace to enable (for example, bind, malloc, signal). Can be set multiple times"). Added EOF marker comment in the array. |
Public config map declaration include/...flb_config_map* |
Public struct flb_config_map now includes a description (const char *) member so config entries can carry descriptive strings; initializers updated accordingly. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
- Focus review on ensuring all
struct flb_config_mapinitializers across the codebase were updated to include the newdescriptionfield. - Verify build and help output for plugins that use
config_map.
Possibly related PRs
- fluent/fluent-bit#11226 — Adds missing description strings to plugin
config_mapentries (same change pattern). - fluent/fluent-bit#11240 — Adds missing config parameter description strings to plugin
config_mapentries (related plugin changes).
Suggested reviewers
- edsiper
- cosmo0920
Poem
🐰 I hopped through code to leave a clue,
Ringbuf, poll, and Trace — now told true.
Tiny strings that softly gleam,
Guiding users like a dream. ✨
Pre-merge checks and finishing touches
❌ Failed checks (1 warning)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Docstring Coverage | ⚠️ Warning | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | You can run @coderabbitai generate docstrings to improve docstring coverage. |
✅ Passed checks (4 passed)
| Check name | Status | Explanation |
|---|---|---|
| Linked Issues check | ✅ Passed | The PR successfully implements all three required config option descriptions (ringbuf_map_name, poll_ms, Trace) as specified in issue #11221. |
| Out of Scope Changes check | ✅ Passed | All changes are directly scoped to adding the three missing descriptions; no extraneous modifications were introduced. |
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | The title accurately reflects the main change: adding three missing config option descriptions to the in_ebpf plugin, and it references the fixed issue. |
✨ Finishing touches
- [ ] 📝 Generate docstrings
🧪 Generate unit tests (beta)
- [ ] Create PR with unit tests
- [ ] Post copyable unit tests in a comment
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 @coderabbitai help to get the list of available commands and usage tips.
@coderabbitai review
✅ Actions performed
Review triggered.
Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
The CIFuzz crashing here is not due to this PR, it is due to some infrastructure updates on OSS-Fuzz. Ref: https://github.com/fluent/fluent-bit/issues/11227
let's cleanup the history of this PR (commits)
@edsiper fixed the commit messages.
@edsiper squashed commits.