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

Signed-off-by: Marco Pracucci <marco@pracucci.com>
cortexproject · Jul 7, 2019 · a02dfe4 · a02dfe4
1 parent 561033f
commit a02dfe4
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/pkg/chunk/storage/factory.go b/pkg/chunk/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)