docs: administration: configuring: document --dry-run flag and property validation#2521
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughDocumentation update adding a new section explaining Fluent Bit's Changes
Suggested labels
Suggested reviewers
Poem
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@administration/configuring-fluent-bit.md`:
- Line 43: Update the sentence about prior behavior to avoid "failed silently at
startup" and instead state that unknown or misspelled plugin property names were
previously surfaced at runtime rather than during validation; specifically edit
the line referencing Fluent Bit 4.2 and `--dry-run` (and the example mentioning
a `dummy` input) so it reads that prior to 4.2 such property errors would only
surface at runtime, whereas `--dry-run` now performs full property validation
and catches them earlier.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 1739266b-13ea-44f4-a293-eaf9cac2959d
📒 Files selected for processing (1)
administration/configuring-fluent-bit.md
…ty validation
- Add --dry-run subsection under the CLI section explaining validation behavior
- Note that Fluent Bit 4.2+ catches unknown plugin properties at dry-run time
(previously these only failed at runtime)
- Based on commits: 7e49f8f8b + 8b86fdeed — bin: run additional validation for --dry-run
Signed-off-by: Eric D. Schabell <eric@schabell.org>
f917f05 to
6cacd42
Compare
|
@patrick-stephens ready for review! |
Summary by CodeRabbit
--dry-runcommand-line flag, covering how to validate configuration files without starting the service. Includes expected success output, exit code behavior (0 on success, non-zero on failure), and full property validation capabilities introduced in version 4.2, with example error messages.