You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/01/21 16:44:23 UTC
[I] Run lowest deps checks that include lowest version of cross-provider deps as well\ [airflow]
potiuk opened a new issue, #60888:
URL: https://github.com/apache/airflow/issues/60888
### Body
Currently we run lowest-deps tests for individual providers, however, they do not **really** test if the min version of cross-provider dependency is good - because they are installing all providers from sources. This is a problem especially for common.compat - because all providers are using it and they have different min-version requirements.
We are using `uv sync --resolution lowest-direct` - but in the workspace, anyhow the source version of other providers are used.
We should be able to lower-down also cross-provider deps. This can be done by either:
* if possible in uv - exclude other providers from workspace during the uv-sync
or
* if not posible to do - manualy update the workspace to remove those providers from workspace definition so that they are installed from PyPI
This should be skipped for all packages with `# use next version` in the provider being tested.
### Committer
- [x] I acknowledge that I am a maintainer/committer of the Apache Airflow project.
--
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: commits-unsubscribe@airflow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "Prab-27 (via GitHub)" <gi...@apache.org>.
Prab-27 commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-6192338330
I'm really sorry, @dabla, that I couldn't complete this over the last few days. I have been busy - some family responsibilities and a job search and 2 open PRs
I will be much more careful with my words next time!
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-3779899186
Feel Free.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "Prab-27 (via GitHub)" <gi...@apache.org>.
Prab-27 commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-4103230259
I tried, but first solution (`if possible in uv - exclude other providers from workspace during the uv-sync.` ) not possible since the uv sync internally handle this using` uv lock`
I’m working on the second - `if not possible to do - manually update the workspace to remove those providers from the workspace definition so that they are installed from PyPI`
I followed this approach - removed` common-compact` from the `workspace member` or also `dev` if mentioned - that way we can remove it from installing from sources during uv-sync. But I don’t understand what type of group we can add this dependency to so it can be used and also installed from PyPI, or does it get worse?
I think it might be better if we handle this type of dependency from the uv side - for example, by implementing a flag`(# use next version`) like #pypi, or a mechanism like an `Index flag for only PYPI`. In that case, we’d need to implement this as a workspace member that can only be installed from PyPI (e.g., index = PyPI). If yes I'd happy to raise a PR there
@potiuk Would you please share your thoughts on this ?
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "Prab-27 (via GitHub)" <gi...@apache.org>.
Prab-27 commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-3779811523
@potiuk , I’d like to participate in this, if it’s fine for me to work on it
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "Prab-27 (via GitHub)" <gi...@apache.org>.
Prab-27 commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-3987994682
I'm working on this
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "Prab-27 (via GitHub)" <gi...@apache.org>.
Prab-27 commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-5614719801
Restarting my work on this issue !
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "Prab-27 (via GitHub)" <gi...@apache.org>.
Prab-27 commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-3922377167
Working hard; I’ll raise a PR as soon as possible.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Run lowest deps checks that include lowest version of cross-provider deps as well [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on issue #60888:
URL: https://github.com/apache/airflow/issues/60888#issuecomment-3986411445
@Prab-27 We are unassigning you from this issue as part of our updated [assignment policy](https://github.com/apache/airflow/blob/main/contributing-docs/04_how_to_contribute.rst#contribute-code-changes).
This is not meant to discourage your contribution — quite the opposite! You are still very welcome to work on this issue and submit a PR for it. Simply comment that you are working on it and open a PR when ready.
We found that formal assignments were not working well, as they often prevented others from contributing when the assignee was not actively working on the issue.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org