codecae opened a new pull request, #56634: URL: https://github.com/apache/airflow/pull/56634
The current OpenTelemetry implementation has a fairly rigid configuration specification for endpoint and interval configurations. While this is probably okay for some configurations, there are cases in which endpoints do not conform with the standard implied by Airflow's implementation. See https://prometheus.io/docs/guides/opentelemetry/#send-opentelemetry-metrics-to-the-prometheus-server for an example of how the API URL can stray from the basic "/v1/metrics" route, yet still be entirely valid. Instead of adding new configuration values to the metrics and traces sections, this PR adds transparent support for standard all OpenTelemetry environment variables (so far) that would otherwise be overridden by Airflow's configuration. This support has been extended beyond just the metrics and traces endpoints in order to have full transparency. This change includes the support for: - OTEL_EXPORTER_OTLP_ENDPOINT - OTEL_EXPORTER_OTLP_METRICS_ENDPOINT - OTEL_EXPORTER_OTLP_TRACES_ENDPOINT - OTEL_METRIC_EXPORT_INTERVAL -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
