Skip to content

Commit

Permalink
Change default OTLP port number
Browse files Browse the repository at this point in the history
Contributes to #1148

Note that a separate port is used for OTLP/HTTP for now. There is currently work
in progress to confirm that we can use the same port. Once we have the confirmation
I will update the spec again to use one port.
  • Loading branch information
tigrannajaryan committed Nov 10, 2020
1 parent 394e38e commit e92112e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ release.

New:

- Change default OTLP port number
([#1221](https://github.com/open-telemetry/opentelemetry-specification/pull/1221))
- Add performance benchmark specification
([#748](https://github.com/open-telemetry/opentelemetry-specification/pull/748))
- Enforce that the Baggage API must be fully functional, even without an installed SDK.
Expand Down
4 changes: 2 additions & 2 deletions specification/protocol/otlp.md
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@ some in beta.

#### Default Port

The default network port for OTLP/gRPC is 55680.
The default network port for OTLP/gRPC is 4317.

### OTLP/HTTP

Expand Down Expand Up @@ -445,7 +445,7 @@ connections SHOULD be configurable.

#### Default Port

The default network port for OTLP/HTTP is 55681. There is currently an [open
The default network port for OTLP/HTTP is 4318. There is currently an [open
issue](https://github.com/open-telemetry/opentelemetry-collector/issues/1256) to
use the same port for OTLP/gRPC and OTLP/HTTP. In that case this spec will be
updated to use the same default port for OTLP/gRPC and OTLP/HTTP.
Expand Down

0 comments on commit e92112e

Please sign in to comment.