diff options
author | Robert Speicher <robert@gitlab.com> | 2018-09-14 19:19:09 +0300 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2018-09-14 19:19:09 +0300 |
commit | 1eaaef89d86b0f40b98e2e917bf90f8fdc7d54c4 (patch) | |
tree | 07a0633d7df30c8fd07bd1fd038266f038444b2d /app | |
parent | 99612476e576881e57512a44fbb8e5e040c425b0 (diff) | |
parent | f0e7b5e7a321383aa847790d36b1328a70c2fe2c (diff) |
Merge branch 'fix-rubocop-master-failures' into 'master'
Cleaned up CI runner administration code
See merge request gitlab-org/gitlab-ce!21741
Diffstat (limited to 'app')
-rw-r--r-- | app/controllers/admin/runners_controller.rb | 2 | ||||
-rw-r--r-- | app/finders/admin/runners_finder.rb | 3 | ||||
-rw-r--r-- | app/models/ci/runner.rb | 11 |
3 files changed, 12 insertions, 4 deletions
diff --git a/app/controllers/admin/runners_controller.rb b/app/controllers/admin/runners_controller.rb index 2ac14ecd79c..911603bac17 100644 --- a/app/controllers/admin/runners_controller.rb +++ b/app/controllers/admin/runners_controller.rb @@ -1,14 +1,12 @@ class Admin::RunnersController < Admin::ApplicationController before_action :runner, except: :index - # rubocop: disable CodeReuse/ActiveRecord def index finder = Admin::RunnersFinder.new(params: params) @runners = finder.execute @active_runners_count = Ci::Runner.online.count @sort = finder.sort_key end - # rubocop: enable CodeReuse/ActiveRecord def show assign_builds_and_projects diff --git a/app/finders/admin/runners_finder.rb b/app/finders/admin/runners_finder.rb index 7adee486e33..3c2d7ee7d76 100644 --- a/app/finders/admin/runners_finder.rb +++ b/app/finders/admin/runners_finder.rb @@ -43,8 +43,7 @@ class Admin::RunnersFinder < UnionFinder end def sort! - sort = sort_key == 'contacted_asc' ? { contacted_at: :asc } : { created_at: :desc } - @runners = @runners.order(sort) + @runners = @runners.order_by(sort_key) end def paginate! diff --git a/app/models/ci/runner.rb b/app/models/ci/runner.rb index 45fd15a6211..eabb41c29d7 100644 --- a/app/models/ci/runner.rb +++ b/app/models/ci/runner.rb @@ -72,6 +72,9 @@ module Ci .project_type end + scope :order_contacted_at_asc, -> { order(contacted_at: :asc) } + scope :order_created_at_desc, -> { order(created_at: :desc) } + validate :tag_constraints validates :access_level, presence: true validates :runner_type, presence: true @@ -124,6 +127,14 @@ module Ci ONLINE_CONTACT_TIMEOUT.ago end + def self.order_by(order) + if order == 'contacted_asc' + order_contacted_at_asc + else + order_created_at_desc + end + end + def set_default_values self.token = SecureRandom.hex(15) if self.token.blank? end |