Update OTel-http port to match the specification

[vasireddy99]

Description:

This PR updates the OTel http port to 4318, it includes the changes that might have missed in previous PR's

Reference:

• Change default OTLP/HTTP port number open-telemetry/opentelemetry-collector#3743

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[bryan-aguilar]

We should be consistent in whether or not we provide default points. I suggest we do not provide the endpoint field at all for any otlp receivers.

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4318

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4318

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4318

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4318

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4317

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4317

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4317

[bryan-aguilar]

Suggested change

endpoint: 0.0.0.0:4317

[vasireddy99]

We should be consistent in whether or not we provide default points. I suggest we do not provide the endpoint field at all for any otlp receivers.

We are currently providing default end points in all of the configs such as in here, since default will likely to not change much often from now, I think it will be okay?.

[bryan-aguilar]

We should be consistent in whether or not we provide default points. I suggest we do not provide the endpoint field at all for any otlp receivers.

We are currently providing default end points in all of the configs such as in here, since default will likely to not change much often from now, I think it will be okay?.

Ahh I see, so the apprunner config is the only one that does not specify defaults. So this is merely a preference of styling. I could go either way on this and don't know if there is a good argument for keeping or leaving them.