Merge branch '5178-geo-verification-rename-log' into 'master'

Use geo.log for secondary repository verification logging

Closes #5178

See merge request gitlab-org/gitlab-ee!5095

ee/app/controllers/ee/admin/logs_controller.rb

@@ -6,8 +6,7 @@ module EE::Admin::LogsController
   def loggers
     strong_memoize(:loggers) do
       super + [
-        Gitlab::GeoLogger,
-        Gitlab::Geo::RepositoryVerificationLogger
+        Gitlab::GeoLogger
       ]
     end
   end

ee/app/services/geo/repository_verify_secondary_service.rb

 module Geo
   class RepositoryVerifySecondaryService
-    include Gitlab::Geo::RepositoryVerificationLogHelpers
+    include Gitlab::Geo::ProjectLogHelpers
 
     delegate :project, to: :registry
 
@@ -66,7 +66,7 @@ module Geo
       }
 
       if failure
-        log_error(failure, exception, type: type, repository_path: repository_path, full_path: path_to_repo)
+        log_error(failure, exception, type: type, repository_full_path: path_to_repo)
       end
 
       registry.update!(attrs)

ee/changelogs/unreleased/5178-geo-verification-rename-log.yml

+---
+title: Geo secondary repository verification messages now appear in geo.log
+merge_request: 5095
+author:
+type: fixed

ee/lib/gitlab/geo/repository_verification_log_helpers.rb

-module Gitlab
-  module Geo
-    module RepositoryVerificationLogHelpers
-      include ProjectLogHelpers
-
-      protected
-
-      def geo_logger
-        Gitlab::Geo::RepositoryVerificationLogger
-      end
-    end
-  end
-end

ee/lib/gitlab/geo/repository_verification_logger.rb

-module Gitlab
-  module Geo
-    class RepositoryVerificationLogger < ::Gitlab::Geo::Logger
-      def self.file_name_noext
-        'geo_repository_verification'
-      end
-    end
-  end
-end