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

[extension/jaegerremotesampling] mark as alpha #13411

Conversation

frzifus
Copy link
Member

@frzifus frzifus commented Aug 18, 2022

Signed-off-by: Benedikt Bongartz [email protected]

Description:

Link to tracking Issue:
Closes #13005

Testing:

Documentation:


@jpkrohling Is that all we have to do or is there something else I haven't considered?

@frzifus frzifus requested a review from a team August 18, 2022 12:43
@frzifus frzifus requested a review from jpkrohling as a code owner August 18, 2022 12:43
@frzifus frzifus force-pushed the mark_jaegerremotesampling_extention_as_alpha branch from 3ac1428 to 9d9a4b6 Compare August 18, 2022 15:30
extension/jaegerremotesampling/README.md Outdated Show resolved Hide resolved
@frzifus frzifus force-pushed the mark_jaegerremotesampling_extention_as_alpha branch from 9d9a4b6 to 7ccfcb2 Compare August 22, 2022 14:27
@frzifus frzifus requested a review from jpkrohling August 22, 2022 14:27
@frzifus
Copy link
Member Author

frzifus commented Aug 22, 2022

mh.. the deb step failed for some reason.

@jpkrohling jpkrohling merged commit 4066fcb into open-telemetry:main Aug 22, 2022
@jpkrohling
Copy link
Member

Merging it anyway, as it has passed before removing the single whitespace from the readme. I don't think it's related to that change.

@frzifus frzifus deleted the mark_jaegerremotesampling_extention_as_alpha branch August 22, 2022 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[extension/jaegerremotesampling] Mark as alpha
3 participants