Dev-iL commented on PR #69469: URL: https://github.com/apache/airflow/pull/69469#issuecomment-4984001140
> Looks ok, but maybe we can just install postgres provider for these OL compat tests instead? AFAIU, it will solve the problem? They contain the minimum number of providers already defined in `OPENLINEAGE_E2E_COMPAT_PROVIDERS` in `dev/breeze/src/airflow_breeze/commands/testing_commands.py:1441`. I don't think it will break anything, and will definitely make everything simpler. Tried this - looks like it's not a complete solution. postgres isn't actually imported by any of the OL system-test DAGs (only openlineage/standard/common.compat are), so adding it only mattered as a side channel to get psycopg (v3) physically installed. That part worked for 3.1.8/3.2.2/3.3.0 (which already pin SQLAlchemy 2.0.x in their constraints), but 3.0.6 fails since its pyproject.toml hard-caps `sqlalchemy[asyncio]>=1.4.49,<2.0` (which doesn't have the `+psycopg` dialect) - so no package we install changes that. I reverted back to the explicit-psycopg2 compose override, which is version-agnostic and already proven on CI. -- 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]
