scripts: dont run subshell in for loop (#60611)

For same reason as https://github.com/sourcegraph/sourcegraph/pull/60610, cleaning up the remaining scripts

## Test plan

https://stackoverflow.com/questions/44014857/function-invocation-in-for-loop-does-not-fail-even-with-set-e
This commit is contained in:
Noah S-C 2024-02-19 14:48:33 +00:00 committed by GitHub
parent b771497115
commit 4d76a70fa2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 6 additions and 3 deletions

View File

@ -17,7 +17,8 @@ GOOGLE_IMAGE_NAME="${NAME}"
"$gcloud" compute images update --project=sourcegraph-ci "${GOOGLE_IMAGE_NAME}" --family="${IMAGE_FAMILY}"
if [ "${EXECUTOR_IS_TAGGED_RELEASE}" = "true" ]; then
for region in $(jq -r '.[]' <aws_regions.json); do
REGIONS=$(jq -r '.[]' <aws_regions.json)
for region in $REGIONS; do
AWS_AMI_ID=$(aws ec2 --region="${region}" describe-images --filter "Name=name,Values=${NAME}" --query 'Images[*].[ImageId]' --output text)
# Make AMI usable outside of Sourcegraph.
aws ec2 --region="${region}" modify-image-attribute --image-id "${AWS_AMI_ID}" --launch-permission "Add=[{Group=all}]"

View File

@ -2,7 +2,8 @@
set -eo pipefail
for dbname in $(psql -Xc "copy (select datname from pg_database where datname like 'sourcegraph-test-%') to stdout"); do
DB_NAMES=$(psql -Xc "copy (select datname from pg_database where datname like 'sourcegraph-test-%') to stdout")
for dbname in $DB_NAMES; do
dropdb "$dbname"
echo "dropped $dbname"
done

View File

@ -63,7 +63,8 @@ TAG="${TAG:-${NEW_TAG:?"$MISSING_MESSAGE"}}"
NEW_TAG_AND_DIGEST="$(get_new_tag_and_digest "$NEW_IMAGE" "$TAG")"
for file in $(fd --glob Dockerfile\* .); do
DOCKERFILES=$(fd --glob Dockerfile\* .)
for file in $DOCKERFILES; do
update_image_reference "$OLD_IMAGE" "$NEW_IMAGE" "$NEW_TAG_AND_DIGEST" "$file"
echo "$file"
done