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.

Sign up for GitHub

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

Jump to bottom

[FEATURE REQ] AddAzureMonitorLogExporter should support IOptions #38991

Closed
scrocquesel-ml150 opened this issue Sep 27, 2023 · 3 comments
Closed

[FEATURE REQ] AddAzureMonitorLogExporter should support IOptions #38991

scrocquesel-ml150 opened this issue Sep 27, 2023 · 3 comments
Assignees
@vishweshbankwar
Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. Monitor - Exporter Monitor OpenTelemetry Exporter question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@scrocquesel-ml150
Copy link

Library name

Azure.Monitor.OpenTelemetry.Exporter

Please describe the feature.

Is it possible to configure the Log exporter with traditional IOptions idioms, similar to Trace and Metric exporters? Currently, options are only provided through the configure delegate passed to the extension method. I have a use case where I want to separate AddAzureMonitorLogExporter from its configuration.
@github-actions github-actions bot added Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. Monitor - Exporter Monitor OpenTelemetry Exporter needs-team-triage Workflow: This issue needs the team to triage. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Sep 27, 2023
@vishweshbankwar
Copy link
Contributor

@scrocquesel-ml150 Thanks for raising this. This is currently a limitation from OTel SDK side. Please follow
open-telemetry/opentelemetry-dotnet#4014
open-telemetry/opentelemetry-dotnet#4433.

@jsquire jsquire added issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. and removed needs-team-triage Workflow: This issue needs the team to triage. labels Sep 28, 2023
@github-actions
Copy link

Hi @scrocquesel-ml150. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text "/unresolve" to remove the "issue-addressed" label and continue the conversation.

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

Hi @scrocquesel-ml150, since you haven’t asked that we /unresolve the issue, we’ll close this out. If you believe further discussion is needed, please add a comment /unresolve to reopen the issue.

@github-actions github-actions bot closed this as completed Oct 5, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@vishweshbankwar vishweshbankwar

Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. Monitor - Exporter Monitor OpenTelemetry Exporter question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jsquire @vishweshbankwar @scrocquesel-ml150