Skip to content

docs: administration: scheduling-and-retries: document retry deferral under flush limit - #2764

Merged
eschabell merged 1 commit into
fluent:masterfrom
eschabell:erics_scheduling_retries_flush_in_flight
Oct 9, 2026
Merged

eschabell merged 1 commit into
fluent:masterfrom
eschabell:erics_scheduling_retries_flush_in_flight

Conversation

@eschabell

@eschabell eschabell commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator
  • Document that a retry is rescheduled without counting against
    Retry_Limit when too many flush requests are in flight

Signed-off-by: Eric D. Schabell eric@schabell.org

Summary by CodeRabbit

  • Documentation
    • Clarified that in Fluent Bit v5.1.3 and later, a retry is rescheduled without counting against Retry_Limit when too many flush requests are in flight.

… under flush limit

  - Document that a retry is rescheduled without counting against
    Retry_Limit when too many flush requests are in flight

Signed-off-by: Eric D. Schabell <eric@schabell.org>
@eschabell
eschabell requested review from a team and patrick-stephens as code owners October 6, 2026 19:29
@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4dc7f843-0369-4c64-aa11-cdf86f9e519f
📥 Commits

Reviewing files that changed from the base of the PR and between e586da8 and 6c3b146.

📒 Files selected for processing (1)
  • administration/scheduling-and-retries.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Walkthrough

Walkthrough

The scheduling and retries guide adds a note that Fluent Bit v5.1.3 or later reschedules a retry without counting it against Retry_Limit when too many flush requests are in flight.

Changes

Retry scheduling documentation

Layer / File(s) Summary
Document retry limit behavior
administration/scheduling-and-retries.md
The guide documents that, in Fluent Bit v5.1.3 or later, a retry due while too many flush requests are in flight is rescheduled without counting against Retry_Limit.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 6c3b1

The documentation accurately describes retry deferral starting in v5.1.3; no actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the documentation change: retry deferral when the flush limit is reached.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@eschabell eschabell self-assigned this Oct 6, 2026
@eschabell eschabell added waiting-on-review Waiting on a review from mainteners 5.1.3 labels Oct 6, 2026
@eschabell

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

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.

@eschabell

Copy link
Copy Markdown
Collaborator Author

@patrick-stephens ready for review!

@eschabell eschabell removed the waiting-on-review Waiting on a review from mainteners label Oct 9, 2026
@eschabell
eschabell merged commit 7cd8a15 into fluent:master Oct 9, 2026
8 checks passed
@eschabell
eschabell deleted the erics_scheduling_retries_flush_in_flight branch October 9, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants