[PATCH compass-ci] container/openresty-proxy-cache: fix build error

when execute openresty-proxy-cache/build, if openresty already exists [error] fatal: destination path 'openresty' already exists and is not an empty directory. [how] delete openresty after finish build container. Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- container/openresty-proxy-cache/build | 2 ++ 1 file changed, 2 insertions(+) diff --git a/container/openresty-proxy-cache/build b/container/openresty-proxy-cache/build index 0846197..e77c19e 100755 --- a/container/openresty-proxy-cache/build +++ b/container/openresty-proxy-cache/build @@ -5,3 +5,5 @@ git clone https://gitee.com/cuiyili/openresty.git || exit 1 docker build -t openresty:proxy_cache . -f openresty/Dockerfile + +rm -rf openresty -- 2.23.0

Reviewed-by: Xiao Shenwei <xiaoshenwei96@163.com> On Wed, Feb 03, 2021 at 09:32:30AM +0800, Liu Yinsi wrote:
when execute openresty-proxy-cache/build, if openresty already exists [error] fatal: destination path 'openresty' already exists and is not an empty directory.
[how] delete openresty after finish build container.
Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- container/openresty-proxy-cache/build | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/container/openresty-proxy-cache/build b/container/openresty-proxy-cache/build index 0846197..e77c19e 100755 --- a/container/openresty-proxy-cache/build +++ b/container/openresty-proxy-cache/build @@ -5,3 +5,5 @@ git clone https://gitee.com/cuiyili/openresty.git || exit 1
docker build -t openresty:proxy_cache . -f openresty/Dockerfile + +rm -rf openresty -- 2.23.0

On Wed, Feb 03, 2021 at 09:32:30AM +0800, Liu Yinsi wrote:
when execute openresty-proxy-cache/build, if openresty already exists [error] fatal: destination path 'openresty' already exists and is not an empty directory.
[how] delete openresty after finish build container.
Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- container/openresty-proxy-cache/build | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/container/openresty-proxy-cache/build b/container/openresty-proxy-cache/build index 0846197..e77c19e 100755 --- a/container/openresty-proxy-cache/build +++ b/container/openresty-proxy-cache/build @@ -5,3 +5,5 @@ git clone https://gitee.com/cuiyili/openresty.git || exit 1
docker build -t openresty:proxy_cache . -f openresty/Dockerfile + +rm -rf openresty
How about move the 'rm' step at the before of 'docker build'? -------- Thanks Yu Chuan
-- 2.23.0

+++ b/container/openresty-proxy-cache/build @@ -5,3 +5,5 @@ git clone https://gitee.com/cuiyili/openresty.git || exit 1
docker build -t openresty:proxy_cache . -f openresty/Dockerfile + +rm -rf openresty
How about move the 'rm' step at the before of 'docker build'?
good Thanks, Yinsi
-------- Thanks Yu Chuan
-- 2.23.0
participants (3)
-
Liu Yinsi
-
Xiao Shenwei
-
Yu Chuan