Merge branch '216531-remove-unused-classes-on-initial-ci-ref-implementation' into 'master'

Remove unused classes on initial Ci::Ref implementation

Closes #216531

See merge request gitlab-org/gitlab!41077

app/services/ci/update_ci_ref_status_service.rb

-# frozen_string_literal: true
-
-# NOTE: This class is unused and to be removed in 13.1~
-module Ci
-  class UpdateCiRefStatusService
-    include Gitlab::OptimisticLocking
-
-    attr_reader :pipeline
-
-    def initialize(pipeline)
-      @pipeline = pipeline
-    end
-
-    def call
-      save.tap { |success| after_save if success }
-    end
-
-    private
-
-    def save
-      might_insert = ref.new_record?
-
-      begin
-        retry_optimistic_lock(ref) do
-          next false if ref.persisted? &&
-            (ref.last_updated_by_pipeline_id || 0) > pipeline.id
-
-          ref.update(status: next_status(ref.status, pipeline.status),
-                     last_updated_by_pipeline: pipeline)
-        end
-      rescue ActiveRecord::RecordNotUnique
-        if might_insert
-          @ref = pipeline.reset.ref_status
-          might_insert = false
-          retry
-        else
-          raise
-        end
-      end
-    end
-
-    def next_status(ref_status, pipeline_status)
-      if ref_status == 'failed' && pipeline_status == 'success'
-        'fixed'
-      else
-        pipeline_status
-      end
-    end
-
-    def after_save
-      enqueue_pipeline_notification
-    end
-
-    def enqueue_pipeline_notification
-      PipelineNotificationWorker.perform_async(pipeline.id, ref_status: ref.status)
-    end
-
-    def ref
-      @ref ||= pipeline.ref_status || build_ref
-    end
-
-    def build_ref
-      Ci::Ref.new(ref: pipeline.ref, project: pipeline.project, tag: pipeline.tag)
-    end
-  end
-end

app/workers/all_queues.yml

@@ -1019,14 +1019,6 @@
   :weight: 3
   :idempotent:
   :tags: []
-- :name: pipeline_default:pipeline_update_ci_ref_status
-  :feature_category: :continuous_integration
-  :has_external_dependencies:
-  :urgency: :high
-  :resource_boundary: :cpu
-  :weight: 3
-  :idempotent:
-  :tags: []
 - :name: pipeline_hooks:build_hooks
   :feature_category: :continuous_integration
   :has_external_dependencies:

app/workers/pipeline_update_ci_ref_status_worker.rb

-# frozen_string_literal: true
-
-# NOTE: This class is unused and to be removed in 13.1~
-class PipelineUpdateCiRefStatusWorker # rubocop:disable Scalability/IdempotentWorker
-  include ApplicationWorker
-  include PipelineQueue
-
-  urgency :high
-  worker_resource_boundary :cpu
-
-  def perform(pipeline_id)
-    pipeline = Ci::Pipeline.find_by_id(pipeline_id)
-
-    return unless pipeline
-
-    Ci::UpdateCiRefStatusService.new(pipeline).call
-  end
-end

changelogs/unreleased/233653-replace-bootstrap-alerts-in-ee-app-views-admin-licenses-_repeat_tr.yml

+---
+title: Removes unused classes on initial Ci::Ref implementation
+merge_request: 41077
+author: Jacopo Beschi @jacopo-beschi
+type: removed

spec/workers/pipeline_update_ci_ref_status_worker_service_spec.rb

-# frozen_string_literal: true
-
-require 'spec_helper'
-
-# NOTE: This class is unused and to be removed in 13.1~
-RSpec.describe PipelineUpdateCiRefStatusWorker do
-  let(:worker) { described_class.new }
-  let(:pipeline) { create(:ci_pipeline) }
-
-  describe '#perform' do
-    it 'updates the ci_ref status' do
-      expect(Ci::UpdateCiRefStatusService).to receive(:new)
-        .with(pipeline)
-        .and_return(double(call: true))
-
-      worker.perform(pipeline.id)
-    end
-  end
-end