-
Notifications
You must be signed in to change notification settings - Fork 887
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
Add process.uptime
and system.uptime
metrics to semantic conventions
#2824
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,6 +16,7 @@ instruments not explicitly defined in the specification. | |
<!-- toc --> | ||
|
||
- [Metric Instruments](#metric-instruments) | ||
* [`system.` - General system metrics](#system---general-system-metrics) | ||
* [`system.cpu.` - Processor metrics](#systemcpu---processor-metrics) | ||
* [`system.memory.` - Memory metrics](#systemmemory---memory-metrics) | ||
* [`system.paging.` - Paging/swap metrics](#systempaging---pagingswap-metrics) | ||
|
@@ -29,6 +30,14 @@ instruments not explicitly defined in the specification. | |
|
||
## Metric Instruments | ||
|
||
### `system.` - General system metrics | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A question I asked myself - would it make sense to have a namespace for system metadata like this? Perhaps |
||
|
||
**Description:** General system metrics. | ||
|
||
| Name | Description | Units | Instrument Type ([*](README.md#instrument-types)) | Value Type | Attribute Key(s) | Attribute Values | | ||
| ---------------------- | -------------------------------------------------------------------------------------------------------- | ----- | ------------------------------------------------- | ---------- | ---------------- | ----------------------------------- | | ||
| system.uptime | Number of seconds that the system has been running. | s | Counter | Int64 | | | | ||
jamesmoessis marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
### `system.cpu.` - Processor metrics | ||
|
||
**Description:** System level processor metrics. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be a counter or gauge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it should be a gauge, the value represents the uptime of the system at the given time of recording.
Any further aggregation or calculations hold no additional statistical meaning.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, gauge is more appropriateThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should be debating whether this is a Counter or an UpDownCounter. I will put my rationale in the main thread.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we agree on "what is uptime"? I suspect we are not on the same page 🤣
https://en.wikipedia.org/wiki/Uptime#Using_uptime
If let's say have a Chrome browser running 10 tabs (which might give us 11 processes), do we expect the uptime to be added across the 11 processes, and what does that mean?
If the browser is closed, then reopened, what does the uptime mean?
The Linux uptime command seems to be focusing on "how long has it been since the operating system started" https://en.wikipedia.org/wiki/Uptime#Linux, and the uptime would reset if the system restarted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we take the semantic here https://en.wikipedia.org/wiki/Uptime#Records
A Cisco router has been reported to have been running continuously for 21 years.
then making it a counter sounds like wrong?