summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGrzegorz Bizon <grzegorz@gitlab.com>2018-05-09 06:55:20 +0000
committerFilipa Lacerda <filipa@gitlab.com>2018-05-09 10:02:26 +0100
commitae992c6a85018462c775457c3d8af15022a2e17f (patch)
tree821f01e099be45370bd9d74f40d89a4d8ce35ad7
parent7c3c068537d2186dba414f68a78e85fe34b36f78 (diff)
downloadgitlab-ce-ae992c6a85018462c775457c3d8af15022a2e17f.tar.gz
Merge branch 'add-runner-type-to-cluster-app' into 'master'
Set `runner_type` for cluster/application See merge request gitlab-org/gitlab-ce!18832
-rw-r--r--app/models/clusters/applications/runner.rb10
-rw-r--r--spec/models/clusters/applications/runner_spec.rb3
2 files changed, 10 insertions, 3 deletions
diff --git a/app/models/clusters/applications/runner.rb b/app/models/clusters/applications/runner.rb
index 16efe90fa27..b881b4eaf36 100644
--- a/app/models/clusters/applications/runner.rb
+++ b/app/models/clusters/applications/runner.rb
@@ -43,12 +43,20 @@ module Clusters
def create_and_assign_runner
transaction do
- project.runners.create!(name: 'kubernetes-cluster', tag_list: %w(kubernetes cluster)).tap do |runner|
+ project.runners.create!(runner_create_params).tap do |runner|
update!(runner_id: runner.id)
end
end
end
+ def runner_create_params
+ {
+ name: 'kubernetes-cluster',
+ runner_type: :project_type,
+ tag_list: %w(kubernetes cluster)
+ }
+ end
+
def gitlab_url
Gitlab::Routing.url_helpers.root_url(only_path: false)
end
diff --git a/spec/models/clusters/applications/runner_spec.rb b/spec/models/clusters/applications/runner_spec.rb
index 64d995a73c1..5f2c723d483 100644
--- a/spec/models/clusters/applications/runner_spec.rb
+++ b/spec/models/clusters/applications/runner_spec.rb
@@ -74,9 +74,8 @@ describe Clusters::Applications::Runner do
it 'assigns the new runner to runner' do
subject
- gitlab_runner.reload
- expect(gitlab_runner.runner).not_to be_nil
+ expect(gitlab_runner.reload.runner).to be_project_type
end
end