diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-11-07 06:10:53 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-11-07 06:10:53 +0300 |
commit | 2ce8e7fcf32b18db57a5547fda35044e55cdc1eb (patch) | |
tree | 64bbbdd27d96d158ac1642d496aeaa199dd16dd1 /app/services | |
parent | 611897b987d439b0d736eb87415b8ca32bdaa282 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'app/services')
-rw-r--r-- | app/services/ci/cancel_pipeline_service.rb | 13 | ||||
-rw-r--r-- | app/services/packages/update_tags_service.rb | 3 |
2 files changed, 10 insertions, 6 deletions
diff --git a/app/services/ci/cancel_pipeline_service.rb b/app/services/ci/cancel_pipeline_service.rb index 52d38520fc3..38053b13921 100644 --- a/app/services/ci/cancel_pipeline_service.rb +++ b/app/services/ci/cancel_pipeline_service.rb @@ -24,11 +24,7 @@ module Ci end def execute - unless can?(current_user, :cancel_pipeline, pipeline) - return ServiceResponse.error( - message: 'Insufficient permissions to cancel the pipeline', - reason: :insufficient_permissions) - end + return permission_error_response unless can?(current_user, :cancel_pipeline, pipeline) force_execute end @@ -103,6 +99,13 @@ module Ci job.cancel end + def permission_error_response + ServiceResponse.error( + message: 'Insufficient permissions to cancel the pipeline', + reason: :insufficient_permissions + ) + end + # For parent child-pipelines only (not multi-project) def cancel_children pipeline.all_child_pipelines.each do |child_pipeline| diff --git a/app/services/packages/update_tags_service.rb b/app/services/packages/update_tags_service.rb index cf1acc6ee19..014d5501b76 100644 --- a/app/services/packages/update_tags_service.rb +++ b/app/services/packages/update_tags_service.rb @@ -32,7 +32,8 @@ module Packages package_id: @package.id, name: tag, created_at: now, - updated_at: now + updated_at: now, + project_id: @package.project_id } end end |