[PATCH compass-ci] sparrow/3-code: umask 002 before download git trees
because ENV CCI_SRC LKP_SRC, REPO_SRC all under /c, some containers service start need umask 002, set it before download git trees. Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- sparrow/3-code/git | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sparrow/3-code/git b/sparrow/3-code/git index 5588696..494107e 100755 --- a/sparrow/3-code/git +++ b/sparrow/3-code/git @@ -2,6 +2,9 @@ # SPDX-License-Identifier: MulanPSL-2.0+ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved. +# fix files permissions cause container service start failed +umask 002 + # download git trees cd /c || exit -- 2.23.0
On Wed, Nov 18, 2020 at 02:30:10PM +0800, Liu Yinsi wrote:
because ENV CCI_SRC LKP_SRC, REPO_SRC all under /c, some containers service start need umask 002, set it before download git trees.
Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- sparrow/3-code/git | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/sparrow/3-code/git b/sparrow/3-code/git index 5588696..494107e 100755 --- a/sparrow/3-code/git +++ b/sparrow/3-code/git @@ -2,6 +2,9 @@ # SPDX-License-Identifier: MulanPSL-2.0+ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved.
+# fix files permissions cause container service start failed +umask 002 + # download git trees
cd /c || exit
What about /c? Should /c also be need to modify umask before creating /c? And further, how about the follow: yuchuan@crystal ~/compass-ci/sparrow% gg mkdir 1-storage/large:92:mkdir -p /srv/redis 1-storage/large:93:mkdir -p /srv/etcd 1-storage/large:94:mkdir -p /srv/es 1-storage/large:95:mkdir -p /srv/os 1-storage/large:96:mkdir -p /srv/result 1-storage/large:97:mkdir -p /srv/initrd 1-storage/large:98:mkdir -p /srv/git 1-storage/large:107:mkdir -p /cci 1-storage/large:109:mkdir -p /backup 1-storage/large:111:mkdir -p /var/lib/docker 1-storage/large:113:mkdir -p /c 1-storage/tiny:25:mkdir -p "${dirs[@]}" 3-code/dev-env:17:mkdir -p /etc/compass-ci/defaults 4-docker/buildall:30: mkdir $tmpdir/$container_name 2>/dev/null && 5-build/ipxe:23:mkdir -p /tftpboot/ipxe/bin-arm64-efi/ -------- Thanks Yu Chuan
-- 2.23.0
+umask 002 + # download git trees
cd /c || exit
What about /c? Should /c also be need to modify umask before creating /c?
And further, how about the follow: yuchuan@crystal ~/compass-ci/sparrow% gg mkdir 1-storage/large:92:mkdir -p /srv/redis 1-storage/large:93:mkdir -p /srv/etcd 1-storage/large:94:mkdir -p /srv/es 1-storage/large:95:mkdir -p /srv/os 1-storage/large:96:mkdir -p /srv/result 1-storage/large:97:mkdir -p /srv/initrd 1-storage/large:98:mkdir -p /srv/git 1-storage/large:107:mkdir -p /cci 1-storage/large:109:mkdir -p /backup 1-storage/large:111:mkdir -p /var/lib/docker 1-storage/large:113:mkdir -p /c 1-storage/tiny:25:mkdir -p "${dirs[@]}" 3-code/dev-env:17:mkdir -p /etc/compass-ci/defaults 4-docker/buildall:30: mkdir $tmpdir/$container_name 2>/dev/null && 5-build/ipxe:23:mkdir -p /tftpboot/ipxe/bin-arm64-efi/
good catch Often, directory is 775, file is 664, maybe some new files have missing, i will check. Thanks, Yinsi compass-ci/sparrow/1-storage% cat permission chown lkp:lkp /srv/result chown lkp:lkp /srv/initrd chown -R lkp:lkp /srv/es chown lkp:lkp /srv/cache/netdata_cache chown lkp:lkp /srv/cache/netdata_lib chown lkp:lkp /srv/cci/serial/fluentd-pos chown lkp:committer /srv/git chgrp team /srv/cci/serial/logs chmod 775 /srv/result chmod 775 /srv/es/logging-es chmod 775 /srv/es chmod 775 /srv/cache/netdata_cache chmod 775 /srv/cache/netdata_lib chmod 775 /srv/cci/serial/logs chmod 775 /srv/cci/serial/fluentd-pos chgrp team /srv/dc chmod g+ws /srv/dc
-------- Thanks Yu Chuan
-- 2.23.0
participants (2)
-
Liu Yinsi -
Yu Chuan