open-telemetry / opentelemetry-js

OpenTelemetry JavaScript Client
https://opentelemetry.io
Apache License 2.0
2.65k stars 765 forks source link

Consolidate SDK configuration #4224

Open dyladan opened 10 months ago

dyladan commented 10 months ago

Currently our configurations are spread all over. Some environment variables are read directly by the components and some are ready by the SDK package. This also does not set us up well for file-based configuration which should be coming around the corner, and now is a good time to prepare for that.

I propose creating a separate component for configuration management which handles reading configurations (from environment variables, direct user input, and eventually configuration files), and outputs a config object for consumption by components. For the SDK I believe this will be fairly straightforward, but we should also consider instrumentations which currently have no direct dependence on the SDK.

github-actions[bot] commented 8 months ago

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.