potiuk commented on code in PR #42963: URL: https://github.com/apache/airflow/pull/42963#discussion_r1797825820
########## Dockerfile.ci: ########## @@ -1181,7 +1181,7 @@ ARG PYTHON_BASE_IMAGE ARG AIRFLOW_IMAGE_REPOSITORY="https://github.com/apache/airflow"; # By increasing this number we can do force build of all dependencies -ARG DEPENDENCIES_EPOCH_NUMBER="11" +ARG DEPENDENCIES_EPOCH_NUMBER="12" Review Comment: Actually it seems that it does not work this way. The sqlalchemy-redshift connector is still installed in our "main" caching step. You can check it by looking at the constraints generated as artifacts in this build - sqlalchemy-redshift connector is still there. The reason is, because we are using caching from the `main` installation for this PR - and in main, the dependency is still there. I think I will need to add a new label that you should be able to set for such PR to not use cache during the image build. In breeze this is done with `--docker-cache disabled` and in CI this is triggered when `disable image cache` label is set. I will apply the label and re-run it. If it will work, I will update the docs/comments to make it clear that you need to do it as well. -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
