From 32008bc1b41bedcee2dddfc0b2333a7b37c35f15 Mon Sep 17 00:00:00 2001
From: Liam Newman <96086065+liam-verta@users.noreply.github.com>
Date: Mon, 13 Jun 2022 01:10:29 -0700
Subject: [PATCH] Report error when component_unpack tar command fails (#257)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Report error when component_unpack tar command fails

Signed-off-by: Liam Newman <liam@verta.ai>

* Apply suggestions from code review

Co-authored-by: Juan José Martos <jotamartos@gmail.com>
Signed-off-by: Liam Newman <liam@verta.ai>

Co-authored-by: Juan José Martos <jotamartos@gmail.com>
---
 .../debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh    | 2 +-
 .../debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh    | 2 +-
 .../debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh    | 2 +-
 .../debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh    | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/bitnami/kafka/2.8/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh b/bitnami/kafka/2.8/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
index e3098a93b7c0..86cd48369ed4 100644
--- a/bitnami/kafka/2.8/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
+++ b/bitnami/kafka/2.8/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
@@ -60,6 +60,6 @@ component_unpack() {
         echo "Verifying package integrity"
         echo "$package_sha256  ${base_name}.tar.gz" | sha256sum --check - || exit "$?"
     fi
-    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2
+    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2 || return "$?"
     rm "${base_name}.tar.gz"
 }
diff --git a/bitnami/kafka/3.0/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh b/bitnami/kafka/3.0/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
index e3098a93b7c0..86cd48369ed4 100644
--- a/bitnami/kafka/3.0/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
+++ b/bitnami/kafka/3.0/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
@@ -60,6 +60,6 @@ component_unpack() {
         echo "Verifying package integrity"
         echo "$package_sha256  ${base_name}.tar.gz" | sha256sum --check - || exit "$?"
     fi
-    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2
+    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2 || return "$?"
     rm "${base_name}.tar.gz"
 }
diff --git a/bitnami/kafka/3.1/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh b/bitnami/kafka/3.1/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
index e3098a93b7c0..86cd48369ed4 100644
--- a/bitnami/kafka/3.1/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
+++ b/bitnami/kafka/3.1/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
@@ -60,6 +60,6 @@ component_unpack() {
         echo "Verifying package integrity"
         echo "$package_sha256  ${base_name}.tar.gz" | sha256sum --check - || exit "$?"
     fi
-    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2
+    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2 || return "$?"
     rm "${base_name}.tar.gz"
 }
diff --git a/bitnami/kafka/3.2/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh b/bitnami/kafka/3.2/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
index e3098a93b7c0..86cd48369ed4 100644
--- a/bitnami/kafka/3.2/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
+++ b/bitnami/kafka/3.2/debian-10/prebuildfs/opt/bitnami/scripts/libcomponent.sh
@@ -60,6 +60,6 @@ component_unpack() {
         echo "Verifying package integrity"
         echo "$package_sha256  ${base_name}.tar.gz" | sha256sum --check - || exit "$?"
     fi
-    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2
+    tar --directory "${directory}" --extract --gunzip --file "${base_name}.tar.gz" --no-same-owner --strip-components=2 || return "$?"
     rm "${base_name}.tar.gz"
 }