Skip to content

Commit

Permalink
added spec changes per open-telemetry#1647
Browse files Browse the repository at this point in the history
  • Loading branch information
bryce-b committed May 4, 2021
1 parent 492718f commit 7186829
Showing 1 changed file with 19 additions and 1 deletion.
20 changes: 19 additions & 1 deletion specification/trace/semantic_conventions/span-general.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,25 @@ the `net.peer.*` properties of a client are equal to the `net.host.*` properties
| `net.host.ip` | string | Like `net.peer.ip` but for the host IP. Useful in case of a multi-IP host. | `192.168.0.1` | No |
| `net.host.port` | int | Like `net.peer.port` but for the host port. | `35555` | No |
| `net.host.name` | string | Local hostname or similar, see note below. | `localhost` | No |

| `net.host.connection_type` | string | The current network connection type in common terms | `wifi` | No |
| `net.host.carrier.name` | string | Name of the mobile carrier. | `sprint` | No |
| `net.host.carrier.mcc` | string | [Mobile country code](https://en.wikipedia.org/wiki/Mobile_country_code) | `310` | No |
| `net.host.carrier.mnc` | string | [Mobile network code](https://en.wikipedia.org/wiki/Mobile_country_code) | `001` | No |
| `net.host.carrier.icc` | string | [ISO country code (two character)](https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2) | `DE` | No |

`net.host.connection_type` SHOULD be one of the following :

| Value | Description |
| --- | --- |
| `wifi` | connected via wifi |
| `1G` | connected via a first gen cellular technology, such as `AMPS` or equivalent |
| `2G` | connected via a second gen cellular technology, such as `GPRS` |
| `3G` | connected via a third gen cellular technology, such as `UMTS` or `1xEV-DO` |
| `4G` | connected via a fourth gen cellular technology, such as `eHRPD` or `LTE` |
| `5G` | connected via a fifth gen cellular technology, such as `5G NR` |
| `unavailable` | network is disconnected or unavailable. |


`net.transport` MUST be one of the following:

| Value | Description |
Expand Down

0 comments on commit 7186829

Please sign in to comment.