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

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