Skip to content

Commit 5ddc12b

Browse files
committed
Panic on eivctor creation failures
Signed-off-by: Essam Eldaly <eeldaly@amazon.com>
1 parent 9bb0269 commit 5ddc12b

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

pkg/querier/querier.go

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -265,16 +265,16 @@ func New(cfg Config, limits *validation.Overrides, distributor Distributor, stor
265265
if evictionCfg.Enabled() && resourceMonitor != nil {
266266
metricFunc, err := queryeviction.ResolveMetricFunc(evictionCfg.EvictionMetric)
267267
if err != nil {
268-
level.Error(logger).Log("msg", "invalid eviction metric", "err", err)
269-
} else {
270-
queryRegistry = queryeviction.NewQueryRegistry(metricFunc)
271-
queryEvictor, err = queryeviction.NewQueryEvictor(
272-
resourceMonitor, queryRegistry, evictionCfg,
273-
logger, reg, "querier",
274-
)
275-
if err != nil {
276-
level.Error(logger).Log("msg", "failed to create query evictor", "err", err)
277-
}
268+
panic(fmt.Sprintf("invalid eviction metric %q: %v", evictionCfg.EvictionMetric, err))
269+
}
270+
271+
queryRegistry = queryeviction.NewQueryRegistry(metricFunc)
272+
queryEvictor, err = queryeviction.NewQueryEvictor(
273+
resourceMonitor, queryRegistry, evictionCfg,
274+
logger, reg, "querier",
275+
)
276+
if err != nil {
277+
panic(fmt.Sprintf("failed to create query evictor: %v", err))
278278
}
279279
}
280280

0 commit comments

Comments
 (0)