Skip to content

Commit

Permalink
Error reporting fixed for checking status of minio (#2378)
Browse files Browse the repository at this point in the history
* error responding fixed for minio check

* Comment in build/test.sh updated

Co-authored-by: Vivek Singh <vsingh.ggits.2010@gmail.com>

---------

Co-authored-by: Vivek Singh <vsingh.ggits.2010@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
  • Loading branch information
3 people committed Oct 5, 2023
1 parent e01c8eb commit f281f51
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down

0 comments on commit f281f51

Please sign in to comment.