From df34e9197861df0a94cbec11076d2cf9159d63f1 Mon Sep 17 00:00:00 2001 From: Binbin Zhang Date: Mon, 29 Nov 2021 22:37:19 +0800 Subject: [PATCH] osbuilder: fix missing cpio package when building rootfs-initrd image 1. install cpio package before building rootfs-initrd image 2. add `pipefaili;errexit` check to the scripts Fixes: #3144 Signed-off-by: Binbin Zhang (cherry picked from commit 8ee67aae4ff02526b65ba669a954fdf2b5eb1223) --- tools/osbuilder/image-builder/image_builder.sh | 6 ++++-- tools/osbuilder/initrd-builder/initrd_builder.sh | 6 ++++-- .../kata-deploy/local-build/dockerbuild/Dockerfile | 13 +++++++------ 3 files changed, 15 insertions(+), 10 deletions(-) diff --git a/tools/osbuilder/image-builder/image_builder.sh b/tools/osbuilder/image-builder/image_builder.sh index 5883ad2a4..a1cf97eae 100755 --- a/tools/osbuilder/image-builder/image_builder.sh +++ b/tools/osbuilder/image-builder/image_builder.sh @@ -4,9 +4,11 @@ # # SPDX-License-Identifier: Apache-2.0 -set -e +[ -z "${DEBUG}" ] || set -x -[ -n "${DEBUG}" ] && set -x +set -o errexit +# set -o nounset +set -o pipefail DOCKER_RUNTIME=${DOCKER_RUNTIME:-runc} diff --git a/tools/osbuilder/initrd-builder/initrd_builder.sh b/tools/osbuilder/initrd-builder/initrd_builder.sh index 531350289..b556d4f4a 100755 --- a/tools/osbuilder/initrd-builder/initrd_builder.sh +++ b/tools/osbuilder/initrd-builder/initrd_builder.sh @@ -4,9 +4,11 @@ # # SPDX-License-Identifier: Apache-2.0 -set -e +[ -z "${DEBUG}" ] || set -x -[ -n "$DEBUG" ] && set -x +set -o errexit +# set -o nounset +set -o pipefail script_name="${0##*/}" script_dir="$(dirname $(readlink -f $0))" diff --git a/tools/packaging/kata-deploy/local-build/dockerbuild/Dockerfile b/tools/packaging/kata-deploy/local-build/dockerbuild/Dockerfile index 8df4c1474..89b1f0447 100644 --- a/tools/packaging/kata-deploy/local-build/dockerbuild/Dockerfile +++ b/tools/packaging/kata-deploy/local-build/dockerbuild/Dockerfile @@ -2,7 +2,7 @@ # # SPDX-License-Identifier: Apache-2.0 -FROM ubuntu +FROM ubuntu:20.04 ENV DEBIAN_FRONTEND=noninteractive ENV INSTALL_IN_GOPATH=false @@ -27,11 +27,12 @@ RUN sh -c "echo '${IMG_USER} ALL=NOPASSWD: ALL' >> /etc/sudoers" #FIXME: gcc is required as agent is build out of a container build. RUN apt-get update && \ - apt install -y \ - gcc \ - git \ - make \ - xz-utils + apt install --no-install-recommends -y \ + cpio \ + gcc \ + git \ + make \ + xz-utils ENV USER ${IMG_USER} USER ${UID}:${GID}