diff options
author | Kamil Trzciński <ayufan@ayufan.eu> | 2017-09-05 09:49:55 +0000 |
---|---|---|
committer | Kamil Trzciński <ayufan@ayufan.eu> | 2017-09-05 09:49:55 +0000 |
commit | 3d61421fb2ed22d64a6b20701d600a38db1458f5 (patch) | |
tree | 09865dc5eb46f29b7ee43d3c6918c814caedbec1 /app/services | |
parent | 89efaf2aa2d65ae41ce5502059d40fa068be945f (diff) | |
parent | 48f017d1e84498eec38d276d94918021a985bfee (diff) | |
download | gitlab-ce-3d61421fb2ed22d64a6b20701d600a38db1458f5.tar.gz |
Merge branch 'fix/sm/35650-remove-createtriggerrequestservice-and-forbid-to-persist-variables-on-ci-triggerrequest' into 'master'
Removes `CreateTriggerRequestService` and add a blocker to prevent saving variables on `Ci::TriggerRequest`
Closes #35650
See merge request !13792
Diffstat (limited to 'app/services')
-rw-r--r-- | app/services/ci/create_trigger_request_service.rb | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/app/services/ci/create_trigger_request_service.rb b/app/services/ci/create_trigger_request_service.rb deleted file mode 100644 index b2aa457bbd5..00000000000 --- a/app/services/ci/create_trigger_request_service.rb +++ /dev/null @@ -1,19 +0,0 @@ -# This class is deprecated because we're closing Ci::TriggerRequest. -# New class is PipelineTriggerService (app/services/ci/pipeline_trigger_service.rb) -# which is integrated with Ci::PipelineVariable instaed of Ci::TriggerRequest. -# We remove this class after we removed v1 and v3 API. This class is still being -# referred by such legacy code. -module Ci - module CreateTriggerRequestService - Result = Struct.new(:trigger_request, :pipeline) - - def self.execute(project, trigger, ref, variables = nil) - trigger_request = trigger.trigger_requests.create(variables: variables) - - pipeline = Ci::CreatePipelineService.new(project, trigger.owner, ref: ref) - .execute(:trigger, ignore_skip_ci: true, trigger_request: trigger_request) - - Result.new(trigger_request, pipeline) - end - end -end |