Skip to content

Fixed resources, fluentd & fluent bit, and what is fluent bit doc sections under the About header#2386

Merged
eschabell merged 4 commits intofluent:masterfrom
eschabell:erics_about_section_updates
Feb 27, 2026
Merged

Fixed resources, fluentd & fluent bit, and what is fluent bit doc sections under the About header#2386
eschabell merged 4 commits intofluent:masterfrom
eschabell:erics_about_section_updates

Conversation

@eschabell
Copy link
Collaborator

@eschabell eschabell commented Feb 24, 2026

  - Update Amazon workshop URL to point to Fluent Bit pod logging page
  - Update section heading to reflect actual workshop content
  - Update description to accurately reflect CloudWatch Logs routing
  - Update Fluent Bit memory footprint from ~1 MB to ~450 KB
  - Update deployment stat to reflect 15 billion global deployments
  - Change "sub-project" to "graduated project" in front matter description

Fixes #2385

Summary by CodeRabbit

  • Documentation
    • Updated Fluent Bit deployment stat to "deployed over 15 billion times globally" and adjusted memory footprint to "Approximately 450 KB".
    • Revised workshop title and description to focus on pod-level Fluent Bit logging on Kubernetes and routing to CloudWatch Logs; updated embedded reference.
    • Updated project designation to "CNCF graduated project" under Fluentd.
  • Style
    • Added a new linting token ignore pattern to documentation analysis configuration.

  - Change "sub-project" to "graduated project" in front matter description

Applies to fluent#2385

Signed-off-by: Eric D. Schabell <eric@schabell.org>
…stats

  - Update Fluent Bit memory footprint from ~1 MB to ~450 KB
  - Update deployment stat to reflect 15 billion global deployments

Applies to fluent#2385

Signed-off-by: Eric D. Schabell <eric@schabell.org>
@eschabell eschabell self-assigned this Feb 24, 2026
@eschabell eschabell requested a review from a team as a code owner February 24, 2026 19:48
@eschabell eschabell added waiting-on-review Waiting on a review from mainteners 5.0 labels Feb 24, 2026
@eschabell eschabell force-pushed the erics_about_section_updates branch from d347473 to 51d55ee Compare February 24, 2026 19:51
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 24, 2026

Warning

Rate limit exceeded

@eschabell has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 21 minutes and 36 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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.

📥 Commits

Reviewing files that changed from the base of the PR and between a2c64b9 and 438cf05.

📒 Files selected for processing (1)
  • .vale.ini
📝 Walkthrough

Walkthrough

This PR updates about-site docs: revises Fluent Bit's CNCF classification, adjusts global deployment and memory-footprint figures, modifies a workshop entry to focus on Fluent Bit pod logging on Kubernetes with an updated embed URL, and adds a Vale TokenIgnores pattern.

Changes

Cohort / File(s) Summary
CNCF status
about/what-is-fluent-bit.md
Changed front-matter description from "CNCF sub-project" to "CNCF graduated project".
Metrics & comparison tables
about/fluentd-and-fluent-bit.md
Updated deployment wording to "deployed over 15 billion times globally" and changed Fluent Bit memory footprint entries from "Approximately 1 MB" to "Approximately 450 KB" in overview and detailed table.
Workshop / resources
about/resources.md
Renamed workshop title and updated description to "Fluent Bit pod logging on Kubernetes", switched routing reference to CloudWatch Logs, and replaced embedded source URL with a Fluent Bit setup page.
Linting config
.vale.ini
Added a new TokenIgnores pattern (\{%[^%]*%\}) to ignore templated tokens during Vale analysis.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • cosmo0920
  • patrick-stephens

Poem

🐇 I hopped through docs with careful paws,

Trimmed the bytes and fixed the laws,
A kinder stat, a clearer link,
The docs now sparkle — take a blink! ✨

