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.

Sign up for GitHub

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

Jump to bottom

[extension/jaegerremotesampling] Mark as alpha #13005

Closed
jpkrohling opened this issue Aug 8, 2022 · 4 comments · Fixed by #13411
Closed

[extension/jaegerremotesampling] Mark as alpha #13005

jpkrohling opened this issue Aug 8, 2022 · 4 comments · Fixed by #13411
Assignees
@frzifus
Labels
bug Something isn't working extension/jaegerremotesampling good first issue Good for newcomers priority:p2 Medium

Comments

@jpkrohling
Copy link
Member

This component is new and hasn't had exposure to production workloads yet. As such, it should be classified as alpha instead of beta.
@jpkrohling jpkrohling added bug Something isn't working good first issue Good for newcomers extension/jaegerremotesampling labels Aug 8, 2022
@jpkrohling
Copy link
Member Author

@frzifus, are you interested in this?

@frzifus
Copy link
Member

frzifus commented Aug 8, 2022

Sure, thanks. I will have a more detailed look by the end of the week.

@dmitryax dmitryax added the priority:p2 Medium label Aug 12, 2022
@frzifus frzifus mentioned this issue Aug 18, 2022
Merged
@frzifus
Copy link
Member

frzifus commented Aug 18, 2022

Another thing, currently extention/jaegerremotesampling is not a registered component. On what does the registration depend?

@jpkrohling
Copy link
Member Author

This step is done at the same logical step as adding to a distribution. In the past, this would be required to have the component as part of the binary, while today I think it's mostly for convenience during tests.

@frzifus frzifus mentioned this issue Aug 25, 2022
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frzifus frzifus

Labels
bug Something isn't working extension/jaegerremotesampling good first issue Good for newcomers priority:p2 Medium
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[extension/jaegerremotesampling] mark as alpha frzifus/opentelemetry-collector-contrib
3 participants
@jpkrohling @dmitryax @frzifus