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

Rename handles to "bound instrument" #468

Closed
lzchen opened this issue Mar 6, 2020 · 2 comments
Closed

Rename handles to "bound instrument" #468

lzchen opened this issue Mar 6, 2020 · 2 comments
Assignees
@lzchen
Labels
metrics

Comments

@lzchen
Copy link
Contributor

lzchen commented Mar 6, 2020 •
edited
Loading

From OTEP

Specs
@lzchen lzchen mentioned this issue Mar 6, 2020
Closed
7 tasks
Closed
@c24t c24t closed this as completed Mar 9, 2020
@lzchen lzchen self-assigned this Mar 9, 2020
@c24t
Copy link
Member

c24t commented Mar 9, 2020

Looks like this is still open, cc @mauriciovasquezbernal.

@lzchen lzchen mentioned this issue Mar 9, 2020
Merged
@lzchen lzchen reopened this Mar 9, 2020
@mauriciovasquezbernal
Copy link
Member

Solved by #470.

@lzchen lzchen mentioned this issue Mar 11, 2020
Merged
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this issue Nov 1, 2020
@dyladan @mayurkale22
feat: validate metric names (open-telemetry#468)
2953a29 
* feat: validate metric names

* fix: log invalid metric names

* test: fix compilation errors in tests

* style: fix linting issues

* test: add name tests to measure metric
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lzchen lzchen

Labels
metrics
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lzchen @mauriciovasquezbernal @c24t