Skip to content

Commit

Permalink
Fix ensure-e2e-infra-state-bucket
Browse files Browse the repository at this point in the history
ensure-e2e-infra-state-bucket is currently using `id` but this no
longer seems to work. The field `name` seems to work, so replace it.

Also make the message brighter so it's obvious why we're
recreating.
  • Loading branch information
zmerlynn committed Mar 19, 2024
1 parent c95f591 commit 1f47442
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build/includes/google-cloud.mk
Original file line number Diff line number Diff line change
Expand Up @@ -58,9 +58,9 @@ gcloud-e2e-infra-state-bucket:

ensure-e2e-infra-state-bucket: GCP_PROJECT ?= $(shell $(current_project))
ensure-e2e-infra-state-bucket:
@buckets=$$(docker run --rm $(common_mounts) $(build_tag) gcloud storage buckets describe gs://$(GCP_PROJECT)-e2e-infra-bucket-tfstate --format="value(id)");\
@buckets=$$(docker run --rm $(common_mounts) $(build_tag) gcloud storage buckets describe gs://$(GCP_PROJECT)-e2e-infra-bucket-tfstate --format="value(name)");\
if [ -z $${buckets} ]; then\
echo "E2E infra state bucket $(GCP_PROJECT)-e2e-infra-bucket-tfstate does not exist, creating...";\
echo "** E2E infra state bucket $(GCP_PROJECT)-e2e-infra-bucket-tfstate does not exist (or gcloud failed), creating...";\
$(MAKE) gcloud-e2e-infra-state-bucket;\
fi

Expand Down

0 comments on commit 1f47442

Please sign in to comment.