Skip to content
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

Register OTLP/HTTP port 4318 with IANA #1846

Closed
tigrannajaryan opened this issue Aug 4, 2021 · 10 comments
Closed

Register OTLP/HTTP port 4318 with IANA #1846

tigrannajaryan opened this issue Aug 4, 2021 · 10 comments
Assignees
Labels
spec:protocol Related to the specification/protocol directory

Comments

@tigrannajaryan
Copy link
Member

We have recently added a new OTLP/HTTP port 4318 to the specification (in addition to previously existing OTLP/gRPC port 4317).

We need to register the new port with IANA. Port 4317 description currently says "OpenTelemetry Protocol". For port 4318 we can probably use "OpenTelemetry Protocol, HTTP Transport" or "OpenTelemetry Protocol, OTLP/HTTP". Ideally also change description for port 4317 to be uniform to something like "OpenTelemetry Protocol, gRPC Transport" or "OpenTelemetry Protocol, OTLP/gRPC".

@mtwo you did the IANA submission for port 4317, can you help with the new port as well?

@tigrannajaryan tigrannajaryan added the spec:protocol Related to the specification/protocol directory label Aug 4, 2021
@tigrannajaryan
Copy link
Member Author

Apparently there is new evidence that one port may work after all: https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

I suggest we hold this issue and don't submit to IANA just yet until we have more clarity on this. Depending on the outcome we may need to do things a bit differently here in the spec.

@carlosalberto
Copy link
Contributor

@tigrannajaryan We are expected to do a monthly release by EOW. What do you think we should do? Revert this change for now?

@tigrannajaryan
Copy link
Member Author

@tigrannajaryan We are expected to do a monthly release by EOW. What do you think we should do? Revert this change for now?

@carlosalberto yes, I think reverting is a good idea. Let's aim to settle this for the release after this one.

@tigrannajaryan
Copy link
Member Author

I will submit a PR.

tigrannajaryan added a commit to tigrannajaryan/opentelemetry-specification that referenced this issue Aug 4, 2021
This reverts commit cce1d59 until we have the final
decision about the ability to have a single port, see:

open-telemetry#1846 (comment)
tigrannajaryan added a commit to tigrannajaryan/opentelemetry-specification that referenced this issue Aug 4, 2021
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d59 until we have the final
decision about the ability to have a single port, see:

open-telemetry#1846 (comment)
@tigrannajaryan
Copy link
Member Author

Reverting PR submitted: #1847

tigrannajaryan added a commit that referenced this issue Aug 6, 2021
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d59 until we have the final
decision about the ability to have a single port, see:

#1846 (comment)
@tigrannajaryan
Copy link
Member Author

Closing this issue for now to avoid accidentally moving forward with IANA. I will reopen if needed.

tigrannajaryan added a commit to tigrannajaryan/opentelemetry-proto that referenced this issue Apr 20, 2023
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
tigrannajaryan added a commit to tigrannajaryan/opentelemetry-proto that referenced this issue Apr 21, 2023
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
tigrannajaryan added a commit to open-telemetry/opentelemetry-proto that referenced this issue Apr 27, 2023
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
@jpkrohling
Copy link
Member

I think we should reconsider this: port 4318 is documented and used quite often, and I think it might make sense to register this port with IANA. Should a new issue be created, or is it OK to just reopen this?

@tigrannajaryan
Copy link
Member Author

IIRC, IANA did not want to register a second port.

@mtwo
Copy link
Member

mtwo commented Mar 15, 2024 via email

joaopgrassi pushed a commit to dynatrace-oss-contrib/semantic-conventions that referenced this issue Mar 21, 2024
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
@jpkrohling
Copy link
Member

Thank you for the extra context, I was not aware of those attempts.

@bogdandrutu bogdandrutu reopened this Apr 16, 2024
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
Apparently there is new evidence that one port may work after all:
https://github.com/open-telemetry/opentelemetry-collector/pull/3765/files

This reverts commit cce1d5996873de38a68e05eafa4d5e224df9b8f1 until we have the final
decision about the ability to have a single port, see:

open-telemetry/opentelemetry-specification#1846 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:protocol Related to the specification/protocol directory
Projects
None yet
Development

No branches or pull requests

6 participants