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

add OTEL_LOGS_EXPORTER env variable #2320

Merged

Conversation

codeboten
Copy link
Contributor

@codeboten codeboten commented Dec 13, 2021

As per the change in the specification (open-telemetry/opentelemetry-specification@9fc9757) making the OTEL_LOGS_EXPORTER variable official.

Fixes #2315

Type of change

  • New feature (non-breaking change which adds functionality)

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Documentation has been updated

@codeboten codeboten requested a review from a team December 13, 2021 19:11
As per the change in the specification, adding the variable. There will be a follow up PR to use the variable.
@codeboten codeboten force-pushed the codeboten/add-logs-exporter-env-var branch from 1370586 to 83d1e2e Compare December 13, 2021 19:14
@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 14, 2021
@ocelotl ocelotl merged commit 8d4a804 into open-telemetry:main Dec 14, 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.

Add support for OTEL_LOGS_EXPORTER
3 participants