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

Update README #102

Merged
merged 1 commit into from
Sep 24, 2019
Merged

Update README #102

merged 1 commit into from
Sep 24, 2019

Conversation

elskwid
Copy link
Member

@elskwid elskwid commented Sep 23, 2019

Overview

Updates the README with badges, contributing information, initial
release schedule, and useful links.

Details

  • Takes a little from otel-python and otel-js
  • Leaves out details on the packages/gems
  • Current schedule is Unknown

@fbogsany and/or @mwear feel free to chime in with a good initial schedule and I'll update this PR before it's merged.

@elskwid
Copy link
Member Author

elskwid commented Sep 23, 2019

I signed it

Copy link
Contributor

@fbogsany fbogsany left a comment

Choose a reason for hiding this comment

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

Looks good so far! I've added comments with recommended dates.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
| Tracing SDK | Alpha | Unknown |
| Metrics API | Alpha | Unknown |
| Metrics SDK | Alpha | Unknown |
| Jaeger Trace Exporter | Alpha | Unknown |
Copy link
Contributor

Choose a reason for hiding this comment

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

Leave this as Unknown for now. I started scoping the work today, but I'm not sure exactly what's expected (UDP export to the agent only? Thrift? Protobuf? grpc? HTTP?). AFAICT we don't have an existing Jaeger client for Ruby to piggyback on.

Copy link
Member Author

Choose a reason for hiding this comment

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

Perhaps the jaeger-client-ruby lib linked in this issue has something we can use. From my quick glance this looks more like an OpenTracing Tracer than a Jaeger client library.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for the link! We can definitely use that, either as a starting point or we can wrap it. I’ll dig into it some more.

README.md Show resolved Hide resolved
Updates the README with badges, contributing information, initial
release schedule, and useful links.
@elskwid
Copy link
Member Author

elskwid commented Sep 24, 2019

@fbogsany I've updated the schedule with the dates we have now and removed the Zipkin exporter from the list. Thanks for the review!

@fbogsany fbogsany merged commit 374ae0d into open-telemetry:master Sep 24, 2019
@elskwid elskwid deleted the update-readme branch September 24, 2019 15:35
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.

2 participants