-
Notifications
You must be signed in to change notification settings - Fork 14.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Speeds up Docker build process by combining RUN and ENV commands #15438
Conversation
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the workflow link to check the reason. |
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
19a28fe
to
dbcca85
Compare
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*. |
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*. |
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*. |
80c9805
to
e652c80
Compare
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the workflow link to check the reason. |
Triggering re-build |
The Dockerfile is more "packed" and certain ARG/ENVs are in separate parts of it but we save minutes in certain scenarios when the images are built (especially when they are built in parallell, the difference might be significant) This change also removes some of the old, already unused CASS_DRIVER ARGS and ENVS. They are not needed any more as cassandra drivers do not require CPYTHON compilation any more.
e652c80
to
76269ce
Compare
…che#15438) The Dockerfile is more "packed" and certain ARG/ENVs are in separate parts of it but we save minutes in certain scenarios when the images are built (especially when they are built in parallell, the difference might be significant) This change also removes some of the old, already unused CASS_DRIVER ARGS and ENVS. They are not needed any more as cassandra drivers do not require CPYTHON compilation any more. (cherry picked from commit 043a88d)
The Dockerfile is more "packed" and certain ARG/ENVs are in separate
parts of it but we save minutes in certain scenarios when the images
are built (especially when they are built in parallell, the
difference might be significant)
This change also removes some of the old, already unused CASS_DRIVER
ARGS and ENVS. They are not needed any more as cassandra drivers do
not require CPYTHON compilation any more.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.