Commit 395a3f53 authored by Dmytro Zaporozhets (DZ)'s avatar Dmytro Zaporozhets (DZ)

Merge branch 'ci-remove-ff-for-pipeline-async-processing-326217' into 'master'

Remove feature flag for async pipeline processing [RUN ALL RSPEC] [RUN AS-IF-FOSS]

See merge request gitlab-org/gitlab!58901
parents 336a058f 7d1890c8
---
title: Move initial pipeline processing to Sidekiq
merge_request: 58901
author:
type: changed
---
name: ci_async_initial_pipeline_processing
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/57589
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/326217
milestone: '13.11'
type: development
group: group::continuous integration
default_enabled: false
...@@ -8,11 +8,7 @@ module Gitlab ...@@ -8,11 +8,7 @@ module Gitlab
# After pipeline has been successfully created we can start processing it. # After pipeline has been successfully created we can start processing it.
class Process < Chain::Base class Process < Chain::Base
def perform! def perform!
if ::Feature.enabled?(:ci_async_initial_pipeline_processing, project, default_enabled: :yaml) ::Ci::InitialPipelineProcessWorker.perform_async(pipeline.id)
::Ci::InitialPipelineProcessWorker.perform_async(pipeline.id)
else
::Ci::ProcessPipelineService.new(pipeline).execute
end
end end
def break? def break?
......
...@@ -23,21 +23,6 @@ RSpec.describe Gitlab::Ci::Pipeline::Chain::Pipeline::Process do ...@@ -23,21 +23,6 @@ RSpec.describe Gitlab::Ci::Pipeline::Chain::Pipeline::Process do
perform perform
end end
context 'with async processing disabled' do
before do
stub_feature_flags(ci_async_initial_pipeline_processing: false)
end
it 'processes pipeline inline' do
expect(::Ci::ProcessPipelineService)
.to receive(:new)
.with(pipeline)
.and_call_original
perform
end
end
end end
describe '#break?' do describe '#break?' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment