potiuk commented on a change in pull request #20664:
URL: https://github.com/apache/airflow/pull/20664#discussion_r782044205



##########
File path: Dockerfile
##########
@@ -327,34 +329,6 @@ RUN if [[ -f /docker-context-files/requirements.txt ]]; 
then \
         pip install --no-cache-dir --user -r 
/docker-context-files/requirements.txt; \
     fi
 

Review comment:
       BTW. We do not need those labels any more because we are not pushing the 
'build' segment to our repo any more. This yet another great feature of 
buildkit cache, that you can built it with "mode=max" where the cache will 
contain layers for all segments - the final and intermediate ones. 




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


Reply via email to