diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2017-06-29 08:18:57 +0000 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2017-06-29 08:18:57 +0000 |
commit | e3a6fedb1350ffcb4838b35cde86593cb1fe1715 (patch) | |
tree | 11d1a00d8d1debd3bdcc87956171734867c49c06 /app/workers | |
parent | 36e17d0fa878d7284672292378731d1b80a57bc8 (diff) | |
parent | e7470534e5d55e9635ae7d866eeefb13bd6811f5 (diff) | |
download | gitlab-ce-e3a6fedb1350ffcb4838b35cde86593cb1fe1715.tar.gz |
Merge branch 'postreceive-gl-repository' into 'master'
Use gl_repository exclusively on PostReceive
Closes gitaly#201
See merge request !12290
Diffstat (limited to 'app/workers')
-rw-r--r-- | app/workers/post_receive.rb | 21 |
1 files changed, 3 insertions, 18 deletions
diff --git a/app/workers/post_receive.rb b/app/workers/post_receive.rb index 89286595ca6..b8f8d3750d9 100644 --- a/app/workers/post_receive.rb +++ b/app/workers/post_receive.rb @@ -2,11 +2,11 @@ class PostReceive include Sidekiq::Worker include DedicatedSidekiqQueue - def perform(project_identifier, identifier, changes) - project, is_wiki = parse_project_identifier(project_identifier) + def perform(gl_repository, identifier, changes) + project, is_wiki = Gitlab::GlRepository.parse(gl_repository) if project.nil? - log("Triggered hook for non-existing project with identifier \"#{project_identifier}\"") + log("Triggered hook for non-existing project with gl_repository \"#{gl_repository}\"") return false end @@ -59,21 +59,6 @@ class PostReceive # Nothing defined here yet. end - # To maintain backwards compatibility, we accept both gl_repository or - # repository paths as project identifiers. Our plan is to migrate to - # gl_repository only with the following plan: - # 9.2: Handle both possible values. Keep Gitlab-Shell sending only repo paths - # 9.3 (or patch release): Make GitLab Shell pass gl_repository if present - # 9.4 (or patch release): Make GitLab Shell always pass gl_repository - # 9.5 (or patch release): Handle only gl_repository as project identifier on this method - def parse_project_identifier(project_identifier) - if project_identifier.start_with?('/') - Gitlab::RepoPath.parse(project_identifier) - else - Gitlab::GlRepository.parse(project_identifier) - end - end - def log(message) Gitlab::GitLogger.error("POST-RECEIVE: #{message}") end |