This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch v1-10-test in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 40ebe6eb41a935cb83bff4ef80106347aa929f8c Author: Jarek Potiuk <[email protected]> AuthorDate: Mon Nov 9 22:01:58 2020 +0100 Fix permissions of mounted /tmp directory for Breeze (#12157) The "tmp" directory is mounted from the host (from tmp folder in the source airflow directory). This is needed to get some of our docker-in-docker tools (such as gcloud/aws/java) and get them working on demand. Thanks to that we do not have to increase the size of CI image unnecessarily. Those tools were introduced and made to work in #9376 However this causes some of the standard tools (such as apt-get) to not work inside the container unless the mounted /tmp folder has write permission for groups/other. This PR fixes it. (cherry picked from commit a42bbe21c899f4048b0861df2c84eb2ba9c5eeb5) --- scripts/in_container/entrypoint_ci.sh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/scripts/in_container/entrypoint_ci.sh b/scripts/in_container/entrypoint_ci.sh index 37d3199..057d630 100755 --- a/scripts/in_container/entrypoint_ci.sh +++ b/scripts/in_container/entrypoint_ci.sh @@ -31,6 +31,11 @@ function disable_rbac_if_requested() { # shellcheck source=scripts/in_container/_in_container_script_init.sh . /opt/airflow/scripts/in_container/_in_container_script_init.sh +# Add "other" and "group" write permission to the tmp folder +# Note that it will also change permissions in the /tmp folder on the host +# but this is necessary to enable some of our CLI tools to work without errors +chmod 1777 /tmp + AIRFLOW_SOURCES=$(cd "${IN_CONTAINER_DIR}/../.." || exit 1; pwd) PYTHON_MAJOR_MINOR_VERSION=${PYTHON_MAJOR_MINOR_VERSION:=3.6}
