From ce718920c6a14f33f79992e90a2740a91c0ba4f7 Mon Sep 17 00:00:00 2001 From: Eryu Guan <45746212+eryugey@users.noreply.github.com> Date: Mon, 16 May 2022 14:10:00 +0800 Subject: [PATCH] fix: remove container after generating protoc (#1306) Add '--rm' option to docker command, so container will be removed automatically. Otherwise we'll have many exited but not removed containers as we run 'make protoc'. Signed-off-by: Eryu Guan --- hack/protoc.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hack/protoc.sh b/hack/protoc.sh index e6cfb0e23..fe551dd86 100755 --- a/hack/protoc.sh +++ b/hack/protoc.sh @@ -9,7 +9,7 @@ proto_modules="base cdnsystem dfdaemon manager scheduler" echo "generate protos..." for module in ${proto_modules}; do - if docker run -v $PWD:/defs ${PROTOC_ALL_IMAGE} \ + if docker run --rm -v $PWD:/defs ${PROTOC_ALL_IMAGE} \ -d ${PROTO_PATH}/$module -i . \ -l ${LANGUAGE} -o . \ --go-source-relative \