diff options
author | John Cai <jcai@gitlab.com> | 2022-05-13 00:56:25 +0300 |
---|---|---|
committer | John Cai <jcai@gitlab.com> | 2022-05-13 04:43:29 +0300 |
commit | 92fc011f905a5e80f719d5310b9831243eb31630 (patch) | |
tree | 1a6af1bd6e3de2c0430b84926a6121ddf4eeead1 | |
parent | 42fab8fc526215f9426bc9f459f9e6da0951c574 (diff) |
repository: Log repo size calculationsjc-log-repo-size
Because we are switching to a new way of calculating repo sizes, it
would be helpful to know what the discrepancy is so that if it is large
and unexpedted, there's a way to know.
Changelog: changed
-rw-r--r-- | internal/gitaly/service/repository/size.go | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/internal/gitaly/service/repository/size.go b/internal/gitaly/service/repository/size.go index a6bb0d33e..3ac49f405 100644 --- a/internal/gitaly/service/repository/size.go +++ b/internal/gitaly/service/repository/size.go @@ -26,6 +26,11 @@ func (s *server) RepositorySize(ctx context.Context, in *gitalypb.RepositorySize excludes = append(excludes, prefix+"*") } + path, err := repo.Path() + if err != nil { + return nil, err + } + if featureflag.RevlistForRepoSize.IsEnabled(ctx) { size, err = repo.Size( ctx, @@ -36,11 +41,13 @@ func (s *server) RepositorySize(ctx context.Context, in *gitalypb.RepositorySize } // return the size in kb to remain consistent size = size / 1024 + + duSize := getPathSize(ctx, path) + + ctxlogrus.Extract(ctx). + WithField("repo_size_revlist", size). + WithField("repo_size_du", duSize).Info("repository size calculated") } else { - path, err := repo.Path() - if err != nil { - return nil, err - } size = getPathSize(ctx, path) } |