From 7f855be1f96905c44599b944a0d8449ae528ac39 Mon Sep 17 00:00:00 2001 From: Jianxun Zhang Date: Mon, 5 Aug 2019 12:26:45 -0700 Subject: docker: support passing extra arguments to scripts One of the usages is to specify proxy setting so that these scripts can download packages behind proxies. It looks cleaner and more flexible than tweaking the docker file that's checked in the repo. BUG=None TEST=run commands behind a network proxy ./build_crosvm_base.sh --build-arg http_proxy --build-arg https_proxy ./build_crosvm.sh --build-arg http_proxy --build-arg https_proxy ./wrapped_smoke_test.sh -e http_proxy -e https_proxy Change-Id: Id3729a1343b12c1c7553e047701f770620e3c975 Signed-off-by: Jianxun Zhang Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/1737540 Reviewed-by: Zach Reizner Tested-by: kokoro --- docker/build_crosvm.sh | 7 ++++++- docker/build_crosvm_base.sh | 2 +- docker/wrapped_smoke_test.sh | 1 + 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/docker/build_crosvm.sh b/docker/build_crosvm.sh index 3c6d359..e0d697b 100755 --- a/docker/build_crosvm.sh +++ b/docker/build_crosvm.sh @@ -8,4 +8,9 @@ cd "${0%/*}" src_root="$(realpath ..)" -docker build -t crosvm -f Dockerfile.crosvm --build-arg UID --build-arg GID "${src_root}" +docker build -t crosvm \ + -f Dockerfile.crosvm \ + --build-arg UID \ + --build-arg GID \ + "$@" \ + "${src_root}" diff --git a/docker/build_crosvm_base.sh b/docker/build_crosvm_base.sh index 2041384..0e4274e 100755 --- a/docker/build_crosvm_base.sh +++ b/docker/build_crosvm_base.sh @@ -12,4 +12,4 @@ gen_build_args() { done } -docker build $(gen_build_args) -t crosvm-base . +docker build $(gen_build_args) -t crosvm-base "$@" . diff --git a/docker/wrapped_smoke_test.sh b/docker/wrapped_smoke_test.sh index 324ab7c..1ee296b 100755 --- a/docker/wrapped_smoke_test.sh +++ b/docker/wrapped_smoke_test.sh @@ -14,6 +14,7 @@ docker run \ -e TEST_RUNNER_FLAGS='--format terse' \ -v /dev/log:/dev/log \ -v "${src_root}":/platform/crosvm:ro \ + "$@" \ crosvm-base \ bin/smoke_test -- cgit 1.4.1