From f9d57a8d5db3e58f79a1b1958d80c049c63d6cde Mon Sep 17 00:00:00 2001 From: Adhityaa Chandrasekar Date: Wed, 4 Nov 2020 22:19:52 +0000 Subject: [PATCH] APF: use snake_case in metric labels Signed-off-by: Adhityaa Chandrasekar --- .../util/flowcontrol/fairqueuing/queueset/queueset_test.go | 6 +++--- .../apiserver/pkg/util/flowcontrol/metrics/metrics.go | 4 ++-- .../src/k8s.io/component-base/metrics/testutil/promlint.go | 5 ----- test/integration/apiserver/flowcontrol/concurrency_test.go | 2 +- 4 files changed, 6 insertions(+), 11 deletions(-) diff --git a/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/fairqueuing/queueset/queueset_test.go b/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/fairqueuing/queueset/queueset_test.go index 7b26569924a..5ba0ddb81e7 100644 --- a/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/fairqueuing/queueset/queueset_test.go +++ b/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/fairqueuing/queueset/queueset_test.go @@ -167,7 +167,7 @@ func (uss *uniformScenarioState) exercise() { for i, uc := range uss.clients { uss.integrators[i] = fq.NewIntegrator(uss.clk) fsName := fmt.Sprintf("client%d", i) - uss.expectedInqueue = uss.expectedInqueue + fmt.Sprintf(` apiserver_flowcontrol_current_inqueue_requests{flowSchema=%q,priorityLevel=%q} 0%s`, fsName, uss.name, "\n") + uss.expectedInqueue = uss.expectedInqueue + fmt.Sprintf(` apiserver_flowcontrol_current_inqueue_requests{flow_schema=%q,priority_level=%q} 0%s`, fsName, uss.name, "\n") for j := 0; j < uc.nThreads; j++ { ust := uniformScenarioThread{ uss: uss, @@ -309,10 +309,10 @@ func (uss *uniformScenarioState) finalReview() { for i := range uss.clients { fsName := fmt.Sprintf("client%d", i) if atomic.AddInt32(&uss.executions[i], 0) > 0 { - uss.expectedExecuting = uss.expectedExecuting + fmt.Sprintf(` apiserver_flowcontrol_current_executing_requests{flowSchema=%q,priorityLevel=%q} 0%s`, fsName, uss.name, "\n") + uss.expectedExecuting = uss.expectedExecuting + fmt.Sprintf(` apiserver_flowcontrol_current_executing_requests{flow_schema=%q,priority_level=%q} 0%s`, fsName, uss.name, "\n") } if atomic.AddInt32(&uss.rejects[i], 0) > 0 { - expectedRejects = expectedRejects + fmt.Sprintf(` apiserver_flowcontrol_rejected_requests_total{flowSchema=%q,priorityLevel=%q,reason=%q} %d%s`, fsName, uss.name, uss.rejectReason, uss.rejects[i], "\n") + expectedRejects = expectedRejects + fmt.Sprintf(` apiserver_flowcontrol_rejected_requests_total{flow_schema=%q,priority_level=%q,reason=%q} %d%s`, fsName, uss.name, uss.rejectReason, uss.rejects[i], "\n") } } if uss.evalExecutingMetrics && len(uss.expectedExecuting) > 0 { diff --git a/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/metrics/metrics.go b/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/metrics/metrics.go index 940dac48104..fc286fbc032 100644 --- a/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/metrics/metrics.go +++ b/staging/src/k8s.io/apiserver/pkg/util/flowcontrol/metrics/metrics.go @@ -34,8 +34,8 @@ const ( const ( requestKind = "request_kind" - priorityLevel = "priorityLevel" - flowSchema = "flowSchema" + priorityLevel = "priority_level" + flowSchema = "flow_schema" phase = "phase" mark = "mark" ) diff --git a/staging/src/k8s.io/component-base/metrics/testutil/promlint.go b/staging/src/k8s.io/component-base/metrics/testutil/promlint.go index 4e270c4089a..33b83f05c5a 100644 --- a/staging/src/k8s.io/component-base/metrics/testutil/promlint.go +++ b/staging/src/k8s.io/component-base/metrics/testutil/promlint.go @@ -33,11 +33,6 @@ var exceptionMetrics = []string{ // k8s.io/kubernetes/vendor/k8s.io/apiserver/pkg/server/egressselector "apiserver_egress_dialer_dial_failure_count", // counter metrics should have "_total" suffix - // k8s.io/apiserver/pkg/util/flowcontrol/fairqueuing/queueset - "apiserver_flowcontrol_current_inqueue_requests", // label names should be written in 'snake_case' not 'camelCase', - "apiserver_flowcontrol_current_executing_requests", // label names should be written in 'snake_case' not 'camelCase' - "apiserver_flowcontrol_rejected_requests_total", // label names should be written in 'snake_case' not 'camelCase' - // k8s.io/kubernetes/vendor/k8s.io/apiserver/pkg/server/healthz "apiserver_request_total", // label names should be written in 'snake_case' not 'camelCase' diff --git a/test/integration/apiserver/flowcontrol/concurrency_test.go b/test/integration/apiserver/flowcontrol/concurrency_test.go index 0e1d440ea3e..8ba921cb932 100644 --- a/test/integration/apiserver/flowcontrol/concurrency_test.go +++ b/test/integration/apiserver/flowcontrol/concurrency_test.go @@ -45,7 +45,7 @@ const ( sharedConcurrencyMetricsName = "apiserver_flowcontrol_request_concurrency_limit" dispatchedRequestCountMetricsName = "apiserver_flowcontrol_dispatched_requests_total" rejectedRequestCountMetricsName = "apiserver_flowcontrol_rejected_requests_total" - labelPriorityLevel = "priorityLevel" + labelPriorityLevel = "priority_level" timeout = time.Second * 10 )