Follow-up from "Add info about update_mirror_service_json.log"
In 7f637e2a we introduced the json logger update_mirror_service_json
to track errors related to LFS mirroring updates.
Originally, the idea was to track also other errors of the update mirroring process in this file, nevertheless, any error fallback to the Rails.logger
in RepositoryUpdateMirrorWorker
.
It would be great if we could track all update mirroring errors in the same file and avoid logging them to the general Rails.logger
. That way, those will be easier to track.
Basically, for this change, we will need to create the existing logger UpdateMirrorServiceJsonLogger
in the RepositoryUpdateMirrorWorker
and pass it to UpdateMirrorService
in case it needs to log something.
Important: before implementing this change we need to make an MR to index the update_mirror_service_json
file in fluentd. If we implement this before the indexing, we will lose the capacity to see these errors because they will no longer be in the general logger file.