Skip to content
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

out_azure_logs_ingestion: create stream_name parameter #1305

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jlaundry
Copy link

@jlaundry jlaundry commented Feb 9, 2024

Documentation update for fluent/fluent-bit#8472

Signed-off-by: Jed Laundry <jlaundry@jlaundry.com>
@@ -33,6 +33,7 @@ To get more details about how to setup these components, please refer to the fol
| client\_secret| _Required_ - The client secret of the AAD application ([App Secret](https://docs.microsoft.com/en-us/azure/active-directory/develop/howto-create-service-principal-portal#option-2-create-a-new-application-secret)). ||
| dce\_url | _Required_ - Data Collection Endpoint(DCE) URL. ||
| dcr\_id | _Required_ - Data Collection Rule (DCR) immutable ID (see [this document](https://learn.microsoft.com/en-us/azure/azure-monitor/logs/tutorial-logs-ingestion-portal#collect-information-from-the-dcr) to collect the immutable id) ||
| stream\_name | _Required_ - Data Collection Rule (DCR) stream name (see [this document](https://learn.microsoft.com/en-us/azure/azure-monitor/logs/tutorial-logs-ingestion-portal#collect-information-from-the-dcr) to collect the stream name) ||
Copy link
Contributor

Choose a reason for hiding this comment

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

Checking the link: it doesn't mention streams elaborately (only mentioned once in doc and once in a JSON).
Should we rethink how best to document this property?

@jlaundry jlaundry requested review from a team as code owners November 28, 2024 01:43
@lecaros lecaros added the dco DCO failing label Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants