mirror of
https://github.com/kubernetes-sigs/external-dns.git
synced 2026-05-04 14:21:33 +02:00
* feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_total Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track ownership conflicts Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track ownership conflicts Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track ownership conflicts Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(metrics): added skipped_records_owner_mismatch_per_sync to track ownership conflicts Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> --------- Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
171 lines
5.3 KiB
Go
171 lines
5.3 KiB
Go
/*
|
|
Copyright 2025 The Kubernetes Authors.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package testutils
|
|
|
|
import (
|
|
"fmt"
|
|
"sort"
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/prometheus/client_golang/prometheus"
|
|
dto "github.com/prometheus/client_model/go"
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
// TestHelperVerifyMetricsGaugeVectorWithLabels verifies that a prometheus.GaugeVec metric with specific labels has the expected value.
|
|
// Supports partial label matching - if only some labels are provided, it sums all metrics matching those labels.
|
|
//
|
|
// Example usage:
|
|
//
|
|
// Exact match (all labels)
|
|
// labels := map[string]string{"method": "GET", "status": "200"}
|
|
// TestHelperVerifyMetricsGaugeVectorWithLabels(t, 42.0, myGaugeVec, labels)
|
|
//
|
|
// Partial match (sum all metrics with method=GET)
|
|
// labels := map[string]string{"method": "GET"}
|
|
// TestHelperVerifyMetricsGaugeVectorWithLabels(t, 100.0, myGaugeVec, labels)
|
|
func TestHelperVerifyMetricsGaugeVectorWithLabels(t *testing.T, expected float64, metric prometheus.GaugeVec, labels map[string]string) {
|
|
TestHelperVerifyMetricsGaugeVectorWithLabelsFunc(t, expected, assert.Equal, metric, labels)
|
|
}
|
|
|
|
// TestHelperVerifyMetricsGaugeVectorWithLabelsFunc is a helper function that verifies a prometheus.GaugeVec metric with specific labels using a custom assertion function.
|
|
// Supports partial label matching - if only some labels are provided, it sums all metrics matching those labels.
|
|
func TestHelperVerifyMetricsGaugeVectorWithLabelsFunc(t *testing.T, expected float64, aFunc assert.ComparisonAssertionFunc, metric prometheus.GaugeVec, labels map[string]string) {
|
|
t.Helper()
|
|
|
|
// Collect all metrics and find matching ones
|
|
actual := sumMetricsWithLabels(&metric, labels)
|
|
|
|
if !aFunc(t, expected, actual, "Expected gauge value does not match the actual value", labels) {
|
|
t.Logf("Available metrics:\n%s", collectGaugeVecMetrics(&metric))
|
|
}
|
|
}
|
|
|
|
// collectAll drains all current observations from a GaugeVec into a slice.
|
|
func collectAll(metric *prometheus.GaugeVec) []*dto.Metric {
|
|
ch := make(chan prometheus.Metric, 1024)
|
|
go func() {
|
|
metric.Collect(ch)
|
|
close(ch)
|
|
}()
|
|
var result []*dto.Metric
|
|
for m := range ch {
|
|
var dm dto.Metric
|
|
if err := m.Write(&dm); err != nil {
|
|
continue
|
|
}
|
|
result = append(result, &dm)
|
|
}
|
|
return result
|
|
}
|
|
|
|
// sumMetricsWithLabels sums all metric values that match the provided labels (partial match supported).
|
|
// Label matching is case-insensitive since metrics are stored in lowercase.
|
|
func sumMetricsWithLabels(metric *prometheus.GaugeVec, matchLabels map[string]string) float64 {
|
|
var sum float64
|
|
for _, dm := range collectAll(metric) {
|
|
// Check if all matchLabels are present with correct values (case-insensitive)
|
|
metricLabels := make(map[string]string)
|
|
for _, lp := range dm.Label {
|
|
metricLabels[lp.GetName()] = lp.GetValue()
|
|
}
|
|
|
|
matches := true
|
|
for k, v := range matchLabels {
|
|
if !strings.EqualFold(metricLabels[k], v) {
|
|
matches = false
|
|
break
|
|
}
|
|
}
|
|
|
|
if matches && dm.Gauge != nil {
|
|
sum += dm.Gauge.GetValue()
|
|
}
|
|
}
|
|
|
|
return sum
|
|
}
|
|
|
|
// collectGaugeVecMetrics collects all metrics from a GaugeVec and returns a formatted string.
|
|
// Shows both per-label aggregates and detailed metrics.
|
|
func collectGaugeVecMetrics(metric *prometheus.GaugeVec) string {
|
|
// Collect all metrics and aggregate by label
|
|
type metricEntry struct {
|
|
labels map[string]string
|
|
value float64
|
|
}
|
|
var entries []metricEntry
|
|
aggregates := make(map[string]map[string]float64) // labelName -> labelValue -> sum
|
|
|
|
for _, dm := range collectAll(metric) {
|
|
entry := metricEntry{labels: make(map[string]string)}
|
|
for _, lp := range dm.Label {
|
|
name, value := lp.GetName(), lp.GetValue()
|
|
entry.labels[name] = value
|
|
|
|
if aggregates[name] == nil {
|
|
aggregates[name] = make(map[string]float64)
|
|
}
|
|
if dm.Gauge != nil {
|
|
aggregates[name][value] += dm.Gauge.GetValue()
|
|
}
|
|
}
|
|
if dm.Gauge != nil {
|
|
entry.value = dm.Gauge.GetValue()
|
|
}
|
|
entries = append(entries, entry)
|
|
}
|
|
|
|
if len(entries) == 0 {
|
|
return " (no metrics collected)"
|
|
}
|
|
|
|
var sb strings.Builder
|
|
|
|
// Output aggregates by label (sorted)
|
|
sb.WriteString("Totals by label:\n")
|
|
var labelNames []string
|
|
for name := range aggregates {
|
|
labelNames = append(labelNames, name)
|
|
}
|
|
sort.Strings(labelNames)
|
|
|
|
for _, name := range labelNames {
|
|
values := aggregates[name]
|
|
var pairs []string
|
|
for v, sum := range values {
|
|
pairs = append(pairs, fmt.Sprintf("%s=%.0f", v, sum))
|
|
}
|
|
sort.Strings(pairs)
|
|
sb.WriteString(fmt.Sprintf(" %s: %s\n", name, strings.Join(pairs, ", ")))
|
|
}
|
|
|
|
// Output detailed metrics
|
|
sb.WriteString("\nAll metrics:\n")
|
|
for _, e := range entries {
|
|
var labels []string
|
|
for k, v := range e.labels {
|
|
labels = append(labels, fmt.Sprintf("%s=%q", k, v))
|
|
}
|
|
sort.Strings(labels)
|
|
sb.WriteString(fmt.Sprintf(" {%s} = %.2f\n", strings.Join(labels, ", "), e.value))
|
|
}
|
|
|
|
return sb.String()
|
|
}
|