-
Notifications
You must be signed in to change notification settings - Fork 887
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
Add OpenTracing section to the compliance matrix. #2046
Add OpenTracing section to the compliance matrix. #2046
Conversation
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
@carlosalberto I've reassigned to you to move this forward as per e-mail from @bhs. |
@yurishkuro Updated, please review - I'm keeping the |
ping @carlosalberto on this (and @yurishkuro's comments)... this PR is on the critical path to formally archiving the OpenTracing project within CNCF so would be good to keep it moving, though obviously not P0 or anything. :) |
@yurishkuro Applied feedback. The only remaining bit is the one regarding custom propagators - which is useful in case users want to use a different propagator for OpenTracing's |
@yurishkuro Please review. Removed some parts (such as the Left the error mapping one without a link as it is mentioned in both sections. It can probably be also removed though, although I'd personally keep that one in order to signal implementers of its importance ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for picking this up Carlos.
As we are looking to stabilize the OpenTracing compatibility section, we want to keep track of the actual features in different SIGs.