Skip to content

Conversation

@GabrielYamin
Copy link

@GabrielYamin GabrielYamin commented Jan 28, 2026

Summary by CodeRabbit

  • Documentation
    • Added configuration parameter documentation for S3 output encryption options: sse and sse_kms_key_id supporting SSE-S3, SSE-KMS, and DSSE-KMS encryption modes.
    • Included configuration examples demonstrating SSE-KMS encryption setup for S3 output plugin across multiple deployment scenarios.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Gabriel Yamin <gabrielyamin98@gmail.com>
@GabrielYamin GabrielYamin requested review from a team and eschabell as code owners January 28, 2026 14:47
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

📝 Walkthrough

Walkthrough

The S3 output plugin documentation is updated with two new public configuration parameters (sse and sse_kms_key_id) describing server-side encryption options. Example Fluent Bit configuration blocks are added to demonstrate SSE-KMS encryption usage in both general and PutObject scenarios.

Changes

Cohort / File(s) Summary
S3 Output Plugin Documentation
pipeline/outputs/s3.md
Added documentation for sse and sse_kms_key_id configuration parameters covering SSE-S3, SSE-KMS, and DSSE-KMS encryption options. Inserted example Fluent Bit configuration blocks (YAML and conf formats) demonstrating SSE-KMS setup for general usage and PutObject operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

4.2.2

Suggested reviewers

  • esmerel

Poem

🐰 A document grows, encryption takes flight,
SSE and KMS shine oh-so-bright!
Examples now bloom in YAML and conf,
S3 secrets safe—no encryption gone wrong! 🔐

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: s3: SSE configuration docs' directly and concisely summarizes the main change: adding SSE configuration documentation to the S3 output plugin. It accurately reflects the primary objective of the PR.
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.


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.

@patrick-stephens
Copy link
Contributor

@GabrielYamin I think this is functionality waiting for the PR to be merged right?

@GabrielYamin
Copy link
Author

GabrielYamin commented Jan 28, 2026

@patrick-stephens, The BL PR is still in draft, I'll be opening it for review very soon.
fluent/fluent-bit#11410
Thank you for the fast response!!!

@GabrielYamin
Copy link
Author

PR is ready for review @patrick-stephens
Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants