docs: administration: document adaptive flush intervals and FIPS mode - #2636
Merged
eschabell merged 2 commits intoAug 5, 2026
Merged
Conversation
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughThe documentation adds adaptive flush settings and OpenSSL FIPS mode references for command-line, classic configuration, and YAML service configuration. ChangesAdaptive flush documentation
FIPS mode documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@administration/configuring-fluent-bit/yaml/service-section.md`:
- Around line 69-81: Update the adaptive-flush example’s service configuration
so its documented behavior matches the multiplier of 2: adjust the baseline
flush value in the YAML under the “fluent-bit.yaml” tab, while preserving the
existing adaptive min/max interval and high-pressure settings.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: fe2c482f-64e2-49a3-8da5-4c52ae786852
📒 Files selected for processing (3)
administration/configuring-fluent-bit.mdadministration/configuring-fluent-bit/classic-mode/configuration-file.mdadministration/configuring-fluent-bit/yaml/service-section.md
Add the eight flush.adaptive service keys and security.fips_mode to the YAML and classic service configuration tables, with sections describing how the adaptive interval responds to chunk backpressure and what FIPS mode requires and changes. Document the matching --enable-fips command line flag. The adaptive flush example set flush to 1 while the surrounding text described an idle interval of three seconds. Because the idle pressure level multiplies flush by 2, that configuration produced a two second idle interval and never reached the configured max_interval of 3. Note: this update is from code merges without docs PR. Signed-off-by: Eric D. Schabell <eric@schabell.org>
eschabell
force-pushed
the
erics_admin_flush_fips_updates
branch
from
July 31, 2026 16:09
8afdfc0 to
d83738f
Compare
Collaborator
Author
|
@coderabbitai review |
Contributor
✅ Action performedReview finished.
|
Collaborator
Author
|
@patrick-stephens ready for review! |
patrick-stephens
approved these changes
Aug 5, 2026
patrick-stephens
left a comment
patrick-stephens
left a comment
Contributor
There was a problem hiding this comment.
Curious what happens when we use packages still built against OpenSSL 1.x
Signed-off-by: Eric D. Schabell <eric@schabell.org>
patrick-stephens
pushed a commit
that referenced
this pull request
Aug 7, 2026
…#2636) Add the eight flush.adaptive service keys and security.fips_mode to the YAML and classic service configuration tables, with sections describing how the adaptive interval responds to chunk backpressure and what FIPS mode requires and changes. Document the matching --enable-fips command line flag. The adaptive flush example set flush to 1 while the surrounding text described an idle interval of three seconds. Because the idle pressure level multiplies flush by 2, that configuration produced a two second idle interval and never reached the configured max_interval of 3. Note: this update is from code merges without docs PR. Signed-off-by: Eric D. Schabell <eric@schabell.org> Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the eight flush.adaptive service keys and security.fips_mode to the YAML and classic service configuration tables, with sections describing how the adaptive interval responds to chunk backpressure and what FIPS mode requires and changes. Document the matching --enable-fips command line flag.
Note: this update is from code merges without docs PR.
Summary by CodeRabbit
--enable-fipscommand-line option, availability, and FIPS requirements.