summaryrefslogtreecommitdiff
path: root/app/services/ci/runners/unassign_runner_service.rb
diff options
context:
space:
mode:
Diffstat (limited to 'app/services/ci/runners/unassign_runner_service.rb')
-rw-r--r--app/services/ci/runners/unassign_runner_service.rb10
1 files changed, 8 insertions, 2 deletions
diff --git a/app/services/ci/runners/unassign_runner_service.rb b/app/services/ci/runners/unassign_runner_service.rb
index 1e46cf6add8..c40e5e0d44e 100644
--- a/app/services/ci/runners/unassign_runner_service.rb
+++ b/app/services/ci/runners/unassign_runner_service.rb
@@ -13,9 +13,15 @@ module Ci
end
def execute
- return false unless @user.present? && @user.can?(:assign_runner, @runner)
+ unless @user.present? && @user.can?(:assign_runner, @runner)
+ return ServiceResponse.error(message: 'user not allowed to assign runner')
+ end
- @runner_project.destroy
+ if @runner_project.destroy
+ ServiceResponse.success
+ else
+ ServiceResponse.error(message: 'failed to destroy runner project')
+ end
end
private