k8s-ttl-controller icon indicating copy to clipboard operation
k8s-ttl-controller copied to clipboard

Use logr instead of log/slog

Open hiddenmarten opened this issue 1 year ago • 4 comments

Describe the feature request

Use logr instead of log/slog

Why do you personally want this feature to be implemented?

https://github.com/TwiN/k8s-ttl-controller/pull/102#issuecomment-2534486024

How long have you been using this project?

No response

Additional information

No response

hiddenmarten avatar Dec 15 '24 19:12 hiddenmarten

Thank you for suggesting this library - I appreciate the opportunity to explore it. I gave it a try and found that it currently doesn't seem to support logs in JSON format, which is something I was hoping for in our use case.

If there’s a workaround or something I might have missed, I’d be happy to revisit it.

hiddenmarten avatar Dec 15 '24 19:12 hiddenmarten

Oh I was suggesting you implement it in the library itself (if you were up to the challenge).

Again, though, only if you're up for it. If not, ill pick this up when I have some time

TwiN avatar Dec 16 '24 03:12 TwiN

Thank you for considering me, but I don’t feel confident enough to take on this challenge.

I truly appreciate this project and would love to contribute since I’m also a user. However, improving the logr library is beyond the scope of this project.

hiddenmarten avatar Dec 19 '24 20:12 hiddenmarten

No worries :)

TwiN avatar Dec 20 '24 02:12 TwiN