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

OTLP HTTP exporter for Metrics #1371

Closed
lalitb opened this issue May 4, 2022 · 7 comments · Fixed by #1487
Closed

OTLP HTTP exporter for Metrics #1371

lalitb opened this issue May 4, 2022 · 7 comments · Fixed by #1487
Assignees

Comments

@lalitb
Copy link
Member

lalitb commented May 4, 2022

Before opening a feature request against this repo, consider whether the feature should/could be implemented in the other OpenTelemetry client libraries. If so, please open an issue on opentelemetry-specification first.

Is your feature request related to a problem?
If so, provide a concise description of the problem.

Describe the solution you'd like
What do you want to happen instead? What is the expected behavior?

Describe alternatives you've considered
Which alternative solutions or features have you considered?

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

@lalitb lalitb added this to the Metrics SDK - Beta Release milestone May 4, 2022
@esigo esigo self-assigned this May 4, 2022
@owent
Copy link
Member

owent commented May 5, 2022

I can help to implement OTLP http exporter if necessary.

@esigo
Copy link
Member

esigo commented May 5, 2022

I can help to implement OTLP http exporter if necessary.

Hi @owent, thanks for your interest. We need to first upgrade Protos #1290, which is blocked by #1291. Would you be interested in taking over #1291?
Thanks 🙏

@owent
Copy link
Member

owent commented May 5, 2022

I can help to implement OTLP http exporter if necessary.

Hi @owent, thanks for your interest. We need to first upgrade Protos #1290, which is blocked by #1291. Would you be interested in taking over #1291? Thanks 🙏

Sure.This can assign to me.
How can I make changes to #1290 ? Just raise a PR to esigo:upgrade-opentelemetry-proto-to-0.15.0 ?

@esigo
Copy link
Member

esigo commented May 5, 2022

I can help to implement OTLP http exporter if necessary.

Hi @owent, thanks for your interest. We need to first upgrade Protos #1290, which is blocked by #1291. Would you be interested in taking over #1291? Thanks 🙏

Sure.This can assign to me.
How can I make changes to #1290 ? Just raise a PR to esigo:upgrade-opentelemetry-proto-to-0.15.0 ?

Thank you very much. You may just raise another PR. I'll close mine.

@lalitb
Copy link
Member Author

lalitb commented May 5, 2022

Sure.This can assign to me.

Thanks @owent for helping on this.

@lalitb lalitb changed the title OTLP exporter for Metrics OTLP gRPC exporter for Metrics Jun 14, 2022
@lalitb lalitb changed the title OTLP gRPC exporter for Metrics OTLP HTTP exporter for Metrics Jun 14, 2022
@lalitb lalitb removed this from the Metrics SDK - Release Candidate milestone Jun 15, 2022
@lalitb
Copy link
Member Author

lalitb commented Jun 15, 2022

@owent - Have assigned to you as this has a dependency on async changes. Can be taken up once async changes are merged. Let me know if this is not ok.

@owent
Copy link
Member

owent commented Jun 16, 2022

OK.

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 a pull request may close this issue.

3 participants