Skip to content

Commit

Permalink
Revert "add metrics to replace metrics with direction (open-telemet…
Browse files Browse the repository at this point in the history 
…ry#2617)"

Contributes to open-telemetry#2726

This reverts open-telemetry#2617

We are reverting open-telemetry#2617 until we are certain how to resolve issue open-telemetry#2726

Also reverts the corresponding schema file changes done in
open-telemetry#2688
  • Loading branch information
@tigrannajaryan
tigrannajaryan committed Aug 23, 2022
1 parent 2ce0831 commit 68f6011
Show file tree
Hide file tree
Showing 4 changed files with 40 additions and 145 deletions.
3 changes: 0 additions & 3 deletions CHANGELOG.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,6 @@ release.
([#2618](https://github.com/open-telemetry/opentelemetry-specification/pull/2618))
- Change `faas.document.time` and `faas.time` level from `required` to `recommended`
([#2627](https://github.com/open-telemetry/opentelemetry-specification/pull/2627))
- Remove `direction` dimension, instead creating metrics with names reflecting those
dimensions.
([#2617](https://github.com/open-telemetry/opentelemetry-specification/pull/2617))
- Add `rpc.grpc.status_code` to RPC metric semantic conventions
([#2604](https://github.com/open-telemetry/opentelemetry-specification/pull/2604)).
- Add `http.*.*.size` metric semantic conventions for tracking size of requests
Expand Down
77 changes: 0 additions & 77 deletions schemas/1.13.0
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,83 +4,6 @@ versions:
1.13.0:
metrics:
changes:
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.paging.operations
by_attribute: direction
metrics_from_attributes:
system.paging.operations.in: in
system.paging.operations.out: out
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.disk.io
by_attribute: direction
metrics_from_attributes:
system.disk.io.read: read
system.disk.io.write: write
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.disk.operations
by_attribute: direction
metrics_from_attributes:
system.disk.operations.read: read
system.disk.operations.write: write
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.disk.operation_time
by_attribute: direction
metrics_from_attributes:
system.disk.operation_time.read: read
system.disk.operation_time.write: write
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.disk.merged
by_attribute: direction
metrics_from_attributes:
system.disk.merged.read: read
system.disk.merged.write: write
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.network.dropped
by_attribute: direction
metrics_from_attributes:
system.network.dropped.receive: receive
system.network.dropped.transmit: transmit
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.network.packets
by_attribute: direction
metrics_from_attributes:
system.network.packets.receive: receive
system.network.packets.transmit: transmit
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.network.errors
by_attribute: direction
metrics_from_attributes:
system.network.errors.receive: receive
system.network.errors.transmit: transmit
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: system.network.io
by_attribute: direction
metrics_from_attributes:
system.network.io.receive: receive
system.network.io.transmit: transmit
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: process.disk.io
by_attribute: direction
metrics_from_attributes:
process.disk.io.read: read
process.disk.io.write: write
# https://github.com/open-telemetry/opentelemetry-specification/pull/2617
- split:
apply_to_metric: process.network.io
by_attribute: direction
metrics_from_attributes:
process.network.io.receive: receive
process.network.io.transmit: transmit
# https://github.com/open-telemetry/opentelemetry-specification/pull/2675
- split:
apply_to_metric: system.network.connections
Expand Down
22 changes: 9 additions & 13 deletions specification/metrics/semantic_conventions/process-metrics.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,19 +31,15 @@ metrics](runtime-environment-metrics.md).

Below is a table of Process metric instruments.

| Name | Instrument Type ([*](README.md#instrument-types)) | Units | Description | Labels |
| --------------------------------- | ------------------------------------------------- | ---------- | ----------------------------------------------------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `process.cpu.time` | Counter | s | Total CPU seconds broken down by different states. | `state`, if specified, SHOULD be one of: `system`, `user`, `wait`. A process SHOULD be characterized _either_ by data points with no `state` labels, _or only_ data points with `state` labels. |
| `process.cpu.utilization` | Gauge | s | Difference in process.cpu.time since the last measurement, divided by the elapsed time and number of CPUs available to the process. | `state`, if specified, SHOULD be one of: `system`, `user`, `wait`. A process SHOULD be characterized _either_ by data points with no `state` labels, _or only_ data points with `state` labels. |
| `process.memory.usage` | UpDownCounter | By | The amount of physical memory in use. | |
| `process.memory.virtual` | UpDownCounter | By | The amount of committed virtual memory. | |
| `process.disk.io` (deprecated) | Counter | By | Disk bytes transferred. | `direction` SHOULD be one of: `read`, `write` |
| `process.disk.io.read` | Counter | By | Disk bytes read. | |
| `process.disk.io.write` | Counter | By | Disk bytes written. | |
| `process.network.io` (deprecated) | Counter | By | Network bytes transferred. | `direction` SHOULD be one of: `receive`, `transmit` |
| `process.network.io.receive` | Counter | By | Network bytes received. | |
| `process.network.io.transmit` | Counter | By | Network bytes transmitted. | |
| `process.threads` | UpDownCounter | {threads} | Process threads count. | |
| Name | Instrument Type ([*](README.md#instrument-types)) | Units | Description | Labels |
|---------------------------|---------------------------------------------------|-------|-------------------------------------------------------------------------------------------------------------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| `process.cpu.time` | Counter | s | Total CPU seconds broken down by different states. | `state`, if specified, SHOULD be one of: `system`, `user`, `wait`. A process SHOULD be characterized _either_ by data points with no `state` labels, _or only_ data points with `state` labels. |
| `process.cpu.utilization` | Gauge | s | Difference in process.cpu.time since the last measurement, divided by the elapsed time and number of CPUs available to the process. | `state`, if specified, SHOULD be one of: `system`, `user`, `wait`. A process SHOULD be characterized _either_ by data points with no `state` labels, _or only_ data points with `state` labels. |
| `process.memory.usage` | UpDownCounter | By | The amount of physical memory in use. | |
| `process.memory.virtual` | UpDownCounter | By | The amount of committed virtual memory. | |
| `process.disk.io` | Counter | By | Disk bytes transferred. | `direction` SHOULD be one of: `read`, `write` |
| `process.network.io` | Counter | By | Network bytes transferred. | `direction` SHOULD be one of: `receive`, `transmit` |
| `process.threads` | UpDownCounter | {threads} | Process threads count. | |

## Attributes

Expand Down
Loading

0 comments on commit 68f6011

Please sign in to comment.