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.

Sign up for GitHub

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

Jump to bottom

Add Metrics API #51

Closed
mayurkale22 opened this issue Jun 20, 2019 · 1 comment · Fixed by #105
Closed

Add Metrics API #51

mayurkale22 opened this issue Jun 20, 2019 · 1 comment · Fixed by #105
Assignees
@bg451
Milestone
API complete

Comments

@mayurkale22
Copy link
Member

Specs: open-telemetry/opentelemetry-specification#137

Metrics proposal doc: https://docs.google.com/document/d/1-oFRwkaHVlLM8hTB8NPy-rTcfCZFX5PR4PhZCiK4rCE/edit
@mayurkale22 mayurkale22 added the up-for-grabs Good for taking. Extra help will be provided by maintainers label Jun 20, 2019
@mayurkale22 mayurkale22 added this to the API complete milestone Jun 20, 2019
@mayurkale22 mayurkale22 removed the up-for-grabs Good for taking. Extra help will be provided by maintainers label Jun 27, 2019
@mayurkale22 mayurkale22 added the API label Jul 8, 2019
@bg451 bg451 mentioned this issue Jul 16, 2019
Merged
@bg451
Copy link
Member

bg451 commented Sep 5, 2019

During the SIG meeting on 9/4/19, we discussed releasing an alpha for the end of september. Since the metrics specification currently has some in flight RFCs, the API is bound to change anyway so it might be best to design against the latest RFCs, specifically RFC 3. I'll be closing #105 and creating a new PR with the RFC's changes.

dyladan added a commit to dyladan/opentelemetry-js that referenced this issue Sep 9, 2022
@dyladan
feat: add function to wrap SpanContext in NonRecordingSpan open-telem…
cd255ce 
…etry#49 (open-telemetry#51)
dyladan added a commit to dyladan/opentelemetry-js that referenced this issue Sep 9, 2022
@dyladan
feat: add function to wrap SpanContext in NonRecordingSpan open-telem…
5820d00 
…etry#49 (open-telemetry#51)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bg451 bg451

Labels
None yet
Projects
None yet
Milestone
API complete
Development

Successfully merging a pull request may close this issue.

Adds Metrics API bg451/opentelemetry-node
2 participants
@bg451 @mayurkale22