synced/coder
1
0
Fork 0
mirror of https://github.com/coder/coder.git synced 2025-07-13 21:36:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

chore: name unlabeled db transaction metrics (#15251)

Browse Source
This commit is contained in:
Steven Masley
2024-10-28 13:15:29 -04:00
committed by GitHub
parent ecb22461bb
commit 7cb20d7b26
2 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
coderd/database/dbmetrics/dbmetrics.go
View File

@ -74,6 +74,11 @@ func (m metricsStore) InTx(f func(database.Store) error, options *database.TxOpt
options = database.DefaultTXOptions()
}
if options.TxIdentifier == "" {
// empty strings are hard to deal with in grafana
options.TxIdentifier = "unlabeled"
}
start := time.Now()
err := m.Store.InTx(f, options)
dur := time.Since(start)
@ -82,13 +87,13 @@ func (m metricsStore) InTx(f func(database.Store) error, options *database.TxOpt
// So IDs should be used sparingly to prevent too much bloat.
m.txDuration.With(prometheus.Labels{
"success": strconv.FormatBool(err == nil),
"tx_id": options.TxIdentifier, // Can be empty string for unlabeled
"tx_id": options.TxIdentifier,
}).Observe(dur.Seconds())
m.txRetries.With(prometheus.Labels{
"success": strconv.FormatBool(err == nil),
"retries": strconv.FormatInt(int64(options.ExecutionCount()-1), 10),
"tx_id": options.TxIdentifier, // Can be empty string for unlabeled
"tx_id": options.TxIdentifier,
}).Inc()
// Log all serializable transactions that are retried.

2
coderd/database/dbmetrics/dbmetrics_test.go
View File

@ -22,7 +22,7 @@ func TestInTxMetrics(t *testing.T) {
successLabels := prometheus.Labels{
"success": "true",
"tx_id": "",
"tx_id": "unlabeled",
}
const inTxHistMetricName = "coderd_db_tx_duration_seconds"
const inTxCountMetricName = "coderd_db_tx_executions_count"