Commit 116aeb7e authored by Sashi Kumar's avatar Sashi Kumar

Rename ForwardDeploymentWorker to DropOlderDeploymentsWorker

This commit introduces a new worker DropOlderDeploymentsWorker replacing
ForwardDeploymentWorker as a part of refactoring introduced
in MR https://gitlab.com/gitlab-org/gitlab/-/merge_requests/41214
parent b6afb1c5
......@@ -73,7 +73,7 @@ class Deployment < ApplicationRecord
next unless deployment.project.forward_deployment_enabled?
deployment.run_after_commit do
Deployments::ForwardDeploymentWorker.perform_async(id)
Deployments::DropOlderDeploymentsWorker.perform_async(id)
end
end
......
......@@ -435,6 +435,14 @@
:weight: 1
:idempotent: true
:tags: []
- :name: deployment:deployments_drop_older_deployments
:feature_category: :continuous_delivery
:has_external_dependencies:
:urgency: :low
:resource_boundary: :unknown
:weight: 3
:idempotent:
:tags: []
- :name: deployment:deployments_execute_hooks
:feature_category: :continuous_delivery
:has_external_dependencies:
......
# frozen_string_literal: true
module Deployments
class DropOlderDeploymentsWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker
queue_namespace :deployment
feature_category :continuous_delivery
def perform(deployment_id)
Deployments::OlderDeploymentsDropService.new(deployment_id).execute
end
end
end
# frozen_string_literal: true
# This worker is deprecated and will be removed in 14.0
# See: https://gitlab.com/gitlab-org/gitlab/-/issues/266381
module Deployments
class FinishedWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker
......
# frozen_string_literal: true
# This worker is deprecated and will be removed in 14.0
# See: https://gitlab.com/gitlab-org/gitlab/-/issues/266381
module Deployments
class ForwardDeploymentWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker
......
# frozen_string_literal: true
# This worker is deprecated and will be removed in 14.0
# See: https://gitlab.com/gitlab-org/gitlab/-/issues/266381
module Deployments
class SuccessWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker
......
......@@ -122,8 +122,8 @@ RSpec.describe Deployment do
deployment.run!
end
it 'executes Deployments::ForwardDeploymentWorker asynchronously' do
expect(Deployments::ForwardDeploymentWorker)
it 'executes Deployments::DropOlderDeploymentsWorker asynchronously' do
expect(Deployments::DropOlderDeploymentsWorker)
.to receive(:perform_async).once.with(deployment.id)
deployment.run!
......
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe Deployments::DropOlderDeploymentsWorker do
subject { described_class.new.perform(deployment&.id) }
describe '#perform' do
let(:deployment) { create(:deployment, :success) }
it 'executes Deployments::OlderDeploymentsDropService' do
expect(Deployments::OlderDeploymentsDropService)
.to receive(:new).with(deployment.id).and_call_original
subject
end
end
end
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