Skip to content
Snippets Groups Projects
Commit a58f5dba authored by Sam Goldstein's avatar Sam Goldstein
Browse files

Update Runner endpoint urgency for new error budget opt in

parent 11586949
No related branches found
No related tags found
1 merge request!85638Update Runner endpoint urgency for new error budget opt in
...@@ -4,6 +4,7 @@ class Admin::RunnerProjectsController < Admin::ApplicationController ...@@ -4,6 +4,7 @@ class Admin::RunnerProjectsController < Admin::ApplicationController
before_action :project, only: [:create] before_action :project, only: [:create]
feature_category :runner feature_category :runner
urgency :low
def create def create
@runner = Ci::Runner.find(params[:runner_project][:runner_id]) @runner = Ci::Runner.find(params[:runner_project][:runner_id])
......
...@@ -9,6 +9,7 @@ class Admin::RunnersController < Admin::ApplicationController ...@@ -9,6 +9,7 @@ class Admin::RunnersController < Admin::ApplicationController
end end
feature_category :runner feature_category :runner
urgency :low
def index def index
end end
......
...@@ -7,6 +7,7 @@ class Groups::RunnersController < Groups::ApplicationController ...@@ -7,6 +7,7 @@ class Groups::RunnersController < Groups::ApplicationController
before_action :runner, only: [:edit, :update, :destroy, :pause, :resume, :show] before_action :runner, only: [:edit, :update, :destroy, :pause, :resume, :show]
feature_category :runner feature_category :runner
urgency :low
def index def index
finder = Ci::RunnersFinder.new(current_user: current_user, params: { group: @group }) finder = Ci::RunnersFinder.new(current_user: current_user, params: { group: @group })
......
...@@ -6,6 +6,7 @@ class Projects::RunnerProjectsController < Projects::ApplicationController ...@@ -6,6 +6,7 @@ class Projects::RunnerProjectsController < Projects::ApplicationController
layout 'project_settings' layout 'project_settings'
feature_category :runner feature_category :runner
urgency :low
def create def create
@runner = Ci::Runner.find(params[:runner_project][:runner_id]) @runner = Ci::Runner.find(params[:runner_project][:runner_id])
......
...@@ -5,6 +5,7 @@ class Projects::RunnersController < Projects::ApplicationController ...@@ -5,6 +5,7 @@ class Projects::RunnersController < Projects::ApplicationController
before_action :runner, only: [:edit, :update, :destroy, :pause, :resume, :show] before_action :runner, only: [:edit, :update, :destroy, :pause, :resume, :show]
feature_category :runner feature_category :runner
urgency :low
def index def index
redirect_to project_settings_ci_cd_path(@project, anchor: 'js-runners-settings') redirect_to project_settings_ci_cd_path(@project, anchor: 'js-runners-settings')
......
...@@ -2,6 +2,7 @@ ...@@ -2,6 +2,7 @@
class RunnerSetupController < ApplicationController class RunnerSetupController < ApplicationController
feature_category :runner feature_category :runner
urgency :low
def platforms def platforms
render json: Gitlab::Ci::RunnerInstructions::OS.merge(Gitlab::Ci::RunnerInstructions::OTHER_ENVIRONMENTS) render json: Gitlab::Ci::RunnerInstructions::OS.merge(Gitlab::Ci::RunnerInstructions::OTHER_ENVIRONMENTS)
......
...@@ -29,7 +29,7 @@ class Runner < ::API::Base ...@@ -29,7 +29,7 @@ class Runner < ::API::Base
mutually_exclusive :maintainer_note, :maintainer_note mutually_exclusive :maintainer_note, :maintainer_note
mutually_exclusive :active, :paused mutually_exclusive :active, :paused
end end
post '/', feature_category: :runner do post '/', urgency: :low, feature_category: :runner do
attributes = attributes_for_keys(%i[description maintainer_note maintenance_note active paused locked run_untagged tag_list access_level maximum_timeout]) attributes = attributes_for_keys(%i[description maintainer_note maintenance_note active paused locked run_untagged tag_list access_level maximum_timeout])
.merge(get_runner_details_from_request) .merge(get_runner_details_from_request)
...@@ -54,7 +54,7 @@ class Runner < ::API::Base ...@@ -54,7 +54,7 @@ class Runner < ::API::Base
params do params do
requires :token, type: String, desc: %q(Runner's authentication token) requires :token, type: String, desc: %q(Runner's authentication token)
end end
delete '/', feature_category: :runner do delete '/', urgency: :low, feature_category: :runner do
authenticate_runner! authenticate_runner!
destroy_conditionally!(current_runner) { ::Ci::Runners::UnregisterRunnerService.new(current_runner, params[:token]).execute } destroy_conditionally!(current_runner) { ::Ci::Runners::UnregisterRunnerService.new(current_runner, params[:token]).execute }
...@@ -66,7 +66,7 @@ class Runner < ::API::Base ...@@ -66,7 +66,7 @@ class Runner < ::API::Base
params do params do
requires :token, type: String, desc: %q(Runner's authentication token) requires :token, type: String, desc: %q(Runner's authentication token)
end end
post '/verify', feature_category: :runner do post '/verify', urgency: :low, feature_category: :runner do
authenticate_runner! authenticate_runner!
status 200 status 200
body "200" body "200"
...@@ -78,7 +78,7 @@ class Runner < ::API::Base ...@@ -78,7 +78,7 @@ class Runner < ::API::Base
params do params do
requires :token, type: String, desc: 'The current authentication token of the runner' requires :token, type: String, desc: 'The current authentication token of the runner'
end end
post '/reset_authentication_token', feature_category: :runner do post '/reset_authentication_token', urgency: :low, feature_category: :runner do
authenticate_runner! authenticate_runner!
current_runner.reset_token! current_runner.reset_token!
......
...@@ -8,6 +8,7 @@ class Runners < ::API::Base ...@@ -8,6 +8,7 @@ class Runners < ::API::Base
before { authenticate! } before { authenticate! }
feature_category :runner feature_category :runner
urgency :low
resource :runners do resource :runners do
desc 'Get runners available for user' do desc 'Get runners available for user' do
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment