Switch to slog

[lucacome]

Proposed changes

After prometheus/exporter-toolkit#240 we can switch to using slog.

Closes #484

Checklist

Before creating a PR, run through this checklist and mark each as complete.

• I have read the CONTRIBUTING guide
• I have proven my fix is effective or that my feature works
• I have checked that all unit tests pass after adding my changes
• I have ensured the README is up to date
• I have rebased my branch onto main
• I will ensure my PR is targeting the main branch and pulling from my branch on my own fork

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 26 lines in your changes missing coverage. Please review.

Project coverage is 3.90%. Comparing base (46f7fbf) to head (c7e6d4d).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
exporter.go	0.00%	15 Missing ⚠️
collector/nginx_plus.go	0.00%	9 Missing ⚠️
collector/nginx.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main    #841   +/-   ##
=====================================
  Coverage   3.90%   3.90%           
=====================================
  Files          5       5           
  Lines       1307    1307           
=====================================
  Hits          51      51           
  Misses      1256    1256           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kate-osborn]

To achieve consistency across all NGF logs would we need to implement slog.Handler and pass that to the slog.Logger?

[lucacome]

logr has FromSlogHandler and ToSlogHandler that I think can be used for this

[kate-osborn]

logr has FromSlogHandler and ToSlogHandler that I think can be used for this

Awesome. Do you mind opening an issue in NGF backlog for that work?