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

split distro package into two modules #189

Closed
wants to merge 1 commit into from

Conversation

codeboten
Copy link
Contributor

Which problem is this PR solving?

Allow end users to configure individual components from the honeycomb-opentelemetry package without the auto-configuration provided by the distro.

Short description of the changes

The first module honeycomb-opentelemetry imports the second module honeycomb-opentelemetry-components. The original package honeycomb-opentelemetry will continue to work as it did before. For users that want to utilize components without using the auto configuration provided by the distro, they can import the components package and configure things manually.

How to verify that this has the expected result

Adding a distroless example that utilizes the honeycomb-opentelemetry-components package.

The first module `honeycomb-opentelemetry` imports the second module `honeycomb-opentelemetry-components`. The original package `honeycomb-opentelemetry` will continue to work as it did before. For users that want to utilize components without using the auto configuration provided by the distro, they can import the components package and configure things manually.

Signed-off-by: Alex Boten <[email protected]>
@codeboten codeboten self-assigned this Apr 9, 2024
@MikeGoldsmith MikeGoldsmith added type: maintenance The necessary chores to keep the dust off. version: bump minor A PR that adds behavior, but is backwards-compatible. labels Apr 10, 2024
@MikeGoldsmith
Copy link
Contributor

We're not going ahead with breaking the distro into separate componetns. The baggage span processor is being contributed to otel-python-contrib instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: maintenance The necessary chores to keep the dust off. version: bump minor A PR that adds behavior, but is backwards-compatible.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants