potiuk commented on pull request #13409:
URL: https://github.com/apache/airflow/pull/13409#issuecomment-753004254


   I believe I managed to improve the "eager constraint update" mechanism in 
the way that it removes all provider-imposed requirements out of the setup.cfg 
/ install_requires (!). It works exactly as before when it comes to constraints 
installation and automated constraint upgrade, but this time those requirements 
are not present in the install_requires section of the `apache-airflow` package.
   
   I simply add the imposed requirements in the images build steps where eager 
upgrade strategy is used. This seems to keep the same properties as previous 
solution where those limits were added to "install-requires" but those limits 
are only used during the image build and airflow package does not have those.
   
   Also with --upgrade-to-newer-dependencies switch of breeze you can run the 
very same "eager upgrade" locally and it makes it much easier to locally test 
new constraints without the need to push them as PR. It now becomes rather 
straightforward how to fix any future similar conflict. 
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to