xBis7 commented on PR #56187: URL: https://github.com/apache/airflow/pull/56187#issuecomment-3407595850
@jason810496 @ferruzzi @ashb It's ready for review. The CI is green. The step for running the task sdk integration tests seems to be flaky. https://github.com/apache/airflow/actions/runs/18505099702/job/52734949908?pr=56187 I ran it locally and it worked, so I triggered the CI again and it passed. To help you understand the changes faster * The new `observability` package has been moved under `shared` * The unit tests have also been moved there * The otel integration tests haven't been moved because I don't think there is a mechanism to run them on the CI if the change is made * `airflow-core` and `task-sdk` both have their own version of `shared` directory. The files aren't copies but symlinks to the original ones * `airflow-core` imports from its `shared` symlinks * `task-sdk` imports from its own `shared` symlinks * `providers` import from the `task-sdk/shared` symlinks * in the `task-sdk` and the `providers`, the imports are wrapped with `try-except` for backwards compat -- 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]
