You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by jo...@apache.org on 2022/11/08 06:21:55 UTC

[impala] branch master updated: IMPALA-11703: Set appropriate permissions on /var/tmp in Docker build

This is an automated email from the ASF dual-hosted git repository.

joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new 1899b2e34 IMPALA-11703: Set appropriate permissions on /var/tmp in Docker build
1899b2e34 is described below

commit 1899b2e34b3fd184bcd3101fb5da8ae80479ef25
Author: Joe McDonnell <jo...@cloudera.com>
AuthorDate: Mon Nov 7 16:21:43 2022 -0800

    IMPALA-11703: Set appropriate permissions on /var/tmp in Docker build
    
    Impala will fail to start if the permissions on /var/tmp do
    not have the sticky bit set (i.e. +t). Some Redhat UBI images
    do not set the sticky bit (+t) on /tmp and /var/tmp. This
    sets the sticky bit on those directories during Docker build.
    
    Testing:
     - Verified that the sticky bit is set on one of the affected
       base images and that Impala can start up
    
    Change-Id: I7ff32a035f40cb41d3a8dc80a07fd9924f41b942
    Reviewed-on: http://gerrit.cloudera.org:8080/19222
    Reviewed-by: Abhishek Rawat <ar...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 docker/install_os_packages.sh | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/docker/install_os_packages.sh b/docker/install_os_packages.sh
index f11fe94b4..0a0cb6ecb 100755
--- a/docker/install_os_packages.sh
+++ b/docker/install_os_packages.sh
@@ -154,6 +154,11 @@ if ! locale -a | grep en_US.utf8 ; then
   exit 1
 fi
 
+# Impala will fail to start if the permissions on /var/tmp are not set to include
+# the sticky bit (i.e. +t). Some versions of Redhat UBI images do not have
+# this set by default, so specifically set the sticky bit for both /tmp and /var/tmp.
+chmod a=rwx,o+t /var/tmp /tmp
+
 # To minimize the size for the Docker image, clean up any unnecessary files.
 if [[ $DISTRIBUTION == Ubuntu ]]; then
   apt-get clean