Skip to content

Check and disable duplicate internal logs in tests #2841

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
Open
cijothomas opened this issue Mar 20, 2025 · 2 comments · May be fixed by #2943
Open

Check and disable duplicate internal logs in tests #2841

cijothomas opened this issue Mar 20, 2025 · 2 comments · May be fixed by #2943
Assignees
@scottgerring

Comments

@cijothomas
Copy link
Member

#2833 (comment)
@cijothomas cijothomas mentioned this issue Mar 20, 2025
Merged
4 tasks
@scottgerring scottgerring self-assigned this Mar 21, 2025
@scottgerring
Copy link
Contributor

Hey @cijothomas , do you happen to have an example of this handy, so I have a thread to pull on?
If not no worries I can grep through.

@cijothomas
Copy link
Member Author

https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-otlp/tests/integration_test/src/test_utils.rs#L51-L54 This is where we enable fmt layer to collect all SDK internal logs. This is not required if we already print logs to stdout in Test configuration.

@scottgerring scottgerring linked a pull request Apr 25, 2025 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@scottgerring scottgerring

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: remove redundant test logging scottgerring/opentelemetry-rust
2 participants
@scottgerring @cijothomas