diff options
| author | Pavlo Strokov <pstrokov@gitlab.com> | 2022-10-20 14:14:56 +0300 |
|---|---|---|
| committer | Pavlo Strokov <pstrokov@gitlab.com> | 2022-11-08 12:28:56 +0300 |
| commit | 91d2650ff7930db296bc068d8577bd0fce2cbedd (patch) | |
| tree | 6fdcd94533a829ab4686153579aff9a7c1a364f5 | |
| parent | e68414294218505f54a7804dfd3bca0035d1dbe0 (diff) | |
praefect: Use pre-defined error
We have pre-defined error that looks the same, so let's use it.
| -rw-r--r-- | internal/praefect/rename_repository.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/internal/praefect/rename_repository.go b/internal/praefect/rename_repository.go index 97e9059d4..d6d4cde35 100644 --- a/internal/praefect/rename_repository.go +++ b/internal/praefect/rename_repository.go @@ -4,6 +4,7 @@ import ( "errors" "fmt" + gitalyerrors "gitlab.com/gitlab-org/gitaly/v15/internal/errors" "gitlab.com/gitlab-org/gitaly/v15/internal/gitaly/storage" "gitlab.com/gitlab-org/gitaly/v15/internal/helper" "gitlab.com/gitlab-org/gitaly/v15/internal/praefect/commonerr" @@ -16,7 +17,7 @@ func validateRenameRepositoryRequest(req *gitalypb.RenameRepositoryRequest, virt // These checks are not strictly necessary but they exist to keep retain compatibility with // Gitaly's tested behavior. if req.GetRepository() == nil { - return helper.ErrInvalidArgumentf("empty Repository") + return helper.ErrInvalidArgument(gitalyerrors.ErrEmptyRepository) } else if req.GetRelativePath() == "" { return helper.ErrInvalidArgumentf("destination relative path is empty") } else if _, ok := virtualStorages[req.GetRepository().GetStorageName()]; !ok { |