artin/sourcegraph
1
0
Fork 0
mirror of https://github.com/sourcegraph/sourcegraph.git synced 2026-02-06 19:21:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't log when repo-updater shuts down (#62121)

Browse Source 
This should reduce some sentry error reporting for non-actionable errors.

Test plan:

Will see if sentry is more silent.
This commit is contained in:
Erik Seliger 2024-04-25 02:04:20 +02:00 committed by GitHub
parent 366e020552
commit 166b416909
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cmd/repo-updater/internal/scheduler/scheduler.go
View File

@ -190,8 +190,8 @@ func (s *UpdateScheduler) runUpdateLoop(ctx context.Context) {
} else {
schedError.WithLabelValues("repoUpdateResponse").Inc()
// We don't want to spam our logs when the rate limiter has been set to block all
// updates
if !strings.Contains(err.Error(), ratelimit.ErrBlockAll.Error()) {
// updates, or when repo-updater is shutting down.
if !strings.Contains(err.Error(), ratelimit.ErrBlockAll.Error()) && ctx.Err() == nil {
subLogger.Error("error updating repo", log.Error(err), log.String("uri", string(repo.Name)))
}
}