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

Adding support for entrypoint loading of log exporters #2253

Merged

Conversation

codeboten
Copy link
Contributor

This replicates the trace exporter pattern for the logging signal. It includes:

  • adding entrypoint for the OTLP and console log exporters
  • loads the log exporter and log emitter provider as part of the configuration for auto-instrumentation

@codeboten codeboten marked this pull request as ready for review November 24, 2021 00:35
@codeboten codeboten requested a review from a team as a code owner November 24, 2021 00:35
@codeboten codeboten changed the base branch from logs to main November 24, 2021 00:36
@codeboten codeboten force-pushed the codeboten/add-console-entrypoint branch from c3f0ea9 to bd44dcb Compare November 24, 2021 00:39
Alex Boten added 4 commits November 29, 2021 08:37
This replicates the trace exporter pattern for the logging signal. It includes:
- adding entrypoint for the OTLP and console log exporters
- loads the log exporter and log emitter provider as part of the configuration for auto-instrumentation
@codeboten codeboten force-pushed the codeboten/add-console-entrypoint branch from bd44dcb to 57397d8 Compare November 29, 2021 16:39
@srikanthccv srikanthccv added the Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary label Dec 1, 2021
@lzchen lzchen merged commit 46f77d0 into open-telemetry:main Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants