Support GL_REPOSITORY from API and pass it to Gitaly on ReceivePack (!162) · Merge requests · GitLab.org / gitlab-workhorse · GitLab

Snippets Groups Projects

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Merged username-removed-367626 requested to merge gl-repository into master 7 years ago

Necessary for https://gitlab.com/gitlab-org/gitlab-ce/issues/29925

Activity

username-removed-367626 @eReGeBe · 7 years ago

Author Developer

WIP until the gitaly-proto changes are released.
Jacob Vosmaer (GitLab) @jacobvosmaer-gitlab · 7 years ago

Contributor

@eReGeBe looks good. We should also set GL_REPOSITORY in startGitCommand. https://gitlab.com/gitlab-org/gitlab-workhorse/blob/32f56a2fca69588ca18bb03584539008e6908696/internal/git/git-http.go#L77-91
Jacob Vosmaer (GitLab) assigned to @eReGeBe 7 years ago

assigned to @eReGeBe
username-removed-367626 added 1 commit 7 years ago
added 1 commit

cd80e2ff - Support GL_REPOSITORY from API and pass it to Gitaly on ReceivePack

Compare with previous version
username-removed-367626 unmarked as a Work In Progress 7 years ago

unmarked as a Work In Progress
username-removed-367626 assigned to @jacobvosmaer-gitlab 7 years ago

assigned to @jacobvosmaer-gitlab
username-removed-367626 @eReGeBe · 7 years ago

Author Developer

Now with GL_REPOSITORY in startGitCommand and correct proto version!
Jacob Vosmaer (GitLab) approved this merge request 7 years ago

approved this merge request
Jacob Vosmaer (GitLab) assigned to @nick.thomas 7 years ago

assigned to @nick.thomas
Nick Thomas @nick.thomas · 7 years ago

Maintainer

LGTM!
Nick Thomas merged 7 years ago

merged
Nick Thomas mentioned in commit ce848485 7 years ago

mentioned in commit ce848485

Please register or sign in to reply