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

config: add stdout exporter for log signal #5432

Open
codeboten opened this issue Apr 26, 2024 · 1 comment
Open

config: add stdout exporter for log signal #5432

codeboten opened this issue Apr 26, 2024 · 1 comment
Labels
area: config Related to config functionality enhancement New feature or request

Comments

@codeboten
Copy link
Contributor

Problem Statement

As it was done for metric and tracing, the logging signal should support configuring a console exporter via config.

Proposed Solution

Add the following code and add unit tests as well.

	if exporter.Console != nil && exporter.OTLP != nil {
	 	return nil, errors.New("must not specify multiple exporters")
	 }

	 if exporter.Console != nil {
	 	return stdoutlog.New(
	 		stdoutlogWithPrettyPrint(),
	 	)
	 }

Additional Context

This was not possible because of a change needed in the configuration schema (open-telemetry/opentelemetry-configuration@834ae3c) which has been merged, but updating the configuration package depends on #4832 to support another change in the configuration schema.

@codeboten codeboten added area: config Related to config functionality enhancement New feature or request labels Apr 26, 2024
@codeboten
Copy link
Contributor Author

Happy to tackle this one once #4832 and #5427 have been merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: config Related to config functionality enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant