Change default OTLP/HTTP port number

[tigrannajaryan]

This implements specification change open-telemetry/opentelemetry-specification#1839

To make transition to new port numbers less painful OTLP receiver will
also accept data on the legacy port numbers when it is configured to
use the default endpoint. Users who use the default Collector config
can continue sending data to the legacy ports and have a graceful period
to update their senders to start sending to the new ports.

[tigrannajaryan]

Should be merged after open-telemetry/opentelemetry-specification#1839 is merged.

[codeboten]

Is there any documentation on what the grace period should be for both port to receive HTTP traffic?

[tigrannajaryan]

Is there any documentation on what the grace period should be for both port to receive HTTP traffic?

There is none AFAIK. Suggestions are welcome. 6 months from now perhaps enough?

[tigrannajaryan]

Spec PR is now merged open-telemetry/opentelemetry-specification#1839
This is ready for review.

sorry, wanted to run a grep first

[jpkrohling]

Looks like there are a few places missing:

$ grep 55681 . -r
./cmd/configschema/docsgen/docsgen/testdata/otlp-receiver.json:              "Default": "0.0.0.0:55681",
./exporter/otlphttpexporter/README.md:- `endpoint` (no default): The target base URL to send data to (e.g.: https://example.com:55681).
./exporter/otlphttpexporter/README.md:- `traces_endpoint` (no default): The target URL to send trace data to (e.g.: https://example.com:55681/v1/traces).
./exporter/otlphttpexporter/README.md:- `metrics_endpoint` (no default): The target URL to send metric data to (e.g.: https://example.com:55681/v1/metrics).
./exporter/otlphttpexporter/README.md:- `logs_endpoint` (no default): The target URL to send log data to (e.g.: https://example.com:55681/v1/logs).
./exporter/otlphttpexporter/README.md:    endpoint: https://example.com:55681/v1/traces
./receiver/otlpreceiver/config.md:| endpoint |string| 0.0.0.0:55681 | Endpoint configures the listening address for the server.  |
./receiver/otlpreceiver/factory.go:	legacyHTTPEndpoint  = "0.0.0.0:55681"
grep: ./bin/otelcol_linux_amd64: binary file matches
./website_docs/configuration.md:    endpoint: https://example.com:55681/v1/traces

[tigrannajaryan]

Looks like there are a few places missing:

$ grep 55681 . -r
./cmd/configschema/docsgen/docsgen/testdata/otlp-receiver.json:              "Default": "0.0.0.0:55681",
./exporter/otlphttpexporter/README.md:- `endpoint` (no default): The target base URL to send data to (e.g.: https://example.com:55681).
./exporter/otlphttpexporter/README.md:- `traces_endpoint` (no default): The target URL to send trace data to (e.g.: https://example.com:55681/v1/traces).
./exporter/otlphttpexporter/README.md:- `metrics_endpoint` (no default): The target URL to send metric data to (e.g.: https://example.com:55681/v1/metrics).
./exporter/otlphttpexporter/README.md:- `logs_endpoint` (no default): The target URL to send log data to (e.g.: https://example.com:55681/v1/logs).
./exporter/otlphttpexporter/README.md:    endpoint: https://example.com:55681/v1/traces
./receiver/otlpreceiver/config.md:| endpoint |string| 0.0.0.0:55681 | Endpoint configures the listening address for the server.  |
./receiver/otlpreceiver/factory.go:	legacyHTTPEndpoint  = "0.0.0.0:55681"
grep: ./bin/otelcol_linux_amd64: binary file matches
./website_docs/configuration.md:    endpoint: https://example.com:55681/v1/traces

Thanks for the catch. I only greped *.go files :-(

[tigrannajaryan]

@jpkrohling PTAL, should be good now.

[theletterf]

@tigrannajaryan Has anything been decided on the grace period?

[tigrannajaryan]

@theletterf 6 months sounds reasonable to me. I think we should start the countdown from the moment Otel spec clearly documents the 4318 port, which we should do asap once the final decision to keep the ports separate is made.