Skip to content

Conversation

@darccio
Copy link

@darccio darccio commented Jan 28, 2026

Description

This is part of a refactor process on datadog-agent/pkg/trace/stats to move the OTel related code and dependencies to their specific package.

This has no impact on the collector, there are no changes in the component behaviour. No need to bump other packages as the new package includes the same code from previous dev version.

The impact is expected to be on reduced dependencies on dd-trace-go.

@darccio darccio requested review from a team, mx-psi and songy23 as code owners January 28, 2026 21:54
@github-actions github-actions bot added the first-time contributor PRs made by new contributors label Jan 28, 2026
@github-actions
Copy link
Contributor

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

@github-actions github-actions bot added the exporter/datadog Datadog components label Jan 28, 2026
@paulojmdias
Copy link
Member

Please take a look into the CI errors

@paulojmdias paulojmdias added Skip Changelog PRs that do not require a CHANGELOG.md entry waiting-for-code-owners labels Jan 28, 2026
@darccio
Copy link
Author

darccio commented Jan 28, 2026

@paulojmdias On it. Sorry for the noise. I'll check tomorrow, but I think I updated all the related go.mod files.

@darccio
Copy link
Author

darccio commented Jan 29, 2026

@paulojmdias There is a last failing pipeline, and it also fails locally (in my branch and in main):

cmd/schemagen/run_schemagen_dir.sh -v -i testdata .
Error: schemagen binary not found or not executable at /home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/.tools/schemagen

I'd appreciate your guidance around this issue.

@paulojmdias
Copy link
Member

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