Skip to content

Commit

Permalink
Fix unset-var bug in 'verify_all.sh' script
Browse files Browse the repository at this point in the history
A variable 'artifact' was being referenced outside of the for-in loop
that created it.  Fixes that bug.
  • Loading branch information
gerlowskija committed Jan 21, 2025
1 parent 8d46989 commit 2259c52
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions hack/release/smoke_test/verify_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down

0 comments on commit 2259c52

Please sign in to comment.