From dbe415fb917db502ddb7d73acd027606b0b4756f Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Tue, 7 Jun 2022 16:39:50 +0800 Subject: [PATCH] Only log non ErrNotExist errors in git.GetNote (#19884) * Fix GetNote * Only log errors if the error is not ErrNotExist Signed-off-by: Andrew Thornton Co-authored-by: Andrew Thornton --- modules/git/notes_nogogit.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/modules/git/notes_nogogit.go b/modules/git/notes_nogogit.go index e3f0a3fee9..1476805dcd 100644 --- a/modules/git/notes_nogogit.go +++ b/modules/git/notes_nogogit.go @@ -46,7 +46,10 @@ func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note) commitID = commitID[2:] } if err != nil { - log.Error("Unable to find git note corresponding to the commit %q. Error: %v", originalCommitID, err) + // Err may have been updated by the SubTree we need to recheck if it's again an ErrNotExist + if !IsErrNotExist(err) { + log.Error("Unable to find git note corresponding to the commit %q. Error: %v", originalCommitID, err) + } return err } }