Re: [PATCH crystal-ci] crystal-ci/container: add container for rsync service

On Thu, Aug 20, 2020 at 08:29:55PM +0800, Zhang Yu wrote:
Signed-off-by: Zhang Yu <zhangyu424@huawei.com> +RUN apk add --no-cache rsync openssh \ + && rm -rf /var/cache/apk/* \ + && adduser lkp -u 1090 -g 1090 -D
in sparrow/1-storage/permission, already add permission chown lkp:lkp /srv/result no need to adduser in Dockerfile.
+docker_rm rsync_server +cmd=( + docker run + -id
-i no need. --restart=always ^ if container needs to start always, add --restart=always.
+ --name=rsync_server + -v /srv/result:/result ^ is /srv/result?
Thanks, Yinsi

On Fri, Aug 21, 2020 at 11:10:18AM +0800, Liu Yinsi wrote:
On Thu, Aug 20, 2020 at 08:29:55PM +0800, Zhang Yu wrote:
Signed-off-by: Zhang Yu <zhangyu424@huawei.com> +RUN apk add --no-cache rsync openssh \ + && rm -rf /var/cache/apk/* \ + && adduser lkp -u 1090 -g 1090 -D
in sparrow/1-storage/permission, already add permission chown lkp:lkp /srv/result no need to adduser in Dockerfile.
Ok, we can delete it and test it ok.
+docker_rm rsync_server +cmd=( + docker run + -id
-i no need.
good.
--restart=always ^ if container needs to start always, add --restart=always.
Ok, got it. Thanks, zhangyu
+ --name=rsync_server + -v /srv/result:/result ^ is /srv/result?
Thanks, Yinsi
participants (2)
-
Liu Yinsi
-
Zhang Yu