Skip to content

Conversation

@Angith
Copy link
Contributor

@Angith Angith commented Nov 7, 2025

Description

Move targetallocator package to internal/targetallocator to prevent external usage.

The targetallocator package was previously exposed as a public API, but it was never intended for external consumption. This change moves the package under the internal/ directory.

Link to tracking issue

Fixes #43996

Testing

  • All existing tests pass
  • Build succeeds: go build ./...

Documentation

No documentation changes needed as this package was not documented for external use.

@Angith Angith requested review from a team, ArthurSens and dashpole as code owners November 7, 2025 19:12
@github-actions github-actions bot added the receiver/prometheus Prometheus receiver label Nov 7, 2025
Copy link
Contributor

@dashpole dashpole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dashpole dashpole added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Nov 7, 2025
@songy23 songy23 merged commit eefd51a into open-telemetry:main Nov 7, 2025
207 of 208 checks passed
@otelbot
Copy link
Contributor

otelbot bot commented Nov 7, 2025

Thank you for your contribution @Angith! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help.

@github-actions github-actions bot added this to the next release milestone Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

receiver/prometheus Prometheus receiver Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

4 participants