From dac65a439e337948eb51ef0ff4b6494eae7b0f66 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Tue, 29 Nov 2022 13:56:54 +0100 Subject: [PATCH] Backport PR #14780 to 7.17: [DRA] Avoid to loose err code on docker save #14785 * [DRA] Force docker save to save directly on file instead of pipe to another command loosing the execution error code (cherry picked from commit 2e5e49d10d78149ba05a998d9389602861c412eb) Co-authored-by: Andrea Selva --- ci/dra_common.sh | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ci/dra_common.sh b/ci/dra_common.sh index d789237cb..5e4d769d6 100755 --- a/ci/dra_common.sh +++ b/ci/dra_common.sh @@ -11,7 +11,11 @@ function save_docker_tarballs { local arch="${1:?architecture required}" local version="${2:?stack-version required}" for image in logstash logstash-oss logstash-ubi8; do - docker save "docker.elastic.co/logstash/${image}:${version}" | gzip -c > "build/${image}-${version}-docker-image-${arch}.tar.gz" + docker save -o "build/${image}-${version}-docker-image-${arch}.tar" \ + "docker.elastic.co/logstash/${image}:${version}" || \ + error "Hit a problem in saving the Docker image for ${image}" + # NOTE: if docker save exited with non-zero the error log already exited the script + gzip "build/${image}-${version}-docker-image-${arch}.tar" done }