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] Migrate traces test to concise format #36165

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

djaglowski
Copy link
Member

Depends on #36158

This is the final PR for migrating the test suite to a concise format. It adds the internal ptraceutiltest package. It also includes some minor style changes motivated by tracing element names, but which should be standard across signals.

@github-actions github-actions bot requested review from jpkrohling and mwear November 4, 2024 14:55
@djaglowski djaglowski changed the title Routing ptraceutiltest [chore][connector/routing] Migrate traces test to concise format Nov 4, 2024
@djaglowski djaglowski added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Nov 4, 2024
@djaglowski djaglowski force-pushed the routing-ptraceutiltest branch from 3abcdb6 to 9201bbd Compare November 4, 2024 15:58
@djaglowski djaglowski force-pushed the routing-ptraceutiltest branch from 9201bbd to 1fcad43 Compare November 4, 2024 16:19
@djaglowski djaglowski marked this pull request as ready for review November 4, 2024 20:51
@djaglowski djaglowski requested a review from a team as a code owner November 4, 2024 20:51
@djaglowski djaglowski merged commit 740d9aa into open-telemetry:main Nov 5, 2024
160 checks passed
@djaglowski djaglowski deleted the routing-ptraceutiltest branch November 5, 2024 00:08
@github-actions github-actions bot added this to the next release milestone Nov 5, 2024
michael-burt pushed a commit to michael-burt/opentelemetry-collector-contrib that referenced this pull request Nov 7, 2024
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connector/routing Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants