Error reporting fixed for checking status of minio (#2378)

* error responding fixed for minio check * Comment in build/test.sh updated Co-authored-by: Vivek Singh <[email protected]> --------- Co-authored-by: Vivek Singh <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
kanisterio · Oct 5, 2023 · f281f51 · f281f51
1 parent e01c8eb
commit f281f51
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/build/test.sh b/build/test.sh
@@ -51,8 +51,8 @@ fi
 echo
 
 check_dependencies() {
-    # Check if minio is already deployed
-    if helm status minio -n minio > /dev/null 2>&1 ; then
+    # Check if minio is already deployed. We suppress only `stdout` and not `stderr` to make sure we catch errors if `helm status` fails
+    if helm status minio -n minio 1> /dev/null ; then
         # Setting env vars to access MinIO
         export S3_COMPLIANT_AWS_ACCESS_KEY_ID="AKIAIOSFODNN7EXAMPLE"
         export S3_COMPLIANT_AWS_SECRET_ACCESS_KEY="wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY"