mirror of
https://github.com/VictoriaMetrics/VictoriaMetrics.git
synced 2024-12-22 16:36:27 +01:00
815e9bf892
This is a follow-up for f60c08a7bd
Changes:
- Make sure all the urls related to NewRelic protocol start from /newrelic . Previously some urls were started from /api/v1/newrelic
- Remove /api/v1 part from NewRelic urls, since it has no sense
- Remove automatic transformation from CamelCase to snake_case for NewRelic labels and metric names,
since it may complicate the transition from NewRelic to VictoriaMetrics. Preserve all the metric names and label names,
so users could query metrics and labels by the same names which are used in NewRelic.
The automatic transformation from CamelCase to snake_case can be added later as a special action for relabeling rules if needed.
- Properly update per-tenant data ingestion stats at app/vmagent/newrelic/request_handler.go . Previously it was always zero.
- Fix NewRelic urls in vmagent when multitenant data ingestion is enabled. Previously they were mistakenly started from `/`.
- Document NewRelic data ingestion url at docs/Cluster-VictoriaMetrics.md
- Remove superflouos memory allocations at lib/protoparser/newrelic
- Improve tests at lib/protoparser/newrelic/*
Updates https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3520
Updates https://github.com/VictoriaMetrics/VictoriaMetrics/pull/4712
230 lines
4.3 KiB
Go
230 lines
4.3 KiB
Go
package newrelic
|
|
|
|
import (
|
|
"fmt"
|
|
"reflect"
|
|
"strings"
|
|
"testing"
|
|
)
|
|
|
|
func TestRowsUnmarshalFailure(t *testing.T) {
|
|
f := func(data string) {
|
|
t.Helper()
|
|
|
|
var r Rows
|
|
if err := r.Unmarshal([]byte(data)); err == nil {
|
|
t.Fatalf("expecting non-nil error")
|
|
}
|
|
}
|
|
|
|
// Empty JSON
|
|
f("")
|
|
|
|
// Invalid JSON
|
|
f("123")
|
|
f("[foo]")
|
|
f(`{"foo":123}`)
|
|
}
|
|
|
|
func TestRowsUnmarshalSuccess(t *testing.T) {
|
|
f := func(data string, expectedRows []Row) {
|
|
t.Helper()
|
|
|
|
var r Rows
|
|
if err := r.Unmarshal([]byte(data)); err != nil {
|
|
t.Fatalf("unexpected error: %s", err)
|
|
}
|
|
if !reflect.DeepEqual(r.Rows, expectedRows) {
|
|
t.Fatalf("unexpected rows parsed\ngot\n%s\nwant\n%s", rowsToString(r.Rows), rowsToString(expectedRows))
|
|
}
|
|
}
|
|
|
|
// empty array
|
|
f(`[]`, nil)
|
|
|
|
// zero events
|
|
f(`[
|
|
{
|
|
"EntityID":28257883748326179,
|
|
"IsAgent":true,
|
|
"Events":[],
|
|
"ReportingAgentID":28257883748326179
|
|
}]`, nil)
|
|
|
|
// A single event
|
|
f(`[{
|
|
"EntityID":28257883748326179,
|
|
"IsAgent":true,
|
|
"Events":[
|
|
{
|
|
"eventType":"SystemSample",
|
|
"timestamp":1690286061,
|
|
"entityKey":"macbook-pro.local",
|
|
"dc": "1",
|
|
"diskWritesPerSecond":-34.21,
|
|
"uptime":762376
|
|
}
|
|
],
|
|
"ReportingAgentID":28257883748326179
|
|
}]`, []Row{
|
|
{
|
|
Tags: []Tag{
|
|
{
|
|
Key: []byte("eventType"),
|
|
Value: []byte("SystemSample"),
|
|
},
|
|
{
|
|
Key: []byte("entityKey"),
|
|
Value: []byte("macbook-pro.local"),
|
|
},
|
|
{
|
|
Key: []byte("dc"),
|
|
Value: []byte("1"),
|
|
},
|
|
},
|
|
Samples: []Sample{
|
|
{
|
|
Name: []byte("diskWritesPerSecond"),
|
|
Value: -34.21,
|
|
},
|
|
{
|
|
Name: []byte("uptime"),
|
|
Value: 762376,
|
|
},
|
|
},
|
|
Timestamp: 1690286061000,
|
|
},
|
|
})
|
|
|
|
// Multiple events
|
|
f(`[
|
|
{
|
|
"EntityID":28257883748326179,
|
|
"IsAgent":true,
|
|
"Events":[
|
|
{
|
|
"eventType":"SystemSample",
|
|
"timestamp":1690286061,
|
|
"entityKey":"macbook-pro.local",
|
|
"dc": "1",
|
|
"diskWritesPerSecond":-34.21,
|
|
"uptime":762376
|
|
}
|
|
],
|
|
"ReportingAgentID":28257883748326179
|
|
},
|
|
{
|
|
"EntityID":282579,
|
|
"IsAgent":true,
|
|
"Events":[
|
|
{
|
|
"eventType":"SystemSample",
|
|
"timestamp":1690286061,
|
|
"entityKey":"macbook-pro.local",
|
|
"diskWritesPerSecond":234.34,
|
|
"timestamp":1690286061.433,
|
|
"uptime":762376
|
|
},
|
|
{
|
|
"eventType":"ProcessSample",
|
|
"timestamp":1690286061987,
|
|
"uptime":1236
|
|
}
|
|
],
|
|
"ReportingAgentID":2879
|
|
}
|
|
]`, []Row{
|
|
{
|
|
Tags: []Tag{
|
|
{
|
|
Key: []byte("eventType"),
|
|
Value: []byte("SystemSample"),
|
|
},
|
|
{
|
|
Key: []byte("entityKey"),
|
|
Value: []byte("macbook-pro.local"),
|
|
},
|
|
{
|
|
Key: []byte("dc"),
|
|
Value: []byte("1"),
|
|
},
|
|
},
|
|
Samples: []Sample{
|
|
{
|
|
Name: []byte("diskWritesPerSecond"),
|
|
Value: -34.21,
|
|
},
|
|
{
|
|
Name: []byte("uptime"),
|
|
Value: 762376,
|
|
},
|
|
},
|
|
Timestamp: 1690286061000,
|
|
},
|
|
{
|
|
Tags: []Tag{
|
|
{
|
|
Key: []byte("eventType"),
|
|
Value: []byte("SystemSample"),
|
|
},
|
|
{
|
|
Key: []byte("entityKey"),
|
|
Value: []byte("macbook-pro.local"),
|
|
},
|
|
},
|
|
Samples: []Sample{
|
|
{
|
|
Name: []byte("diskWritesPerSecond"),
|
|
Value: 234.34,
|
|
},
|
|
{
|
|
Name: []byte("uptime"),
|
|
Value: 762376,
|
|
},
|
|
},
|
|
Timestamp: 1690286061433,
|
|
},
|
|
{
|
|
Tags: []Tag{
|
|
{
|
|
Key: []byte("eventType"),
|
|
Value: []byte("ProcessSample"),
|
|
},
|
|
},
|
|
Samples: []Sample{
|
|
{
|
|
Name: []byte("uptime"),
|
|
Value: 1236,
|
|
},
|
|
},
|
|
Timestamp: 1690286061987,
|
|
},
|
|
})
|
|
|
|
}
|
|
|
|
func rowsToString(rows []Row) string {
|
|
var a []string
|
|
for _, row := range rows {
|
|
s := row.String()
|
|
a = append(a, s)
|
|
}
|
|
return strings.Join(a, "\n")
|
|
}
|
|
|
|
func (r *Row) String() string {
|
|
var a []string
|
|
for _, t := range r.Tags {
|
|
s := fmt.Sprintf("%s=%q", t.Key, t.Value)
|
|
a = append(a, s)
|
|
}
|
|
tagsString := "{" + strings.Join(a, ",") + "}"
|
|
a = a[:0]
|
|
for _, sample := range r.Samples {
|
|
s := fmt.Sprintf("[%s %f]", sample.Name, sample.Value)
|
|
a = append(a, s)
|
|
}
|
|
samplesString := strings.Join(a, ",")
|
|
return fmt.Sprintf("tags=%s, samples=%s, timestamp=%d", tagsString, samplesString, r.Timestamp)
|
|
}
|