diff --git a/cmd/agent/app/processors/thrift_processor_test.go b/cmd/agent/app/processors/thrift_processor_test.go index 9a524fa4b79..dc492ddd251 100644 --- a/cmd/agent/app/processors/thrift_processor_test.go +++ b/cmd/agent/app/processors/thrift_processor_test.go @@ -16,7 +16,6 @@ package processors import ( "errors" - "fmt" "testing" "time" @@ -196,7 +195,7 @@ func assertProcessorCorrectness( metricsFactory *metrics.LocalFactory, sizeF func() int, nameF func() string, - counterType string, + format string, ) { // wait for server to receive for i := 0; i < 1000; i++ { @@ -219,11 +218,9 @@ func assertProcessorCorrectness( } // agentReporter must emit metrics - batchesSubmittedCounter := fmt.Sprintf("tc-reporter.%s.batches.submitted", counterType) - spansSubmittedCounter := fmt.Sprintf("tc-reporter.%s.spans.submitted", counterType) mTestutils.AssertCounterMetrics(t, metricsFactory, []mTestutils.ExpectedMetric{ - {Name: batchesSubmittedCounter, Value: 1}, - {Name: spansSubmittedCounter, Value: 1}, + {Name: "tc-reporter.batches.submitted", Tags: map[string]string{"format": format}, Value: 1}, + {Name: "tc-reporter.spans.submitted", Tags: map[string]string{"format": format}, Value: 1}, {Name: "thrift.udp.server.packets.processed", Value: 1}, }...) } diff --git a/cmd/agent/app/reporter/tchannel/reporter.go b/cmd/agent/app/reporter/tchannel/reporter.go index 1ed9647ead3..b201f85273b 100644 --- a/cmd/agent/app/reporter/tchannel/reporter.go +++ b/cmd/agent/app/reporter/tchannel/reporter.go @@ -34,19 +34,19 @@ const ( type batchMetrics struct { // Number of successful batch submissions to collector - BatchesSubmitted metrics.Counter `metric:"batches.submitted"` + BatchesSubmitted metrics.Counter `metric:"tc-reporter.batches.submitted"` // Number of failed batch submissions to collector - BatchesFailures metrics.Counter `metric:"batches.failures"` + BatchesFailures metrics.Counter `metric:"tc-reporter.batches.failures"` // Number of spans in a batch submitted to collector - BatchSize metrics.Gauge `metric:"batch_size"` + BatchSize metrics.Gauge `metric:"tc-reporter.batch_size"` // Number of successful span submissions to collector - SpansSubmitted metrics.Counter `metric:"spans.submitted"` + SpansSubmitted metrics.Counter `metric:"tc-reporter.spans.submitted"` // Number of failed span submissions to collector - SpansFailures metrics.Counter `metric:"spans.failures"` + SpansFailures metrics.Counter `metric:"tc-reporter.spans.failures"` } // Reporter forwards received spans to central collector tier over TChannel. @@ -71,11 +71,9 @@ func New( zClient := zipkincore.NewTChanZipkinCollectorClient(thriftClient) jClient := jaeger.NewTChanCollectorClient(thriftClient) batchesMetrics := map[string]batchMetrics{} - tcReporterNS := mFactory.Namespace("tc-reporter", nil) for _, s := range []string{zipkinBatches, jaegerBatches} { - nsByType := tcReporterNS.Namespace(s, nil) bm := batchMetrics{} - metrics.Init(&bm, nsByType, nil) + metrics.Init(&bm, mFactory.Namespace("", map[string]string{"format": s}), nil) batchesMetrics[s] = bm } return &Reporter{ diff --git a/cmd/agent/app/reporter/tchannel/reporter_test.go b/cmd/agent/app/reporter/tchannel/reporter_test.go index 4445fc41a11..e5676373f10 100644 --- a/cmd/agent/app/reporter/tchannel/reporter_test.go +++ b/cmd/agent/app/reporter/tchannel/reporter_test.go @@ -15,7 +15,6 @@ package tchannel import ( - "fmt" "testing" "time" @@ -105,16 +104,11 @@ func submitTestJaegerBatch(reporter *Reporter) error { return reporter.EmitBatch(batch) } -func checkCounters(t *testing.T, mf *metrics.LocalFactory, batchesSubmitted, spansSubmitted, batchesFailures, spansFailures int, prefix string) { - batchesCounter := fmt.Sprintf("tc-reporter.%s.batches.submitted", prefix) - batchesFailureCounter := fmt.Sprintf("tc-reporter.%s.batches.failures", prefix) - spansCounter := fmt.Sprintf("tc-reporter.%s.spans.submitted", prefix) - spansFailureCounter := fmt.Sprintf("tc-reporter.%s.spans.failures", prefix) - +func checkCounters(t *testing.T, mf *metrics.LocalFactory, batchesSubmitted, spansSubmitted, batchesFailures, spansFailures int, format string) { mTestutils.AssertCounterMetrics(t, mf, []mTestutils.ExpectedMetric{ - {Name: batchesCounter, Value: batchesSubmitted}, - {Name: spansCounter, Value: spansSubmitted}, - {Name: batchesFailureCounter, Value: batchesFailures}, - {Name: spansFailureCounter, Value: spansFailures}, + {Name: "tc-reporter.batches.submitted", Tags: map[string]string{"format": format}, Value: batchesSubmitted}, + {Name: "tc-reporter.spans.submitted", Tags: map[string]string{"format": format}, Value: spansSubmitted}, + {Name: "tc-reporter.batches.failures", Tags: map[string]string{"format": format}, Value: batchesFailures}, + {Name: "tc-reporter.spans.failures", Tags: map[string]string{"format": format}, Value: spansFailures}, }...) }