[PATCH compass-ci] lkp-initrd: add step: rsync the result lkp.cgz if needed
[Why] About this container: - Everyone will run this container. - Docker permission is required to run this script. Generally, the production env will control docker permission, and will not give docker permission to everyone. So add the rsync step to allow the execution of local rsync script to perform result lkp.cgz synchronization.
local rsync script path: /usr/local/bin/rsync-lkp-cgz
Signed-off-by: Yu Chuan <13186087857@163.com> --- container/lkp-initrd/run | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/container/lkp-initrd/run b/container/lkp-initrd/run index 522f5098ddc2..e2bff72ab83b 100755 --- a/container/lkp-initrd/run +++ b/container/lkp-initrd/run @@ -24,3 +24,8 @@ cmd=( "${cmd[@]}" echo "result: /srv/initrd/lkp/${lkp_initrd_user:-latest}/lkp-${ARCH}.cgz" + +# rsync lkp-${ARCH}.cgz if needed +if [ -f "/usr/local/bin/rsync-lkp-cgz" ]; then + . "/usr/local/bin/rsync-lkp-cgz" +fi -- 2.23.0
On Tue, Nov 03, 2020 at 04:15:51PM +0800, Yu Chuan wrote:
[Why] About this container: - Everyone will run this container. - Docker permission is required to run this script.
Generally, the production env will control docker permission, and will not give docker permission to everyone.
So add the rsync step to allow the execution of local rsync script to perform result lkp.cgz synchronization.
local rsync script path: /usr/local/bin/rsync-lkp-cgz
Signed-off-by: Yu Chuan <13186087857@163.com> --- container/lkp-initrd/run | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/container/lkp-initrd/run b/container/lkp-initrd/run index 522f5098ddc2..e2bff72ab83b 100755 --- a/container/lkp-initrd/run +++ b/container/lkp-initrd/run @@ -24,3 +24,8 @@ cmd=(
"${cmd[@]}" echo "result: /srv/initrd/lkp/${lkp_initrd_user:-latest}/lkp-${ARCH}.cgz" + +# rsync lkp-${ARCH}.cgz if needed +if [ -f "/usr/local/bin/rsync-lkp-cgz" ]; then + . "/usr/local/bin/rsync-lkp-cgz"
must add here? how about add a scripy on $CCI_SRC/sbin ? i think the run scripy should not do this. Thanks, Shenwei
+fi -- 2.23.0
diff --git a/container/lkp-initrd/run b/container/lkp-initrd/run index 522f5098ddc2..e2bff72ab83b 100755 --- a/container/lkp-initrd/run +++ b/container/lkp-initrd/run @@ -24,3 +24,8 @@ cmd=(
"${cmd[@]}" echo "result: /srv/initrd/lkp/${lkp_initrd_user:-latest}/lkp-${ARCH}.cgz" + +# rsync lkp-${ARCH}.cgz if needed +if [ -f "/usr/local/bin/rsync-lkp-cgz" ]; then + . "/usr/local/bin/rsync-lkp-cgz"
must add here? how about add a scripy on $CCI_SRC/sbin ?
i think the run scripy should not do this.
Good point. Here are the reason: 1. if there is no this file in user's env, the normal function of this script won't be effected. 2. if user have the same needs as us, he also could add this custom script to rsync the result lkp.cgz to anywhere he want. -------- Thanks Yu Chuan
Thanks, Shenwei
+fi -- 2.23.0
participants (2)
-
Xiao Shenwei -
Yu Chuan