This project is mirrored from https://gitlab.com/gitlab-org/gitaly.git.
Pull mirroring updated .
- Aug 07, 2023
-
-
Patrick Steinhardt authored
Support the SHA256 object format in the UserDeleteBranch RPC. Changelog: added
-
Patrick Steinhardt authored
One of the UserDeleteBranch tests is still using hardcoded object IDs, which is going to be a problem when we want to test with SHA256. While we could just use `gittest.ObjectHashDependent()` and hardcode both SHA1 and SHA256 object IDs, this would be a test smell. While we could just put the complete service instantiation into a setup function, this would be quite verbose. Instead, we only parameterize the `expectedErr` function so that it gets the expected old object ID as input. While not exactly pretty, it at least works.
-
Patrick Steinhardt authored
Refactor tests for the UserDeleteBranch RPC to not use seed repos. This prepares them for testing with the SHA256 object format.
-
- Aug 04, 2023
-
-
Sami Hiltunen authored
go: Update module gocloud.dev to v0.32.0 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6146 Merged-by:
Sami Hiltunen <shiltunen@gitlab.com> Approved-by:
Sami Hiltunen <shiltunen@gitlab.com> Co-authored-by:
GitLab Renovate Bot <gitlab-bot@gitlab.com>
-
Justin Tobler authored
testhelper: Escape backticks in git check script See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6173 Merged-by:
Justin Tobler <jtobler@gitlab.com> Approved-by:
Justin Tobler <jtobler@gitlab.com> Co-authored-by:
Will Chandler <wchandler@gitlab.com>
-
Christian Couder authored
Wrap server-side errors with ErrSkipped See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6171 Merged-by:
Christian Couder <chriscool@tuxfamily.org> Approved-by:
Christian Couder <chriscool@tuxfamily.org> Approved-by:
Toon Claes <toon@gitlab.com> Co-authored-by:
James Fargher <jfargher@gitlab.com>
-
GitLab Renovate Bot authored
-
Toon Claes authored
go: Update golang.org/x/exp digest to d63ba01 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6168 Merged-by:
Toon Claes <toon@gitlab.com> Approved-by:
Will Chandler <wchandler@gitlab.com> Approved-by:
Toon Claes <toon@gitlab.com> Co-authored-by:
GitLab Renovate Bot <gitlab-bot@gitlab.com>
-
Quang-Minh Nguyen authored
Makefile: test-gitaly-linters requires gotestsum See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6174 Merged-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Approved-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Co-authored-by:
Will Chandler <wchandler@gitlab.com>
-
- Aug 03, 2023
-
-
Will Chandler authored
The `test-gitaly-linters` target requires `gotestsum` to be present to execute successfully, but does not list it as a dependency. In typical operation in CI this is not an issue as we always run tests first, but running it standalone on a clean worktree will fail. Add `${GOTESTSUM}` as a dependency for the task.
-
GitLab Release Tools Bot authored
[ci skip]
-
Will Chandler authored
The unescaped backticks in `testhelper/testdata/home/bin/git` will cause the shell to attempt to execute `gittest.Exec()`. This will fail and print error along with the intended message. command substitution: line 11: syntax error: unexpected end of file Escape the backticks.
-
Will Chandler authored
operations Remove RevertPureGit feature flag See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6165 Merged-by:
Will Chandler <wchandler@gitlab.com> Approved-by:
Will Chandler <wchandler@gitlab.com> Reviewed-by:
karthik nayak <knayak@gitlab.com> Co-authored-by:
John Cai <jcai@gitlab.com>
-
GitLab Release Tools Bot authored
[ci skip]
-
James Fargher authored
These errors were not reporting the original error, and the restore case was not properly translating the GRPC error into the backup.ErrSkipped type. Backups now write an empty ref file for an empty repository. So they are differentiated.
-
- Aug 02, 2023
-
-
Justin Tobler authored
operations: More SHA256 support See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6149 Merged-by:
Justin Tobler <jtobler@gitlab.com> Approved-by:
karthik nayak <knayak@gitlab.com> Approved-by:
Justin Tobler <jtobler@gitlab.com> Reviewed-by:
Patrick Steinhardt <psteinhardt@gitlab.com> Reviewed-by:
karthik nayak <knayak@gitlab.com> Co-authored-by:
Patrick Steinhardt <psteinhardt@gitlab.com>
-
John Cai authored
Differentiate between empty and missing repos when creating a backup See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6090 Merged-by:
John Cai <jcai@gitlab.com> Approved-by:
Christian Couder <chriscool@tuxfamily.org> Approved-by:
John Cai <jcai@gitlab.com> Co-authored-by:
James Fargher <jfargher@gitlab.com>
-
John Cai authored
Now that git2go's revert is not called anywhere, remove it.
-
John Cai authored
Now that the pure git implementation of revert has been running in production for a while without issue, let's remove the feature flag
-
Toon Claes authored
CommitDiff: remove deprecated field `ignore_whitespace_change` Closes #4764 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6113 Merged-by:
Toon Claes <toon@gitlab.com> Approved-by:
Toon Claes <toon@gitlab.com> Reviewed-by:
karthik nayak <knayak@gitlab.com> Co-authored-by:
arkn98 <2424696-arkn98@users.noreply.gitlab.com>
-
Quang-Minh Nguyen authored
tools/golangci-lint: Update module golang.org/x/tools to v0.11.1 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6167 Merged-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Approved-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Approved-by:
Will Chandler <wchandler@gitlab.com> Reviewed-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Co-authored-by:
GitLab Renovate Bot <gitlab-bot@gitlab.com>
-
James Fargher authored
go: Update github.com/ProtonMail/go-crypto digest to 5aa5874 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6104 Merged-by:
James Fargher <jfargher@gitlab.com> Approved-by:
James Fargher <jfargher@gitlab.com> Co-authored-by:
GitLab Renovate Bot <gitlab-bot@gitlab.com>
-
Sami Hiltunen authored
housekeeping: Remove unused object repack strategies See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6158 Merged-by:
Sami Hiltunen <shiltunen@gitlab.com> Approved-by:
Sami Hiltunen <shiltunen@gitlab.com> Approved-by:
James Fargher <jfargher@gitlab.com> Co-authored-by:
Patrick Steinhardt <psteinhardt@gitlab.com>
-
Sami Hiltunen authored
go: Update module github.com/opencontainers/runtime-spec to v1.1.0 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6159 Merged-by:
Sami Hiltunen <shiltunen@gitlab.com> Approved-by:
Sami Hiltunen <shiltunen@gitlab.com> Approved-by:
Christian Couder <chriscool@tuxfamily.org> Co-authored-by:
GitLab Renovate Bot <gitlab-bot@gitlab.com>
-
Sami Hiltunen authored
tools/goimports: Update module golang.org/x/tools to v0.11.1 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6166 Merged-by:
Sami Hiltunen <shiltunen@gitlab.com> Approved-by:
karthik nayak <knayak@gitlab.com> Approved-by:
Sami Hiltunen <shiltunen@gitlab.com> Co-authored-by:
GitLab Renovate Bot <gitlab-bot@gitlab.com>
-
- Aug 01, 2023
-
-
GitLab Renovate Bot authored
-
James Fargher authored
We want to check that the way we generate backups for empty repositories will not cause issues for the current restore logic.
-
James Fargher authored
This method is now unused.
-
James Fargher authored
Having an empty ref list allows us to differentiate between repositories that don't exist and repositories that are empty. Changelog: updated
-
James Fargher authored
These RPC tests are only smoke testing that the backups get created properly. For this RPC, if there are no errors, then a refs file will always exist, but a bundle may not exist. So for the sake of ease, we swap the final check to look for refs instead of a bundle.
-
James Fargher authored
In order to differentiate between an empty repository and a non-existent repository, backups will begin writing an empty ref list file to signify an empty repository. So here we add an option to check for ref lists separately from bundles.
-
James Fargher authored
The default branch of the seed repos was master, so any reference to master needs to be converted to the default branch constant git.DefaultBranch.
-
Will Chandler authored
praefect/config: Support generating configuration via external command Closes #5076 See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6157 Merged-by:
Will Chandler <wchandler@gitlab.com> Approved-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Approved-by:
Will Chandler <wchandler@gitlab.com> Reviewed-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Co-authored-by:
Patrick Steinhardt <psteinhardt@gitlab.com>
-
GitLab Release Tools Bot authored
-
GitLab Bot authored
-
Justin Tobler authored
gitaly-git2go: Remove Merge See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6164 Merged-by:
Justin Tobler <jtobler@gitlab.com> Approved-by:
Justin Tobler <jtobler@gitlab.com> Co-authored-by:
John Cai <jcai@gitlab.com>
-
Will Chandler authored
Assert repository state as seen by a transaction See merge request https://gitlab.com/gitlab-org/gitaly/-/merge_requests/6143 Merged-by:
Will Chandler <wchandler@gitlab.com> Approved-by:
Quang-Minh Nguyen <qmnguyen@gitlab.com> Approved-by:
Will Chandler <wchandler@gitlab.com> Co-authored-by:
Sami Hiltunen <shiltunen@gitlab.com>
-
GitLab Release Tools Bot authored
[ci skip]
-
GitLab Renovate Bot authored
-
GitLab Release Tools Bot authored
[ci skip]
-