ferruzzi opened a new pull request, #30160: URL: https://github.com/apache/airflow/pull/30160
Completes the Breeze OpenTelemetry integration and adds a banner to the UI indicating it is currently experimental. (Final?) PR to add some groundwork to allow OTel support. Follow-up to https://github.com/apache/airflow/pull/29521 After getting a working working PoC, the Breeze integration needed a tweak to work correctly. Changes have been tested against a working* proof-of-concept which can be found [here](https://github.com/ferruzzi/airflow/tree/ferruzzi/otel/poc) for now if anyone wants to try them out. *So far only a subset of metrics are working in that POC, but the ones that ARE working are working end to end. Major changes: - Corrects the OpenTelemetry Collector port to 8889 instead of 8888 - Adds OTel values to the Airflow Config - Adds the required new Environment Variables tot he integration's docker-compose file to set those config values - If Otel is enabled, adds a banner to the UI showing that OTel support is currently experimental - Edits to scripts/ci/docker-compose/otel-collector-config.yml to get it working as expected and trim some fat Minor Changes: - Rephrasing some docs - Cleans up some unnecessary port forwarding in the integration's docker-compose file - Cleans up some extensions in the OTel collector config file that ended up being unnecessary -- 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]
