Skip to content

Log repository full_path in repository check error log

Diana Stanley requested to merge dstanley-master-patch-42066 into master

What does this MR do?

When logging the error message associated with a failed repository check, we do not currently make any attempt to include the repo path in the log message, making the repocheck.log pretty much useless. This MR adds the repo full_path to the logged message.

BEFORE:

E, [2021-07-08T04:16:35.628322 #3861] ERROR -- : Could not fsck repository: error: refs/heads/patch-1: invalid sha1 pointer 0000000000000000000000000000000000000000

AFTER:

E, [2021-07-08T04:45:54.965781 #10066] ERROR -- : root/test.wiki: Could not fsck repository: error: refs/patch-1: invalid sha1 pointer 0000000000000000000000000000000000000000

If #21547 (closed) is ever resolved, this fix will make the logged messages useful instead of maddening.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

featureenhancement

Edited by Diana Stanley

Merge request reports

Loading