From e50f5b916e3a00855053278695846f98f31f8e06 Mon Sep 17 00:00:00 2001 From: Amir Abushareb Date: Mon, 14 Aug 2023 11:09:42 +0100 Subject: [PATCH] docker/bootstrap: remove --no-cache flag It seems like this was added back in 2015 when vitess used to use raw go get commands to build the image, the reasoning behind it was to ensure that the image has the most recent dependencies. This is no longer needed because in the image we correctly add go.* dep manifests and call go mod download. PR where this was added: https://github.com/vitessio/vitess/pull/665 Signed-off-by: Amir Abushareb --- docker/bootstrap/build.sh | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/docker/bootstrap/build.sh b/docker/bootstrap/build.sh index 8cc6a151606..d84e37fced9 100755 --- a/docker/bootstrap/build.sh +++ b/docker/bootstrap/build.sh @@ -60,5 +60,10 @@ while [ $# -gt 0 ]; do done if [ -f "docker/bootstrap/Dockerfile.$flavor" ]; then - docker build --no-cache -f docker/bootstrap/Dockerfile.$flavor -t $image --build-arg bootstrap_version=$version --build-arg image=$base_image . + docker build \ + -f docker/bootstrap/Dockerfile.$flavor \ + -t $image \ + --build-arg bootstrap_version=$version \ + --build-arg image=$base_image \ + . fi