Fix GCS Auth Issues (#554)

* list instead of attrs Signed-off-by: Joe Elliott <number101010@gmail.com> * Added scope read/write Signed-off-by: Joe Elliott <number101010@gmail.com> * Revert "list instead of attrs" This reverts commit 73a260c. * changelog Signed-off-by: Joe Elliott <number101010@gmail.com>
grafana · Feb 24, 2021 · 618654d · 618654d
1 parent 3129aae
commit 618654d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 ## master / unreleased
 
 * [ENHANCEMENT] Add a Shutdown handler to flush data to backend, at "/shutdown". [#526](https://github.com/grafana/tempo/pull/526)
+* [BUGFIX] Fixes permissions errors on startup in GCS. [#554](https://github.com/grafana/tempo/pull/554)
 
 ## v0.6.0
 

diff --git a/tempodb/backend/gcs/gcs.go b/tempodb/backend/gcs/gcs.go
@@ -34,8 +34,9 @@ func New(cfg *Config) (backend.Reader, backend.Writer, backend.Compactor, error)
 	ctx := context.Background()
 
 	customTransport := http.DefaultTransport.(*http.Transport).Clone()
-
-	transportOptions := []option.ClientOption{}
+	transportOptions := []option.ClientOption{
+		option.WithScopes(storage.ScopeReadWrite),
+	}
 	if cfg.Insecure {
 		transportOptions = append(transportOptions, option.WithoutAuthentication())
 		customTransport.TLSClientConfig = &tls.Config{InsecureSkipVerify: true}