You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
BTW the code above is very similar to the current initialization code that runs in Helidon MP, which, because it complies with the MP Telemetry spec, allows otel configuration to appear at the "top level" of MP configuration.
I furnished the above code to Ali as a way to quickly replicate that functionality in an SE app.
We will need to consider carefully where in the config hierarchy we would want to expose the otel settings in SE config.
Other tracing providers (Jaeger, Zipkin) have historically taken their settings from the tracing section. The specific config keys within tracing that are supported would depend on which tracing provider is in use. From that perspective and for consistency we might locate the OTel settings under tracing, perhaps without the otel prefix. That, of course, would be inconsistent with the MP Telemetry approach.
About environment variables... Helidon Config supports environment variables as a config source by default.
Further, OpenTelemetry itself checks for various environment variables and varies its behavior accordingly. So that is another workaround unless/until Helidon SE supports OTel configuration in its own config.
Environment Details
Problem Description
otel configuration is not currently loaded by Helidon:
We can work around this for now using the following code:
However, it would be preferable to be able to load these either by config file or by environment variables.
Steps to reproduce
The text was updated successfully, but these errors were encountered: