xBis7 commented on PR #64902: URL: https://github.com/apache/airflow/pull/64902#issuecomment-4206639363
@jason810496 Thank you, I wasn't aware of this. Airflow needs to talk directly to the `otel-collector` and to be configured to work with it. Additionally, when OTel is enabled for metrics, we have to disable `statsd`. Based on that, it's hard to set it up with `Kustomize` and it should be part of the helm chart. But the 3 observability backends, don't need to interact with Airflow. They are very good example candidates for `Kustomize`. I think I should be able to make it work. As I understand from the Slack discussion, there was a consensus on using `Kustomize` from 1.19.0 and upwards. After this PR, I would like to add integration tests that use OTel and the backends. I don't think setting them up via `Kustomize` will be a problem. I can move forward with the changes. -- 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]
