Skip to content

Commit

Permalink
exporter: Rename ratelimit drops counter
Browse files Browse the repository at this point in the history
There are multiple places where events can be rate limited, and potentially
there could be multiple things being rate limited. Let's make it clear from the
metric name and description what and where is being rate limitted.

Signed-off-by: Anna Kapuscinska <anna@isovalent.com>
  • Loading branch information
lambdanis committed Sep 9, 2024
1 parent 5231093 commit 1848253
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion contrib/upgrade-notes/latest.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ Depending on your setup, changes listed here might require a manual intervention

### Metrics

* TBD
* `tetragon_ratelimit_dropped_total` metric is renamed to `tetragon_export_ratelimit_events_dropped_total`
8 changes: 4 additions & 4 deletions docs/content/en/docs/reference/metrics.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions pkg/exporter/metrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ var (

rateLimitDropped = prometheus.NewCounter(prometheus.CounterOpts{
Namespace: consts.MetricsNamespace,
Name: "ratelimit_dropped_total",
Help: "The total number of rate limit Tetragon drops",
Name: "export_ratelimit_events_dropped_total",
Help: "Number of events dropped on export due to rate limiting",
ConstLabels: nil,
})
)
Expand Down

0 comments on commit 1848253

Please sign in to comment.