Fix Typo on heimdall env: OTEL_TRACES_EXPORTER#110
Conversation
Fixes a typo on the OTEL_TRACES_EXPORTER environment variable name. Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
WalkthroughA single environment variable is renamed in the Helm chart's values configuration for Heimdall's OpenTelemetry tracing setup, changing Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Comment |
There was a problem hiding this comment.
Pull request overview
This PR corrects an OpenTelemetry environment variable name in the Helm values for the Heimdall deployment, ensuring tracing exporter configuration is picked up by the OpenTelemetry SDK as intended.
Changes:
- Rename
OTEL_TRACE_EXPORTER→OTEL_TRACES_EXPORTERunderheimdall.envincharts/lfx-platform/values.yaml.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fixes a typo on the OTEL_TRACES_EXPORTER environment variable name.
Signed-off-by: Trevor Bramwell tbramwell@linuxfoundation.org