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

Use ${env:ENV} style syntax when getting environment variables #761

Closed
mx-psi opened this issue Apr 26, 2023 · 1 comment · Fixed by #767
Closed

Use ${env:ENV} style syntax when getting environment variables #761

mx-psi opened this issue Apr 26, 2023 · 1 comment · Fixed by #767
Assignees
Labels
chart:collector Issue related to opentelemetry-collector helm chart

Comments

@mx-psi
Copy link
Member

mx-psi commented Apr 26, 2023

The OpenTelemetry Collector supports both the legacy ${ENV} syntax as well as the new ${env:ENV} syntax (enabled by default since v0.62.0, see open-telemetry/opentelemetry-collector@b15179e, fully working for the use case of this Helm chart since v0.71.0, see open-telemetry/opentelemetry-collector@202c030). The latter is preferred and we have updated examples in the OpenTelemetry Collector repository to use it (see open-telemetry/opentelemetry-collector-contrib/issues/17299).

The Helm chart uses the legacy syntax since #603, see

receivers:
jaeger:
protocols:
grpc:
endpoint: ${MY_POD_IP}:14250
thrift_http:
endpoint: ${MY_POD_IP}:14268
thrift_compact:
endpoint: ${MY_POD_IP}:6831
otlp:
protocols:
grpc:
endpoint: ${MY_POD_IP}:4317
http:
endpoint: ${MY_POD_IP}:4318

We should use the new ${env:ENV} style syntax here instead. Note that this would break users of the Collector < v0.71.0 (not sure how this is handled in the Helm chart, but we may want to wait a bit).

@JaredTan95
Copy link
Member

I'd like to update this, pls assign it to me :-P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart:collector Issue related to opentelemetry-collector helm chart
Projects
None yet
3 participants