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

[connector/routing] Add ability to route metrics and traces by request context #36143

Merged

Conversation

djaglowski
Copy link
Member

@djaglowski djaglowski commented Nov 1, 2024

This PR follows #36067 and does the same for metrics and traces.

I believe once this PR is merged we can close #19738.

@github-actions github-actions bot requested review from jpkrohling and mwear November 1, 2024 23:04
@djaglowski djaglowski added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Nov 1, 2024
@djaglowski
Copy link
Member Author

Skipping changelog because if this is merged before the next release then the modification it makes to an existing changelog is sufficient.

@djaglowski djaglowski force-pushed the routing-context-metrics-traces branch from 363cfb2 to 3c618bf Compare November 1, 2024 23:27
@djaglowski djaglowski marked this pull request as ready for review November 1, 2024 23:40
@djaglowski djaglowski requested a review from a team as a code owner November 1, 2024 23:40
@djaglowski djaglowski marked this pull request as draft November 4, 2024 13:53
@djaglowski
Copy link
Member Author

I've moved this back to draft until I finish migrating the tests to a more concise format.

@djaglowski djaglowski force-pushed the routing-context-metrics-traces branch from 3c618bf to 5de18e4 Compare November 5, 2024 00:41
@djaglowski djaglowski marked this pull request as ready for review November 5, 2024 00:53
@djaglowski
Copy link
Member Author

This is rebased on the new testing framework.

@djaglowski djaglowski merged commit 07a7ca9 into open-telemetry:main Nov 6, 2024
160 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 6, 2024
@djaglowski djaglowski deleted the routing-context-metrics-traces branch November 6, 2024 02:13
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.

New component: Routing connector
4 participants