Metrics Server: klogr.New is deprecated use textlogger.NewLogger instead

[zroubalik]

Proposal

keda/cmd/adapter/main.go

Line 58 in 9a00d1e

var logger = klogr.New().WithName("keda_metrics_adapter")

cmd/adapter/main.go:58:14: SA1019: klogr.New is deprecated: this uses a custom, out-dated output format. Use textlogger.NewLogger instead. (staticcheck)
var logger = klogr.New().WithName("keda_metrics_adapter")

Let's migrate to the new interface, there might be some changes in the config/parameters etc.

It might also help with issues like: #5139

Use-Case

No response

Is this a feature you are interested in implementing yourself?

Maybe

Anything else?

No response

[JorTurFer]

I think that it's a good moment to unify all the loggers across components, operator uses zap IIRC, maybe can give a try but I think that custom-metrics-apiserver can be the blocker here

[guopeng0]

Hi @zroubalik , I would like to work on this issue can you please assign it to me.

[zroubalik]

@guopeng0 thanks!

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically closed due to inactivity.