validate configs

Signed-off-by: Thor <[email protected]>

Author: Thor

pkg/chunk/storage/factory.go

@@ -61,7 +61,6 @@ func (cfg *Config) RegisterFlags(f *flag.FlagSet) {
 }
 
 // Validate config and returns error on failure
-// TODO need to rebase to call it, because Config validation has been added just recently
 func (cfg *Config) Validate() error {
 	if cfg.Engine != StorageEngineChunks && cfg.Engine != StorageEngineTSDB {
 		return errors.New("unsupported storage engine")

pkg/cortex/cortex.go

@@ -122,6 +122,14 @@ func (c *Config) Validate() error {
 	if err := c.Encoding.Validate(); err != nil {
 		return errors.Wrap(err, "invalid encoding config")
 	}
+
+	if err := c.Storage.Validate(); err != nil {
+		return errors.Wrap(err, "invalid storage config")
+	}
+
+	if err := c.TSDB.Validate(); err != nil {
+		return errors.Wrap(err, "invalid TSDB config")
+	}
 	return nil
 }