Merge pull request #1392 from gouthamve/expose-override-file-config

tomwilkie · web-flow · commit 826f0045472f · 2019-05-16T14:57:52.000+01:00
Expose the limits override config fields to YAML
diff --git a/pkg/util/validation/limits.go b/pkg/util/validation/limits.go
@@ -32,8 +32,8 @@ type Limits struct {
 	CardinalityLimit    int           `yaml:"cardinality_limit"`
 
 	// Config for overrides, convenient if it goes here.
-	PerTenantOverrideConfig string
-	PerTenantOverridePeriod time.Duration
+	PerTenantOverrideConfig string        `yaml:"per_tenant_override_config"`
+	PerTenantOverridePeriod time.Duration `yaml:"per_tenant_override_period"`
 }
 
 // RegisterFlags adds the flags required to config this to the given FlagSet

Original file line number	Diff line number	Diff line change
`@@ -32,8 +32,8 @@ type Limits struct {`
`32`	`32`	CardinalityLimit int `yaml:"cardinality_limit"`
`33`	`33`
`34`	`34`	`// Config for overrides, convenient if it goes here.`
`35`		`- PerTenantOverrideConfig string`
`36`		`- PerTenantOverridePeriod time.Duration`
	`35`	+ PerTenantOverrideConfig string `yaml:"per_tenant_override_config"`
	`36`	+ PerTenantOverridePeriod time.Duration `yaml:"per_tenant_override_period"`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`// RegisterFlags adds the flags required to config this to the given FlagSet`