policyfiltermetrics: Rename metrics

Rename `tetragon_policyfilter_metrics_total` metric to `tetragon_policyfilter_operations_total`, and its `op` label to `operation` (to prevent confusion with ops.OpCode). Signed-off-by: Anna Kapuscinska <[email protected]>
cilium · Aug 9, 2024 · 9a0024a · 9a0024a
1 parent 6feacec
commit 9a0024a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/contrib/upgrade-notes/latest.md b/contrib/upgrade-notes/latest.md
@@ -44,4 +44,5 @@ tetragon:
 
 #### Metrics
 
-* TBD
+* `tetragon_policyfilter_metrics_total` metric is renamed to `tetragon_policyfilter_operations_total`, and its `op`
+  label is renamed to `operation`.
diff --git a/docs/content/en/docs/reference/metrics.md b/docs/content/en/docs/reference/metrics.md
diff --git a/pkg/metrics/policyfiltermetrics/policyfiltermetrics.go b/pkg/metrics/policyfiltermetrics/policyfiltermetrics.go
@@ -72,7 +72,7 @@ var (
 	}
 
 	operationLabel = metrics.ConstrainedLabel{
-		Name:   "op",
+		Name:   "operation",
 		Values: maps.Values(operationLabelValues),
 	}
 
@@ -84,8 +84,8 @@ var (
 
 var (
 	PolicyFilterOpMetrics = metrics.MustNewCounter(metrics.NewOpts(
-		consts.MetricsNamespace, "", "policyfilter_metrics_total",
-		"Policy filter metrics. For internal use only.",
+		consts.MetricsNamespace, "", "policyfilter_operations_total",
+		"Number of policy filter operations.",
 		nil, []metrics.ConstrainedLabel{subsysLabel, operationLabel, errorLabel}, nil,
 	), nil)