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

Move telemetry package (e.g. telemetry.GetLevel) to a public package #1663

Closed
pavolloffay opened this issue Aug 28, 2020 · 1 comment
Closed

Move telemetry package (e.g. telemetry.GetLevel) to a public package #1663

pavolloffay opened this issue Aug 28, 2020 · 1 comment
Labels
area:service enhancement New feature or request priority:p3 Lowest release:after-ga
Milestone
Backlog

Comments

@pavolloffay
Copy link
Member

Is your feature request related to a problem? Please describe.

In Jaeger OpenTelemetry we would like to have access to telemetry.GetLevel() and telemetry.Level symbols to appropriately scope down telemetry data reported by the service.

This is a prerequisite for #1413.

Describe the solution you'd like

Make telemetry package public.

Describe alternatives you've considered

No

Additional context

None
@tigrannajaryan tigrannajaryan added this to the Backlog milestone Sep 2, 2020
@bogdandrutu bogdandrutu mentioned this issue Sep 16, 2020
Merged
@andrewhsu andrewhsu added enhancement New feature or request and removed feature request labels Jan 6, 2021
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this issue Nov 11, 2021
@mrveera @Aneurysm9 @MrAlias
Removed setting error status while recording err as span event (open-…
a5edd79 
…telemetry#1663)

* Fix open-telemetry#1661 Removed setting error status while recording err as span event

* Update CHANGELOG.md

Co-authored-by: Anthony Mirabella <[email protected]>

* Update RecordError method doc

Co-authored-by: Tyler Yahn <[email protected]>

* Fix grammatical errors in Changelog

Co-authored-by: Tyler Yahn <[email protected]>

* Update RecordError method doc

Co-authored-by: Anthony Mirabella <[email protected]>
Co-authored-by: Tyler Yahn <[email protected]>
@bogdandrutu
Copy link
Member

Already part of the component.TelemetrySettings

hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this issue Apr 27, 2023
@jeffreyc-splunk
Fix puppet test failure for opensuse-15 (open-telemetry#1663)
9d76a44
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this issue Jul 5, 2024
@lalitb
Fix open-telemetry#1663 Threading issue between Meter::RegisterSyncMe…
99f2ba4 
…tricStorage and Meter::Collect (open-telemetry#1666)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:service enhancement New feature or request priority:p3 Lowest release:after-ga
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
4 participants
@andrewhsu @bogdandrutu @tigrannajaryan @pavolloffay