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

Rename or remove runtime.uptime metric from instrumentation/runtime package #2625

Closed
jmacd opened this issue Aug 5, 2022 · 2 comments
Closed
Labels
area: instrumentation Related to an instrumentation package enhancement New feature or request instrumentation: runtime

Comments

@jmacd
Copy link
Contributor

jmacd commented Aug 5, 2022

In #1549 added a "process." prefix to the metrics that had been prefixed "runtime.go" already.,
The "runtime.uptime" metric was not renamed, but it also did not have the "runtime.go" prefix.

In my opinion, rutime.uptime should be removed and OTel would ideally specify a resource attribute that would allow deriving the uptime at the moment of any data point, see open-telemetry/opentelemetry-specification#1273. My preference is to remove "runtime.uptime".

An alternative is to rename this "process.runtime.go.uptime", since OTel hasn't specified a cross-lang process uptime metric presently.

@AkhigbeEromo
Copy link
Contributor

Would like to work on this issue @jmacd @MrAlias

@MrAlias
Copy link
Contributor

MrAlias commented Oct 17, 2024

The uptime metric was deprecated in https://github.com/open-telemetry/opentelemetry-go-contrib/releases/tag/v1.29.0. Closing.

@MrAlias MrAlias closed this as completed Oct 17, 2024
@pellared pellared closed this as not planned Won't fix, can't repro, duplicate, stale Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: instrumentation Related to an instrumentation package enhancement New feature or request instrumentation: runtime
Projects
Development

No branches or pull requests

4 participants