Processor plugin docs updates#2369
Conversation
- Add missing `action` configuration parameter - Sort configuration parameters alphabetically - Add Oxford comma in converted_type description Applies to fluent#2368 Signed-off-by: Eric D. Schabell <eric@schabell.org>
- Sort configuration parameters alphabetically - Fix hash example to say SHA-256 (not SHA-1) to match implementation Applies to fluent#2368 Signed-off-by: Eric D. Schabell <eric@schabell.org>
- Sort configuration parameters alphabetically - Fix context default value typo (metrics_name -> metric_name) - Fix context possible value (delete_label -> delete_label_value) Applies to fluent#2368 Signed-off-by: Eric D. Schabell <eric@schabell.org>
- Add missing `debug` configuration parameter - Sort all configuration parameter tables alphabetically - Fix swapped min_spans/max_spans descriptions in span count condition - Add Default column to main config table - Normalize table separator styles Applies to fluent#2368 Signed-off-by: Eric D. Schabell <eric@schabell.org>
📝 WalkthroughWalkthroughDocumentation updates for four Fluent Bit processor plugins: content-modifier, labels, metrics-selector, and sampling. Changes include adding missing configuration parameters, reordering configuration tables, updating default values, and refining parameter descriptions to align with current functionality. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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 |
Summary of fixes applied:
content-modifier.mdissues Found: Missing action param, unsortedlabels.mdissues Found: Unsorted, wrong hash algorithm in examplesampling.mdissues Found: Missing debug param, unsorted tables, swapped min/max descriptionsFixes #2368.
Summary by CodeRabbit