Skip to content

Improve Geo logging of repository errors

James Lopez requested to merge feature/improve-geo-logging into master

Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/2968

From the points taken from the issue above:

Consolidate all Geo-related activity into a common log (e.g. geo.log)

This was practically done, but I encountered a couple of places where we weren't doing this.

Output structured (e.g. JSON) logs in this file

This was already in place.

Store the error message with the ProjectRegistry table

Apart from the repository, I've also added the same for Wiki (in light of my next issue which will require more separation of these two).

Show a list of failed projects and their reasons for failures

This was discussed here and we won't have the UI for 10.3. But, we do need the API. I've added this based on this comment. However, we can consider not passing an ID altogether and just check the current node (a bit less RESTish, we'll need a separate endpoint). => I finally went for this.

Edited by James Lopez

Merge request reports