elasticotel: load resource detector as last#532
Conversation
Since OTel SDK 1.42.0 the order of the resource detectors is preserved. Move the otel resource detector as last so we can override other resource detector attributes from environment variables.
🔍 Preview links for changed docs |
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthrough
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What does this pull request do?
Since OTel SDK 1.42.0 the order of the resource detectors is preserved. Move the
otelresource detector as last so we can override other resource detector attributes from environment variables.Related issues