Skip to content

Remove unused GeoLogger class

Michael Kozono requested to merge mk/remove-unused-geo-logger-class into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Removes unused GeoLogger class.

I found that Gitlab::GeoLogger was added as part of !2845 (merged), and the Admin::LogsController no longer exists since GitLab 13.0.

GeoLogger is not referenced elsewhere in the codebase.

Looking around at common places that output logs to geo.log, they all use Gitlab::Geo::Logger: https://gitlab.com/search?search=Gitlab%3A%3AGeo%3A%3ALogger&nav_source=navbar&project_id=278964&group_id=9970&search_code=true&repository_ref=master

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Michael Kozono

Merge request reports