Merge branch 'remove-gitlab-workhorse-version-option-from-install-task' into 'master'

Don't allow to specify a repo or version when installing Workhorse

I've also updated the "patch version upgrade" doc since I did forget to update it in !6574.

See merge request !7879

doc/install/installation.md

@@ -404,18 +404,6 @@ which is the recommended location.
 
     sudo -u git -H bundle exec rake "gitlab:workhorse:install[/home/git/gitlab-workhorse]" RAILS_ENV=production
 
-You can specify a different Git repository by providing `GITLAB_WORKHORSE_REPO`:
-
-    cd /home/git/gitlab
-
-    sudo -u git -H bundle exec rake "gitlab:workhorse:install[/home/git/gitlab-workhorse]" GITLAB_WORKHORSE_REPO=https://example.com/gitlab-workhorse.git RAILS_ENV=production
-
-You can specify a different version to use by providing `GITLAB_WORKHORSE_VERSION`:
-
-    cd /home/git/gitlab
-
-    sudo -u git -H bundle exec rake "gitlab:workhorse:install[/home/git/gitlab-workhorse]" GITLAB_WORKHORSE_VERSION=0.8.1 RAILS_ENV=production
-
 ### Initialize Database and Activate Advanced Features
 
     # Go to GitLab installation folder

doc/update/patch_versions.md

@@ -45,10 +45,9 @@ sudo -u git -H git checkout v`cat /home/git/gitlab/GITLAB_SHELL_VERSION` -b v`ca
 ### 4. Update gitlab-workhorse to the corresponding version
 
 ```bash
-cd /home/git/gitlab-workhorse
-sudo -u git -H git fetch
-sudo -u git -H git checkout v`cat /home/git/gitlab/GITLAB_WORKHORSE_VERSION` -b v`cat /home/git/gitlab/GITLAB_WORKHORSE_VERSION`
-sudo -u git -H make
+cd /home/git/gitlab
+
+sudo -u git -H bundle exec rake "gitlab:workhorse:install[/home/git/gitlab-workhorse]" RAILS_ENV=production
 ```
 
 ### 5. Install libs, migrations, etc.

lib/tasks/gitlab/workhorse.rake

@@ -7,8 +7,8 @@ namespace :gitlab do
         abort %(Please specify the directory where you want to install gitlab-workhorse:\n  rake "gitlab:workhorse:install[/home/git/gitlab-workhorse]")
       end
 
-      tag = "v#{ENV['GITLAB_WORKHORSE_VERSION'] || Gitlab::Workhorse.version}"
-      repo = ENV['GITLAB_WORKHORSE_REPO'] || 'https://gitlab.com/gitlab-org/gitlab-workhorse.git'
+      tag = "v#{Gitlab::Workhorse.version}"
+      repo = 'https://gitlab.com/gitlab-org/gitlab-workhorse.git'
 
       checkout_or_clone_tag(tag: tag, repo: repo, target_dir: args.dir)
 

spec/tasks/gitlab/workhorse_rake_spec.rb

@@ -41,32 +41,6 @@ describe 'gitlab:workhorse namespace rake task' do
 
         run_rake_task('gitlab:workhorse:install', clone_path)
       end
-
-      context 'given a specific repo' do
-        before do
-          expect(ENV).to receive(:[]).with('GITLAB_WORKHORSE_REPO').and_return('https://gitlab.com/user1/gitlab-workhorse.git')
-        end
-
-        it 'calls checkout_or_clone_tag with the given repo' do
-          expect_any_instance_of(Object).
-            to receive(:checkout_or_clone_tag).with(tag: tag, repo: 'https://gitlab.com/user1/gitlab-workhorse.git', target_dir: clone_path)
-
-          run_rake_task('gitlab:workhorse:install', clone_path)
-        end
-      end
-
-      context 'given a specific version' do
-        before do
-          allow(ENV).to receive(:[]).with('GITLAB_WORKHORSE_VERSION').and_return('42.42.0')
-        end
-
-        it 'calls checkout_or_clone_tag with the given repo' do
-          expect_any_instance_of(Object).
-            to receive(:checkout_or_clone_tag).with(tag: 'v42.42.0', repo: repo, target_dir: clone_path)
-
-          run_rake_task('gitlab:workhorse:install', clone_path)
-        end
-      end
     end
 
     describe 'gmake/make' do