From aca2a38ad6bf1cf7625d41d130b63ee276eb0a70 Mon Sep 17 00:00:00 2001 From: Paul Rogers <129207811+paul1r@users.noreply.github.com> Date: Wed, 27 Mar 2024 07:50:25 -0400 Subject: [PATCH] test: gcs object client test data race (#12324) --- pkg/storage/chunk/client/gcp/gcs_object_client.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pkg/storage/chunk/client/gcp/gcs_object_client.go b/pkg/storage/chunk/client/gcp/gcs_object_client.go index d4b35e48d9df..2e340c038e72 100644 --- a/pkg/storage/chunk/client/gcp/gcs_object_client.go +++ b/pkg/storage/chunk/client/gcp/gcs_object_client.go @@ -319,8 +319,7 @@ func gcsTransport(ctx context.Context, scope string, insecure bool, http2 bool, transportOptions := []option.ClientOption{option.WithScopes(scope)} if insecure { customTransport.TLSClientConfig = &tls.Config{InsecureSkipVerify: true} - // When using `insecure` (testing only), we add a fake API key as well to skip credential chain lookups. - transportOptions = append(transportOptions, option.WithAPIKey("insecure")) + transportOptions = append(transportOptions, option.WithoutAuthentication()) } if serviceAccount.String() != "" { transportOptions = append(transportOptions, option.WithCredentialsJSON([]byte(serviceAccount.String())))