Ensure -dynamodb.url is set when calling NewTableClient() in aws mode (…

…grafana#1492) Signed-off-by: Marco Pracucci <[email protected]>
cyriltovena · Jul 9, 2019 · 7565090 · 7565090
1 parent 45b0b75
commit 7565090
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/storage/factory.go b/storage/factory.go
@@ -157,6 +157,9 @@ func NewTableClient(name string, cfg Config) (chunk.TableClient, error) {
 	case "inmemory":
 		return chunk.NewMockStorage(), nil
 	case "aws", "aws-dynamo":
+		if cfg.AWSStorageConfig.DynamoDB.URL == nil {
+			return nil, fmt.Errorf("Must set -dynamodb.url in aws mode")
+		}
 		path := strings.TrimPrefix(cfg.AWSStorageConfig.DynamoDB.URL.Path, "/")
 		if len(path) > 0 {
 			level.Warn(util.Logger).Log("msg", "ignoring DynamoDB URL path", "path", path)