Skip to content

pipeline: outputs: azure_blob: add log_key to outputs #1540

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tomekwilk
Copy link

@tomekwilk tomekwilk commented Jan 2, 2025

Matching fluent/fluent-bit PR fluent/fluent-bit#9791

Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @tomekwilk! Suggested a minor edit, but LGTM otherwise.

@@ -26,6 +26,7 @@ We expose different configuration properties. The following table lists all the
| sas\_token | Specify the Azure Storage shared access signatures to authenticate against the service. This configuration property is mandatory when `auth_type` is `sas`. | |
| container\_name | Name of the container that will contain the blobs. This configuration property is mandatory | |
| blob\_type | Specify the desired blob type. Fluent Bit supports `appendblob` and `blockblob`. | appendblob |
| log\_key | By default, the whole log record will be sent to blob storage. If you specify a key name with this option, then only the value of that key will be sent. | |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| log\_key | By default, the whole log record will be sent to blob storage. If you specify a key name with this option, then only the value of that key will be sent. | |
| log\_key | By default, the entire log record is sent to blob storage. If you specify a key name with this option, only the value of that key is sent. | |

Signed-off-by: Thomas Wilk <thomas.wilk@cohesity.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants