Merge branch 'mc/feature/remove-ci_drop_bridge_on_downstream_errors-ff' into 'master'

Remove ci_drop_bridge_on_downstream_errors flag

Closes #207948

See merge request gitlab-org/gitlab!27906

app/models/ci/pipeline.rb

@@ -231,14 +231,6 @@ module Ci
         end
       end
 
-      after_transition created: :pending do |pipeline|
-        next if Feature.enabled?(:ci_drop_bridge_on_downstream_errors, pipeline.project, default_enabled: true)
-        next unless pipeline.bridge_triggered?
-        next if pipeline.bridge_waiting?
-
-        pipeline.update_bridge_status!
-      end
-
       after_transition any => [:success, :failed] do |pipeline|
         pipeline.run_after_commit do
           if Feature.enabled?(:ci_pipeline_fixed_notifications)
@@ -761,15 +753,6 @@ module Ci
       end
     end
 
-    def update_bridge_status!
-      raise ArgumentError unless bridge_triggered?
-      raise BridgeStatusError unless source_bridge.active?
-
-      source_bridge.success!
-    rescue => e
-      Gitlab::ErrorTracking.track_exception(e, pipeline_id: id)
-    end
-
     def bridge_triggered?
       source_bridge.present?
     end

app/services/ci/create_cross_project_pipeline_service.rb

@@ -34,8 +34,6 @@ module Ci
         end
 
       downstream_pipeline.tap do |pipeline|
-        next if Feature.disabled?(:ci_drop_bridge_on_downstream_errors, project, default_enabled: true)
-
         update_bridge_status!(@bridge, pipeline)
       end
     end

spec/models/ci/pipeline_spec.rb

@@ -3061,20 +3061,6 @@ describe Ci::Pipeline, :mailer do
             expect(pipeline.source_bridge).to eq bridge
           end
         end
-
-        describe '#update_bridge_status!' do
-          it 'can update bridge status if it is running' do
-            pipeline.update_bridge_status!
-
-            expect(bridge.reload).to be_success
-          end
-
-          it 'can not update bridge status if is not active' do
-            bridge.success!
-
-            expect { pipeline.update_bridge_status! }.not_to change { bridge.status }
-          end
-        end
       end
 
       context 'when an upstream status is a build' do
@@ -3101,16 +3087,6 @@ describe Ci::Pipeline, :mailer do
             expect(pipeline.source_bridge).to be_nil
           end
         end
-
-        describe '#update_bridge_status!' do
-          it 'tracks an ArgumentError and does not update upstream job status' do
-            expect(Gitlab::ErrorTracking)
-              .to receive(:track_exception)
-              .with(instance_of(ArgumentError), pipeline_id: pipeline.id)
-
-            pipeline.update_bridge_status!
-          end
-        end
       end
     end
   end