-
Notifications
You must be signed in to change notification settings - Fork 539
out_azure_kusto: added workload identity #1631
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_kusto: added workload identity #1631
Conversation
Signed-off-by: Tanmaya Panda <[email protected]>
…s-for-buffering-commits Update azure_kusto.md for buffering commits
Signed-off-by: esmerel <[email protected]>
Signed-off-by: Tanmaya Panda <[email protected]>
Signed-off-by: Tanmaya Panda <[email protected]>
Signed-off-by: Tanmaya Panda <[email protected]>
| managed_identity_client_id | _Required if `tenant_id`, `client_id`, and `client_secret` are not set_ - The managed identity ID to authenticate with. Set to `SYSTEM` for system-assigned managed identity, or set to the MI client ID (GUID) for user-assigned managed identity. | | | ||
| ingestion_endpoint | _Required_ - The cluster's ingestion endpoint, usually in the form `https://ingest-cluster_name.region.kusto.windows.net | | ||
| tenant_id | _Required for service principal and workload identity auth_ - The tenant/domain ID of the AAD registered application. | | | ||
| client_id | _Required for service principal and workload identity auth_ - The client ID of the AAD registered application. When using managed identity authentication, set this to 'system' for system-assigned identity or provide the managed identity's client ID. | | |
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.
Shouldn't the first sentence be "Required for service principal and managed identity (instead of workload identity) auth"?
@tanmaya-panda1 can you please tackle the reviewers suggestions above and at the same time resolve the conflicts, as the code PR fluent/fluent-bit#10283 was merged and I'd like to add the docs too? |
references fluent/fluent-bit#10283