Commit 601c6a98 authored by Robert Speicher's avatar Robert Speicher

Merge branch 'rebase-gl-id' into 'master'

Set GL_ID in MergeRequests::RebaseService

We missed this in 6280bcc0 I think.

See merge request !472
parents 8055ca04 4ad0f223
...@@ -27,7 +27,11 @@ module MergeRequests ...@@ -27,7 +27,11 @@ module MergeRequests
end end
# Clone # Clone
output, status = popen(%W(git clone -b #{merge_request.source_branch} -- #{source_project.repository.path_to_repo} #{tree_path})) output, status = popen(
%W(git clone -b #{merge_request.source_branch} -- #{source_project.repository.path_to_repo} #{tree_path}),
nil,
git_env
)
unless status.zero? unless status.zero?
log('Failed to clone repository for rebase:') log('Failed to clone repository for rebase:')
...@@ -36,7 +40,11 @@ module MergeRequests ...@@ -36,7 +40,11 @@ module MergeRequests
end end
# Rebase # Rebase
output, status = popen(%W(git pull --rebase #{target_project.repository.path_to_repo} #{merge_request.target_branch}), tree_path) output, status = popen(
%W(git pull --rebase #{target_project.repository.path_to_repo} #{merge_request.target_branch}),
tree_path,
git_env
)
unless status.zero? unless status.zero?
log('Failed to rebase branch:') log('Failed to rebase branch:')
...@@ -45,7 +53,11 @@ module MergeRequests ...@@ -45,7 +53,11 @@ module MergeRequests
end end
# Push # Push
output, status = popen(%W(git push -f origin #{merge_request.source_branch}), tree_path) output, status = popen(
%W(git push -f origin #{merge_request.source_branch}),
tree_path,
git_env
)
unless status.zero? unless status.zero?
log('Failed to push rebased branch:') log('Failed to push rebased branch:')
...@@ -80,5 +92,9 @@ module MergeRequests ...@@ -80,5 +92,9 @@ module MergeRequests
def clean_dir def clean_dir
FileUtils.rm_rf(tree_path) if File.exist?(tree_path) FileUtils.rm_rf(tree_path) if File.exist?(tree_path)
end end
def git_env
{ 'GL_ID' => Gitlab::GlId.gl_id(current_user) }
end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment