mirror of
https://github.com/kyverno/kyverno.git
synced 2025-03-31 03:45:17 +00:00
fix: http request metrics (#7197)
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
This commit is contained in:
parent
a3f3636bec
commit
03df0f83a6
2 changed files with 3 additions and 4 deletions
|
@ -12,7 +12,7 @@ import (
|
|||
)
|
||||
|
||||
func (inner AdmissionHandler) WithAdmission(logger logr.Logger) HttpHandler {
|
||||
return inner.withAdmission(logger).WithTrace("ADMISSION")
|
||||
return inner.withAdmission(logger).WithMetrics(logger).WithTrace("ADMISSION")
|
||||
}
|
||||
|
||||
func (inner AdmissionHandler) withAdmission(logger logr.Logger) HttpHandler {
|
||||
|
|
|
@ -69,11 +69,11 @@ func (inner HttpHandler) WithMetrics(logger logr.Logger, attrs ...attribute.KeyV
|
|||
func (inner HttpHandler) withMetrics(logger logr.Logger, attrs ...attribute.KeyValue) HttpHandler {
|
||||
meter := global.MeterProvider().Meter(metrics.MeterName)
|
||||
requestsMetric, err := meter.Int64Counter(
|
||||
"kyverno_http_requests_total",
|
||||
"kyverno_http_requests",
|
||||
metric.WithDescription("can be used to track the number of http requests"),
|
||||
)
|
||||
if err != nil {
|
||||
logger.Error(err, "Failed to create instrument, kyverno_http_requests_total")
|
||||
logger.Error(err, "Failed to create instrument, kyverno_http_requests")
|
||||
}
|
||||
durationMetric, err := meter.Float64Histogram(
|
||||
"kyverno_http_requests_duration_seconds",
|
||||
|
@ -85,7 +85,6 @@ func (inner HttpHandler) withMetrics(logger logr.Logger, attrs ...attribute.KeyV
|
|||
return func(writer http.ResponseWriter, request *http.Request) {
|
||||
startTime := time.Now()
|
||||
attributes := []attribute.KeyValue{
|
||||
semconv.HTTPRequestContentLengthKey.Int64(request.ContentLength),
|
||||
// semconv.HTTPHostKey.String(request.Host),
|
||||
semconv.HTTPMethodKey.String(request.Method),
|
||||
semconv.HTTPURLKey.String(request.RequestURI),
|
||||
|
|
Loading…
Add table
Reference in a new issue