This is an automated email from the ASF dual-hosted git repository.

gerlowskija pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr-operator.git


The following commit(s) were added to refs/heads/main by this push:
     new 47446f0  Fix unset-var bug in 'verify_all.sh' script (#749)
47446f0 is described below

commit 47446f0973bade8ebd77c0c619f84474c0fdaa48
Author: Jason Gerlowski <[email protected]>
AuthorDate: Thu Jan 23 12:33:20 2025 -0500

    Fix unset-var bug in 'verify_all.sh' script (#749)
    
    A variable 'artifact' was being referenced outside of the for-in loop
    that created it.  Fixes that bug.
---
 hack/release/smoke_test/verify_all.sh | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/hack/release/smoke_test/verify_all.sh 
b/hack/release/smoke_test/verify_all.sh
index 89f559b..38c6dde 100755
--- a/hack/release/smoke_test/verify_all.sh
+++ b/hack/release/smoke_test/verify_all.sh
@@ -112,12 +112,11 @@ echo "Download all artifacts and verify signatures"
           gpg --verify "${artifact}.asc" "${artifact}" \
             || { echo "Invalid signature for ${artifact}. Aborting!"; exit 1; }
         fi
+        # If a helm chart has a provenance file, and a GPG Key was provided, 
then verify the provenance file
+        if [[ -f "${artifact}.prov" && -n "${GPG_KEY:-}" ]]; then
+          helm verify "${artifact}"
+        fi
       done
-
-      # If a helm chart has a provenance file, and a GPG Key was provided, 
then verify the provenance file
-      if [[ -f "${artifact}.prov" && -n "${GPG_KEY:-}" ]]; then
-        helm verify "${artifact}"
-      fi
     )
   done
 

Reply via email to