Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Commit

Permalink
Move metric query metrics to another namespace to avoid conflict with…
Browse files Browse the repository at this point in the history
… spanstore metrics
  • Loading branch information
zjanc committed Mar 6, 2023
1 parent 03aef86 commit 7120d84
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cmd/query/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ func main() {
logger := svc.Logger // shortcut
baseFactory := svc.MetricsFactory.Namespace(metrics.NSOptions{Name: "jaeger"})
metricsFactory := baseFactory.Namespace(metrics.NSOptions{Name: "query"})
metricsQueryFactory := baseFactory.Namespace(metrics.NSOptions{Name: "metrics_query"})
version.NewInfoMetrics(metricsFactory)

traceCfg := &jaegerClientConfig.Configuration{
Expand Down Expand Up @@ -116,7 +117,7 @@ func main() {
logger.Fatal("Failed to create dependency reader", zap.Error(err))
}

metricsQueryService, err := createMetricsQueryService(metricsReaderFactory, v, logger, metricsFactory)
metricsQueryService, err := createMetricsQueryService(metricsReaderFactory, v, logger, metricsQueryFactory)
if err != nil {
logger.Fatal("Failed to create metrics query service", zap.Error(err))
}
Expand Down

0 comments on commit 7120d84

Please sign in to comment.