🚥 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 accurately describes the main changes: updates to three documentation files (resources, fluentd & fluent bit, what is fluent bit) under the About section header.
Linked Issues check ✅ Passed All objectives from issue #2385 are met: deployment statistics updated to 15 billion, memory footprint corrected to 450 KB, workshop link and heading updated to pod logging content, routing descriptions clarified, and CNCF status changed to graduated project.
Out of Scope Changes check ✅ Passed All changes are directly related to the objectives in issue #2385. The .vale.ini update adds a token ignore pattern supporting documentation rendering; this is a supporting change for the documentation updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 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.

❤️ Share

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

@eschabell
Copy link
Collaborator Author

@patrick-stephens and @cosmo0920 ready for review!

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@about/fluentd-and-fluent-bit.md`:
- Line 25: Add inline citations for the "Approximately 450&nbsp;KB" memory
figure in the table row "| Memory       | Greater than 60&nbsp;MB |
Approximately 450&nbsp;KB |" by referencing Fluent Bit's official documentation
(docs.fluentbit.io and fluentbit.io) where the "minimal footprint" / 450 KB
number is stated; update the markdown to include one or two parenthetical or
footnote-style citations immediately after "Approximately 450&nbsp;KB" that
point to the exact Fluent Bit manual pages describing the minimal footprint.
- Line 13: The sentence "Production-grade solutions, with Fluent Bit deployed
over 15 billion times globally." needs a citation—edit that line in
about/fluentd-and-fluent-bit.md to turn the phrase "Fluent Bit" (or the whole
claim) into a Markdown link pointing to Fluent Bit's official site (e.g.
https://fluentbit.io/) so the deployment statistic is verifiable; ensure the
link uses standard Markdown link syntax and preserves the original wording.

In `@about/resources.md`:
- Around line 32-36: The embed block for the Fluent Bit workshop is missing its
closing tag and descriptive text; locate the opening embed with
URL="https://eksworkshop.com/docs/observability/logging/pod-logging/fluentbit-setup"
and add a short descriptive line (e.g., "AWS workshop covering Fluent Bit
pod-level logging on Kubernetes with CloudWatch Logs integration") followed by
the closing tag {% endembed %} so the block matches the pattern used by the
other embeds.
- Line 15: The embed tag uses an uppercase attribute "URL" which breaks GitBook
parsing; change the tag from {% embed
URL="https://o11y-workshops.gitlab.io/workshop-fluentbit/" %} to use a lowercase
attribute name so it reads {% embed
url="https://o11y-workshops.gitlab.io/workshop-fluentbit/" %} (update the embed
tag's attribute only, leaving the rest of the tag unchanged).

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4a18664 and 51d55ee.

📒 Files selected for processing (3)
  • about/fluentd-and-fluent-bit.md
  • about/resources.md
  • about/what-is-fluent-bit.md

Copy link
Contributor

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me but CodeRabbit flagged a few issues with the Gitbook setup that probably need resolving.

  - Update Amazon workshop URL to point to Fluent Bit pod logging page
  - Update section heading to reflect actual workshop content
  - Update description to accurately reflect CloudWatch Logs routing

Applies to fluent#2385

Signed-off-by: Eric D. Schabell <eric@schabell.org>
@eschabell eschabell force-pushed the erics_about_section_updates branch from 51d55ee to b95d857 Compare February 26, 2026 19:28
Signed-off-by: Eric D. Schabell <eric@schabell.org>
@eschabell eschabell force-pushed the erics_about_section_updates branch from a2c64b9 to 438cf05 Compare February 26, 2026 19:36
@eschabell
Copy link
Collaborator Author

@patrick-stephens can I get a final review, as I've fixed the bot issues and adjusted Vale settings to pass on gitbook embedded images (the URL thing).

@eschabell eschabell merged commit dc38f3c into fluent:master Feb 27, 2026
8 checks passed
@eschabell eschabell deleted the erics_about_section_updates branch February 27, 2026 07:42
@eschabell eschabell removed the waiting-on-review Waiting on a review from mainteners label Feb 28, 2026
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.

About section documentation needs updating

3 participants