Skip to content

Commit 294163d

Browse files
committed
Merge branch 'main' into remove-shard-validation-on-sts-create
2 parents fb46163 + f50205a commit 294163d

File tree

3 files changed

+22
-33
lines changed

3 files changed

+22
-33
lines changed

go.mod

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@ require (
66
github.com/KimMachineGun/automemlimit v0.6.1
77
github.com/dgryski/go-jump v0.0.0-20211018200510-ba001c3ffce0
88
github.com/fsnotify/fsnotify v1.7.0
9-
github.com/gobuffalo/flect v1.0.2
9+
github.com/go-logr/logr v1.4.1
10+
github.com/gobuffalo/flect v1.0.3
1011
github.com/google/go-cmp v0.6.0
1112
github.com/oklog/run v1.1.0
12-
github.com/prometheus/client_golang v1.20.3
13+
github.com/prometheus/client_golang v1.20.4
1314
github.com/prometheus/client_model v0.6.1
1415
github.com/prometheus/common v0.59.1
15-
github.com/prometheus/exporter-toolkit v0.12.0
16+
github.com/prometheus/exporter-toolkit v0.13.0
1617
github.com/robfig/cron/v3 v3.0.1
1718
github.com/spf13/cobra v1.8.1
1819
github.com/spf13/viper v1.19.0
@@ -38,9 +39,6 @@ require (
3839
github.com/docker/go-units v0.4.0 // indirect
3940
github.com/emicklei/go-restful/v3 v3.11.0 // indirect
4041
github.com/evanphx/json-patch v5.6.0+incompatible // indirect
41-
github.com/go-kit/log v0.2.1 // indirect
42-
github.com/go-logfmt/logfmt v0.5.1 // indirect
43-
github.com/go-logr/logr v1.4.1 // indirect
4442
github.com/go-openapi/jsonpointer v0.19.6 // indirect
4543
github.com/go-openapi/jsonreference v0.20.2 // indirect
4644
github.com/go-openapi/swag v0.22.3 // indirect

go.sum

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,6 @@ github.com/frankban/quicktest v1.14.6 h1:7Xjx+VpznH+oBnejlPUj8oUpdxnVs4f8XU8WnHk
3030
github.com/frankban/quicktest v1.14.6/go.mod h1:4ptaffx2x8+WTWXmUCuVU6aPUX1/Mz7zb5vbUoiM6w0=
3131
github.com/fsnotify/fsnotify v1.7.0 h1:8JEhPFa5W2WU7YfeZzPNqzMP6Lwt7L2715Ggo0nosvA=
3232
github.com/fsnotify/fsnotify v1.7.0/go.mod h1:40Bi/Hjc2AVfZrqy+aj+yEI+/bRxZnMJyTJwOpGvigM=
33-
github.com/go-kit/log v0.2.1 h1:MRVx0/zhvdseW+Gza6N9rVzU/IVzaeE1SFI4raAhmBU=
34-
github.com/go-kit/log v0.2.1/go.mod h1:NwTd00d/i8cPZ3xOwwiv2PO5MOcx78fFErGNcVmBjv0=
35-
github.com/go-logfmt/logfmt v0.5.1 h1:otpy5pqBCBZ1ng9RQ0dPu4PN7ba75Y/aA+UpowDyNVA=
36-
github.com/go-logfmt/logfmt v0.5.1/go.mod h1:WYhtIu8zTZfxdn5+rREduYbwxfcBr/Vr6KEVveWlfTs=
3733
github.com/go-logr/logr v1.4.1 h1:pKouT5E8xu9zeFC39JXRDukb6JFQPXM5p5I91188VAQ=
3834
github.com/go-logr/logr v1.4.1/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY=
3935
github.com/go-openapi/jsonpointer v0.19.6 h1:eCs3fxoIi3Wh6vtgmLTOjdhSpiqphQ+DaPn38N2ZdrE=
@@ -44,8 +40,8 @@ github.com/go-openapi/swag v0.22.3 h1:yMBqmnQ0gyZvEb/+KzuWZOXgllrXT4SADYbvDaXHv/
4440
github.com/go-openapi/swag v0.22.3/go.mod h1:UzaqsxGiab7freDnrUUra0MwWfN/q7tE4j+VcZ0yl14=
4541
github.com/go-task/slim-sprig v0.0.0-20230315185526-52ccab3ef572 h1:tfuBGBXKqDEevZMzYi5KSi8KkcZtzBcTgAUUtapy0OI=
4642
github.com/go-task/slim-sprig v0.0.0-20230315185526-52ccab3ef572/go.mod h1:9Pwr4B2jHnOSGXyyzV8ROjYa2ojvAY6HCGYYfMoC3Ls=
47-
github.com/gobuffalo/flect v1.0.2 h1:eqjPGSo2WmjgY2XlpGwo2NXgL3RucAKo4k4qQMNA5sA=
48-
github.com/gobuffalo/flect v1.0.2/go.mod h1:A5msMlrHtLqh9umBSnvabjsMrCcCpAyzglnDvkbYKHs=
43+
github.com/gobuffalo/flect v1.0.3 h1:xeWBM2nui+qnVvNM4S3foBhCAL2XgPU+a7FdpelbTq4=
44+
github.com/gobuffalo/flect v1.0.3/go.mod h1:A5msMlrHtLqh9umBSnvabjsMrCcCpAyzglnDvkbYKHs=
4945
github.com/godbus/dbus/v5 v5.0.4 h1:9349emZab16e7zQvpmsbtjc18ykshndd8y2PG3sgJbA=
5046
github.com/godbus/dbus/v5 v5.0.4/go.mod h1:xhWf0FNVPg57R7Z0UbKHbJfkEywrmjJnf7w5xrFpKfA=
5147
github.com/gogo/protobuf v1.3.2 h1:Ov1cvc58UF3b5XjBnZv7+opcTcQFZebYjWzi34vdm4Q=
@@ -125,14 +121,14 @@ github.com/pkg/errors v0.9.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINE
125121
github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
126122
github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 h1:Jamvg5psRIccs7FGNTlIRMkT8wgtp5eCXdBlqhYGL6U=
127123
github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
128-
github.com/prometheus/client_golang v1.20.3 h1:oPksm4K8B+Vt35tUhw6GbSNSgVlVSBH0qELP/7u83l4=
129-
github.com/prometheus/client_golang v1.20.3/go.mod h1:PIEt8X02hGcP8JWbeHyeZ53Y/jReSnHgO035n//V5WE=
124+
github.com/prometheus/client_golang v1.20.4 h1:Tgh3Yr67PaOv/uTqloMsCEdeuFTatm5zIq5+qNN23vI=
125+
github.com/prometheus/client_golang v1.20.4/go.mod h1:PIEt8X02hGcP8JWbeHyeZ53Y/jReSnHgO035n//V5WE=
130126
github.com/prometheus/client_model v0.6.1 h1:ZKSh/rekM+n3CeS952MLRAdFwIKqeY8b62p8ais2e9E=
131127
github.com/prometheus/client_model v0.6.1/go.mod h1:OrxVMOVHjw3lKMa8+x6HeMGkHMQyHDk9E3jmP2AmGiY=
132128
github.com/prometheus/common v0.59.1 h1:LXb1quJHWm1P6wq/U824uxYi4Sg0oGvNeUm1z5dJoX0=
133129
github.com/prometheus/common v0.59.1/go.mod h1:GpWM7dewqmVYcd7SmRaiWVe9SSqjf0UrwnYnpEZNuT0=
134-
github.com/prometheus/exporter-toolkit v0.12.0 h1:DkE5RcEZR3lQA2QD5JLVQIf41dFKNsVMXFhgqcif7fo=
135-
github.com/prometheus/exporter-toolkit v0.12.0/go.mod h1:fQH0KtTn0yrrS0S82kqppRjDDiwMfIQUwT+RBRRhwUc=
130+
github.com/prometheus/exporter-toolkit v0.13.0 h1:lmA0Q+8IaXgmFRKw09RldZmZdnvu9wwcDLIXGmTPw1c=
131+
github.com/prometheus/exporter-toolkit v0.13.0/go.mod h1:2uop99EZl80KdXhv/MxVI2181fMcwlsumFOqBecGkG0=
136132
github.com/prometheus/procfs v0.15.1 h1:YagwOFzUgYfKKHX6Dr+sHT7km/hxC76UB0learggepc=
137133
github.com/prometheus/procfs v0.15.1/go.mod h1:fB45yRUv8NstnjriLhBQLuOUt+WW4BsoGhij/e3PBqk=
138134
github.com/robfig/cron/v3 v3.0.1 h1:WdRxkvbJztn8LMz/QEvLN5sBU+xKpSqwwUO1Pjr4qDs=

pkg/app/server.go

Lines changed: 12 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import (
2121
"crypto/md5" //nolint:gosec
2222
"encoding/binary"
2323
"fmt"
24+
"log/slog"
2425
"net"
2526
"net/http"
2627
"net/http/pprof"
@@ -30,6 +31,8 @@ import (
3031
"strings"
3132
"time"
3233

34+
"github.com/go-logr/logr"
35+
3336
"gopkg.in/yaml.v3"
3437
"k8s.io/client-go/kubernetes"
3538
_ "k8s.io/client-go/plugin/pkg/client/auth" // Initialize common client auth plugins.
@@ -66,19 +69,6 @@ const (
6669
readyzPath = "/readyz"
6770
)
6871

69-
// promLogger implements promhttp.Logger
70-
type promLogger struct{}
71-
72-
func (pl promLogger) Println(v ...interface{}) {
73-
klog.Error(v...)
74-
}
75-
76-
// promLogger implements the Logger interface
77-
func (pl promLogger) Log(v ...interface{}) error {
78-
klog.Info(v...)
79-
return nil
80-
}
81-
8272
// RunKubeStateMetricsWrapper runs KSM with context cancellation.
8373
func RunKubeStateMetricsWrapper(ctx context.Context, opts *options.Options) error {
8474
err := RunKubeStateMetrics(ctx, opts)
@@ -93,7 +83,6 @@ func RunKubeStateMetricsWrapper(ctx context.Context, opts *options.Options) erro
9383
// Any out-of-tree custom resource metrics could be registered by newing a registry factory
9484
// which implements customresource.RegistryFactory and pass all factories into this function.
9585
func RunKubeStateMetrics(ctx context.Context, opts *options.Options) error {
96-
promLogger := promLogger{}
9786
ksmMetricsRegistry := prometheus.NewRegistry()
9887
ksmMetricsRegistry.MustRegister(versionCollector.NewCollector("kube_state_metrics"))
9988
durationVec := promauto.With(ksmMetricsRegistry).NewHistogramVec(
@@ -359,11 +348,14 @@ func RunKubeStateMetrics(ctx context.Context, opts *options.Options) error {
359348
WebConfigFile: &tlsConfig,
360349
}
361350

351+
handler := logr.ToSlogHandler(klog.Background())
352+
sLogger := slog.New(handler)
353+
362354
// Run Telemetry server
363355
{
364356
g.Add(func() error {
365357
klog.InfoS("Started kube-state-metrics self metrics server", "telemetryAddress", telemetryListenAddress)
366-
return web.ListenAndServe(&telemetryServer, &telemetryFlags, promLogger)
358+
return web.ListenAndServe(&telemetryServer, &telemetryFlags, sLogger)
367359
}, func(error) {
368360
ctxShutDown, cancel := context.WithTimeout(ctx, 3*time.Second)
369361
defer cancel()
@@ -374,7 +366,7 @@ func RunKubeStateMetrics(ctx context.Context, opts *options.Options) error {
374366
{
375367
g.Add(func() error {
376368
klog.InfoS("Started metrics server", "metricsServerAddress", metricsServerListenAddress)
377-
return web.ListenAndServe(&metricsServer, &metricsFlags, promLogger)
369+
return web.ListenAndServe(&metricsServer, &metricsFlags, sLogger)
378370
}, func(error) {
379371
ctxShutDown, cancel := context.WithTimeout(ctx, 3*time.Second)
380372
defer cancel()
@@ -393,8 +385,11 @@ func RunKubeStateMetrics(ctx context.Context, opts *options.Options) error {
393385
func buildTelemetryServer(registry prometheus.Gatherer) *http.ServeMux {
394386
mux := http.NewServeMux()
395387

388+
handler := logr.ToSlogHandler(klog.Background())
389+
sLogger := slog.NewLogLogger(handler, slog.LevelError)
390+
396391
// Add metricsPath
397-
mux.Handle(metricsPath, promhttp.HandlerFor(registry, promhttp.HandlerOpts{ErrorLog: promLogger{}}))
392+
mux.Handle(metricsPath, promhttp.HandlerFor(registry, promhttp.HandlerOpts{ErrorLog: sLogger}))
398393

399394
// Add readyzPath
400395
mux.Handle(readyzPath, http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {

0 commit comments

Comments
 (0)