Skip to content

Commit 25229a2

Browse files
committed
Rename metric ready_backend_connections_total to ready_backends
1 parent 565867c commit 25229a2

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

pkg/server/metrics/metrics.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,16 +112,16 @@ func newServerMetrics() *ServerMetrics {
112112
Namespace: Namespace,
113113
Subsystem: Subsystem,
114114
Name: "ready_backend_connections",
115-
Help: "Number of konnectivity agent connected to the proxy server. DEPRECATED, please use ready_backend_connections_total",
115+
Help: "Number of konnectivity agent connected to the proxy server. DEPRECATED, please use ready_backends",
116116
},
117117
[]string{},
118118
)
119119
totalBackendCount := prometheus.NewGaugeVec(
120120
prometheus.GaugeOpts{
121121
Namespace: Namespace,
122122
Subsystem: Subsystem,
123-
Name: "ready_backend_connections_total",
124-
Help: "Total number of konnectivity agent connected to the proxy server",
123+
Name: "ready_backends",
124+
Help: "Number of konnectivity agent connected to the proxy server",
125125
},
126126
[]string{
127127
"proxy_strategy",

pkg/server/server_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -960,7 +960,7 @@ func assertReadyBackendsMetric(t testing.TB, expect int) {
960960
func assertTotalReadyBackendsMetric(t testing.TB, expect map[string]int) {
961961
t.Helper()
962962
if err := metricstest.DefaultTester.ExpectServerTotalReadyBackends(expect); err != nil {
963-
t.Errorf("Expected %s metric for each proxy strategy %+v, but got error: %v", "ready_backend_connections_total", expect, err)
963+
t.Errorf("Expected %s metric for each proxy strategy %+v, but got error: %v", "ready_backends", expect, err)
964964
}
965965
}
966966

pkg/testing/metrics/metrics.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,14 @@ const (
3838
serverPendingDialsSample = `konnectivity_network_proxy_server_pending_backend_dials{} %d`
3939

4040
serverReadyBackendsHeader = `
41-
# HELP konnectivity_network_proxy_server_ready_backend_connections Number of konnectivity agent connected to the proxy server. DEPRECATED, please use ready_backend_connections_total
41+
# HELP konnectivity_network_proxy_server_ready_backend_connections Number of konnectivity agent connected to the proxy server. DEPRECATED, please use ready_backends
4242
# TYPE konnectivity_network_proxy_server_ready_backend_connections gauge`
4343
serverReadyBackendsSample = `konnectivity_network_proxy_server_ready_backend_connections{} %d`
4444

4545
serverTotalReadyBackendsHeader = `
46-
# HELP konnectivity_network_proxy_server_ready_backend_connections_total Total number of konnectivity agent connected to the proxy server
47-
# TYPE konnectivity_network_proxy_server_ready_backend_connections_total gauge`
48-
serverTotalReadyBackendsSample = `konnectivity_network_proxy_server_ready_backend_connections_total{proxy_strategy="%s"} %d`
46+
# HELP konnectivity_network_proxy_server_ready_backends Number of konnectivity agent connected to the proxy server
47+
# TYPE konnectivity_network_proxy_server_ready_backends gauge`
48+
serverTotalReadyBackendsSample = `konnectivity_network_proxy_server_ready_backends{proxy_strategy="%s"} %d`
4949

5050
serverEstablishedConnsHeader = `
5151
# HELP konnectivity_network_proxy_server_established_connections Current number of established end-to-end connections (post-dial).
@@ -116,7 +116,7 @@ func (t *Tester) ExpectServerTotalReadyBackends(expected map[string]int) error {
116116
for proxyStrategy, numOfBackends := range expected {
117117
expect += fmt.Sprintf(serverTotalReadyBackendsSample+"\n", proxyStrategy, numOfBackends)
118118
}
119-
return t.ExpectMetric(server.Namespace, server.Subsystem, "ready_backend_connections_total", expect)
119+
return t.ExpectMetric(server.Namespace, server.Subsystem, "ready_backends", expect)
120120
}
121121

122122
func (t *Tester) ExpectServerEstablishedConns(v int) error {

0 commit comments

Comments
 (0)