Skip to content

out_azure_blob: document path templating#2302

Merged
eschabell merged 1 commit intofluent:masterfrom
nberlee:master
Apr 1, 2026
Merged

out_azure_blob: document path templating#2302
eschabell merged 1 commit intofluent:masterfrom
nberlee:master

Conversation

@nberlee
Copy link
Copy Markdown
Contributor

@nberlee nberlee commented Dec 16, 2025

Summary by CodeRabbit

  • Documentation
    • Improved Azure Blob output docs: reformatted parameter table for clarity, clarified the path parameter to explicitly support path templating with examples, and added a dedicated "Path templating" section listing supported tokens (strftime directives, millisecond/nanosecond placeholders, UUID, tag references) plus YAML and concrete path examples. No other behavior or defaults changed.

@nberlee nberlee requested review from a team as code owners December 16, 2025 09:42
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Dec 16, 2025

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f3d1904e-7fdb-4554-ae5a-bf53f092781b

📥 Commits

Reviewing files that changed from the base of the PR and between be354dd and ec5d8f7.

📒 Files selected for processing (1)
  • pipeline/outputs/azure_blob.md
✅ Files skipped from review due to trivial changes (1)
  • pipeline/outputs/azure_blob.md

📝 Walkthrough

Walkthrough

The Azure Blob output documentation now documents path templating: the path parameter description was rewritten to explain templating and examples, and a new "Path templating" section lists supported tokens/syntax and shows YAML and concrete examples of resolved paths.

Changes

Cohort / File(s) Summary
Azure Blob Documentation
pipeline/outputs/azure_blob.md
Reflowed table formatting; rewrote the path parameter description to explicitly state it supports path templating; added a new "Path templating" section describing supported tokens (strftime directives, %3N, %9N/%L, $UUID, $TAG, $TAG[n]) with YAML examples and a concrete tag→path output example. No behavioral defaults or other parameters changed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

5.0

Suggested reviewers

  • eschabell
  • cosmo0920

Poem

🐰 I hop through docs with tidy cheer,
Tokens and timestamps drawing near.
%Y/%m/%d and $UUID gleam bright,
$TAG trims paths and keeps them right.
🥕✨

🚥 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 'out_azure_blob: document path templating' clearly and specifically summarizes the main change - documentation of path templating functionality for the Azure Blob output plugin.
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

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 eschabell self-assigned this Dec 16, 2025
@eschabell eschabell added the waiting-on-review Waiting on a review from mainteners label Dec 16, 2025
@eschabell eschabell requested review from cosmo0920 and removed request for alexakreizinger January 27, 2026 10:11
@eschabell eschabell added the conflict Waiting on conflict to be resolved by contributor label Mar 12, 2026
@eschabell
Copy link
Copy Markdown
Collaborator

@nberlee rebase to pull in the changes that now conflict with your branch and we can review.

@nberlee
Copy link
Copy Markdown
Contributor Author

nberlee commented Mar 19, 2026

I will put this on draft, there is not a reason for review at this time as the feature is still not merged.

@nberlee nberlee marked this pull request as draft March 19, 2026 09:22
(refs fluent/fluent-bit#11178)

Signed-off-by: Nico Berlee <nico.berlee@on2it.net>
@nberlee nberlee marked this pull request as ready for review April 1, 2026 07:52
@nberlee nberlee requested a review from eschabell as a code owner April 1, 2026 07:52
@nberlee
Copy link
Copy Markdown
Contributor Author

nberlee commented Apr 1, 2026

@eschabell feature is merged this morning, and I rebased. Ready to be reviewed

Copy link
Copy Markdown
Collaborator

@eschabell eschabell left a comment

Choose a reason for hiding this comment

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

The vale issue can be solved but is not crucial, merging!

@eschabell eschabell merged commit 761a52a into fluent:master Apr 1, 2026
8 checks passed
@eschabell eschabell added 5.0.2 and removed conflict Waiting on conflict to be resolved by contributor waiting-on-review Waiting on a review from mainteners labels Apr 1, 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.

2 participants