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

ext/otcollector: Implement OTLP #669

Closed
codeboten opened this issue May 11, 2020 · 3 comments
Closed

ext/otcollector: Implement OTLP #669

codeboten opened this issue May 11, 2020 · 3 comments
Assignees

Comments

@codeboten
Copy link
Contributor

Is your feature request related to a problem?
The otcollector is currently using the opencensus protocol to communicate with the collector, this was done prior to otlp being supported in the collector. OTLP is now natively supported, let's use it.

Describe the solution you'd like
The otcollector supports otlp. The current code could be renamed to opencensus as suggested in this issue #541

Additional context
Add any other context about the feature request here.

@ocelotl ocelotl self-assigned this May 13, 2020
ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue May 18, 2020
@andrewhsu
Copy link
Member

looks like resolving this issue would help: open-telemetry/opentelemetry-proto#91

ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue May 25, 2020
@ocelotl ocelotl mentioned this issue Jun 3, 2020
ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue Jun 5, 2020
ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue Jun 6, 2020
ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue Jun 6, 2020
@ocelotl
Copy link
Contributor

ocelotl commented Jun 17, 2020

@codeboten, can this issue be closed now that we have #786 and #797?

@codeboten
Copy link
Contributor Author

Yup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants