Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitaly.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Steinhardt <psteinhardt@gitlab.com>2023-09-25 13:20:29 +0300
committerPatrick Steinhardt <psteinhardt@gitlab.com>2023-10-11 09:27:58 +0300
commit978635679011c656e7bd5b9ade7d8db5767d03ba (patch)
tree05b43c443db0709e2a0c18e8f580e6c5e0c05d9b
parent6b20847b611c8c43ae4b4348b3d6f1ec338e159a (diff)
service/cleanup: Replace use of ctxlogrus with injected logger
The ctxlogrus package is going away with the replacement being log fields extracted from the context via `log.DebugContext()` et al. Refactor the code to stop using ctxlogrus by injecting a logger and using the new context-based logging methods.
-rw-r--r--internal/gitaly/service/cleanup/cleaner.go7
1 files changed, 4 insertions, 3 deletions
diff --git a/internal/gitaly/service/cleanup/cleaner.go b/internal/gitaly/service/cleanup/cleaner.go
index 81ae71a29..574933f7f 100644
--- a/internal/gitaly/service/cleanup/cleaner.go
+++ b/internal/gitaly/service/cleanup/cleaner.go
@@ -24,6 +24,7 @@ type forEachFunc func(ctx context.Context, oldOID, newOID string, isInternalRef
type cleaner struct {
ctx context.Context
forEach forEachFunc
+ logger log.Logger
// Map of SHA -> reference names
table map[string][]git.ReferenceName
@@ -42,7 +43,7 @@ func newCleaner(ctx context.Context, logger log.Logger, repo git.RepositoryExecu
return nil, err
}
- return &cleaner{ctx: ctx, table: table, repo: repo, forEach: forEach}, nil
+ return &cleaner{ctx: ctx, logger: logger, table: table, repo: repo, forEach: forEach}, nil
}
// applyObjectMap processes an object map file generated by git filter-repo, or
@@ -119,10 +120,10 @@ func (c *cleaner) processEntry(ctx context.Context, updater *updateref.Updater,
return nil
}
- log.FromContext(ctx).WithFields(log.Fields{
+ c.logger.WithFields(log.Fields{
"sha": oldSHA,
"refs": refs,
- }).Info("removing internal references")
+ }).InfoContext(ctx, "removing internal references")
// Remove the internal refs pointing to oldSHA
for _, ref := range refs {