Skip to content

Commit

Permalink
Merge pull request #1446 from miguelvalerio/fix-traefik-request-duration
Browse files Browse the repository at this point in the history
Fix Traefik request-duration metric
  • Loading branch information
aryan9600 authored Aug 28, 2023
2 parents e1c4257 + 7a0e95b commit d021e25
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pkg/metrics/observers/traefik.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ var traefikQueries = map[string]string{
}[{{ interval }}]
)
) by (le)
)`,
) * 1000`,
}

type TraefikObserver struct {
Expand Down
2 changes: 1 addition & 1 deletion pkg/metrics/observers/traefik_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ func TestTraefikObserver_GetRequestSuccessRate(t *testing.T) {
}

func TestTraefikObserver_GetRequestDuration(t *testing.T) {
expected := ` histogram_quantile( 0.99, sum( rate( traefik_service_request_duration_seconds_bucket{ service=~"default-podinfo-canary-[0-9a-zA-Z-]+@kubernetescrd" }[1m] ) ) by (le) )`
expected := ` histogram_quantile( 0.99, sum( rate( traefik_service_request_duration_seconds_bucket{ service=~"default-podinfo-canary-[0-9a-zA-Z-]+@kubernetescrd" }[1m] ) ) by (le) ) * 1000`

ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
promql := r.URL.Query()["query"][0]
Expand Down
2 changes: 1 addition & 1 deletion test/traefik/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

set -o errexit

TRAEFIK_CHART_VERSION="10.20.1" # traefik 2.7.0
TRAEFIK_CHART_VERSION="24.0.0" # traefik 2.10.4
REPO_ROOT=$(git rev-parse --show-toplevel)

mkdir -p ${REPO_ROOT}/bin
Expand Down

0 comments on commit d021e25

Please sign in to comment.