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

[chore][connector/routing] Add utilities for concisely testing logs #36150

Merged
merged 1 commit into from
Nov 3, 2024

Conversation

djaglowski
Copy link
Member

Same idea as #36149. I stopped short of updating the actual routing tests because there are already so many deleted files in this PR. A followup PR will handle the remainder.

@github-actions github-actions bot requested review from jpkrohling and mwear November 2, 2024 16:14
@djaglowski djaglowski force-pushed the routing-logstestutil branch from 5829861 to ba6109e Compare November 2, 2024 16:23
@djaglowski djaglowski marked this pull request as ready for review November 2, 2024 17:32
@djaglowski djaglowski requested a review from a team as a code owner November 2, 2024 17:32
@djaglowski djaglowski force-pushed the routing-logstestutil branch from ba6109e to 76904e3 Compare November 2, 2024 18:54
@djaglowski djaglowski merged commit a5807e5 into open-telemetry:main Nov 3, 2024
158 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 3, 2024
@djaglowski djaglowski deleted the routing-logstestutil branch November 3, 2024 15:20
djaglowski added a commit that referenced this pull request Nov 4, 2024
Follows
#36150

Finishes migrating logs tests to new concise format. 

Also includes minor nonfunctional refactoring in metrics test to match
what was needed for logs.
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
…elemetry#36158)

Follows
open-telemetry#36150

Finishes migrating logs tests to new concise format. 

Also includes minor nonfunctional refactoring in metrics test to match
what was needed for logs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants