Skip to content

Commit

Permalink
fix: remove uuid from metrics
Browse files Browse the repository at this point in the history
  • Loading branch information
vsukhin committed Feb 15, 2024
1 parent c997b88 commit e660188
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions internal/app/api/metrics/metrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,12 @@ import (
var testExecutionCount = promauto.NewCounterVec(prometheus.CounterOpts{
Name: "testkube_test_executions_count",
Help: "The total number of test executions",
}, []string{"type", "name", "result", "labels", "test_uri", "execution_uri"})
}, []string{"type", "name", "result", "labels", "test_uri"})

var testSuiteExecutionCount = promauto.NewCounterVec(prometheus.CounterOpts{
Name: "testkube_testsuite_executions_count",
Help: "The total number of test suite executions",
}, []string{"name", "result", "labels", "testsuite_uri", "execution_uri"})
}, []string{"name", "result", "labels", "testsuite_uri"})

var testCreationCount = promauto.NewCounterVec(prometheus.CounterOpts{
Name: "testkube_test_creations_count",
Expand Down Expand Up @@ -121,8 +121,6 @@ func (m Metrics) IncExecuteTest(execution testkube.Execution, dashboardURI strin
"result": status,
"labels": strings.Join(labels, ","),
"test_uri": fmt.Sprintf("%s/tests/%s", dashboardURI, execution.TestName),
"execution_uri": fmt.Sprintf("%s/tests/%s/executions/%s", dashboardURI,
execution.TestName, execution.Id),
}).Inc()
}

Expand Down Expand Up @@ -153,8 +151,6 @@ func (m Metrics) IncExecuteTestSuite(execution testkube.TestSuiteExecution, dash
"result": status,
"labels": strings.Join(labels, ","),
"testsuite_uri": fmt.Sprintf("%s/test-suites/%s", dashboardURI, testSuiteName),
"execution_uri": fmt.Sprintf("%s/test-suites/%s/executions/%s", dashboardURI,
testSuiteName, execution.Id),
}).Inc()
}

Expand Down

0 comments on commit e660188

Please sign in to comment.