Merge pull request #1461 from ydb-platform/fix-panic

* Fixed panic on usage metrics package from prometheus adapter on `tr…
ydb-platform · Sep 16, 2024 · e3045be · e3045be
2 parents 900393f + 1970ee8
commit e3045be
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,5 @@
+* Fixed panic on usage metrics package from prometheus adapter on `trace.Driver.OnNewStream` callback
+
 ## v3.80.3
 * Added option `ydb.WithSessionPoolSessionIdleTimeToLive` for restrict idle time of query sessions
 * Fixed bug with leak of query transactions

diff --git a/metrics/driver.go b/metrics/driver.go
@@ -65,6 +65,10 @@ func driver(config Config) (t trace.Driver) {
 			if config.Details()&trace.DriverConnStreamEvents != 0 {
 				requestStatuses.With(map[string]string{
 					"status":   errorBrief(info.Error),
+					"endpoint": endpoint,
+					"node_id":  strconv.FormatUint(uint64(nodeID), 10),
+				}).Inc()
+				requestMethods.With(map[string]string{
 					"method":   string(method),
 					"endpoint": endpoint,
 					"node_id":  strconv.FormatUint(uint64(nodeID), 10),