-
Notifications
You must be signed in to change notification settings - Fork 1.7k
out_azure_blob: added buffering to azure blob #10263
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
out_azure_blob: added buffering to azure blob #10263
Conversation
Signed-off-by: Tanmaya Panda <[email protected]>
…ded-buffering-to-azure-blob
Signed-off-by: Tanmaya Panda <[email protected]>
Signed-off-by: Tanmaya Panda <[email protected]>
Signed-off-by: Tanmaya Panda <[email protected]>
Signed-off-by: Tanmaya Panda <[email protected]>
…ded-buffering-to-azure-blob
…ature/added-buffering-to-azure-blob
…ded-buffering-to-azure-blob
docs PR # fluent/fluent-bit-docs#1632 |
We have been running this version in many clusters for the past 24 hours and it's working great |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have reviewed and added more requests for changes, most of them are around codyng style.
Once that is ready, please reduce the number of commits to the minimum necessary
Signed-off-by: Tanmaya Panda <[email protected]>
dc7a6c8
to
f5b094a
Compare
This pull request introduces significant enhancements to the Azure Blob plugin, focusing on adding buffering capabilities and improving URI handling.
Buffering Enhancements
buffering_enabled
,buffer_dir
,store_dir_limit_size
, andcurrent_buffer_size
in theflb_azure_blob
struct. These changes enable the plugin to buffer data before uploading to Azure Blob Storage.appendblob
blob type, as it is incompatible. An error is logged if this configuration is detected.URI Handling Improvements
azb_block_blob_uri
andazb_block_blob_uri_commit
functions to include a newrandom_str
parameter for generating more unique URIs for block blobs. This ensures better handling of concurrent uploads.Enter
[N/A]
in the box, if an item is not applicable to your change.Testing
Before we can approve your change; please submit the following in a comment:
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
ok-package-test
label to test for all targets (requires maintainer to do).Documentation
Backporting
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.