Skip to content

Commit 4706d5a

Browse files
alexey-gavrilov-flantdkostyrev
authored andcommitted
Metrics: Fix namespace in nginx_ingress_controller_ssl_expire_time_seconds. (kubernetes#10274)
1 parent a7dc65c commit 4706d5a

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

internal/ingress/metric/collectors/controller.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -305,6 +305,7 @@ func (cm *Controller) SetSSLExpireTime(servers []*ingress.Server) {
305305
}
306306
labels["host"] = s.Hostname
307307
labels["secret_name"] = s.SSLCert.Name
308+
labels["namespace"] = s.SSLCert.Namespace
308309
labels["identifier"] = s.SSLCert.Identifier()
309310

310311
cm.sslExpireTime.With(labels).Set(float64(s.SSLCert.ExpireTime.Unix()))

internal/ingress/metric/collectors/controller_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,8 @@ func TestControllerCounters(t *testing.T) {
8888
Hostname: "demo",
8989
SSLCert: &ingress.SSLCert{
9090
ExpireTime: t1,
91+
Name: "secret-name",
92+
Namespace: "secret-namespace",
9193
Certificate: &x509.Certificate{
9294
PublicKeyAlgorithm: x509.ECDSA,
9395
Issuer: pkix.Name{
@@ -111,7 +113,7 @@ func TestControllerCounters(t *testing.T) {
111113
want: `
112114
# HELP nginx_ingress_controller_ssl_expire_time_seconds Number of seconds since 1970 to the SSL Certificate expire.\n An example to check if this certificate will expire in 10 days is: "nginx_ingress_controller_ssl_expire_time_seconds < (time() + (10 * 24 * 3600))"
113115
# TYPE nginx_ingress_controller_ssl_expire_time_seconds gauge
114-
nginx_ingress_controller_ssl_expire_time_seconds{class="nginx",host="demo",identifier="abcd1234-100",namespace="default",secret_name=""} 1.351807721e+09
116+
nginx_ingress_controller_ssl_expire_time_seconds{class="nginx",host="demo",identifier="abcd1234-100",namespace="secret-namespace",secret_name="secret-name"} 1.351807721e+09
115117
`,
116118
metrics: []string{"nginx_ingress_controller_ssl_expire_time_seconds"},
117119
},

0 commit comments

Comments
 (0)