1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe BulkImports::EntityWorker, feature_category: :importers do
subject(:worker) { described_class.new }
let_it_be(:entity) { create(:bulk_import_entity, :started) }
let_it_be_with_reload(:pipeline_tracker) do
create(
:bulk_import_tracker,
entity: entity,
pipeline_name: 'Stage0::Pipeline',
stage: 0
)
end
let_it_be_with_reload(:pipeline_tracker_2) do
create(
:bulk_import_tracker,
entity: entity,
pipeline_name: 'Stage1::Pipeline',
stage: 1
)
end
include_examples 'an idempotent worker' do
let(:job_args) { entity.id }
before do
allow(described_class).to receive(:perform_in)
allow(BulkImports::PipelineWorker).to receive(:perform_async)
end
it 'enqueues the pipeline workers of the first stage and then re-enqueues itself' do
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
expect(logger).to receive(:info).with(hash_including('message' => 'Stage starting', 'entity_stage' => 0))
expect(logger).to receive(:info).with(hash_including('message' => 'Stage running', 'entity_stage' => 0))
end
expect(BulkImports::PipelineWorker)
.to receive(:perform_async)
.with(pipeline_tracker.id, pipeline_tracker.stage, entity.id)
expect(described_class).to receive(:perform_in).twice.with(described_class::PERFORM_DELAY, entity.id)
expect { subject }.to change { pipeline_tracker.reload.status_name }.from(:created).to(:enqueued)
end
end
it 'has the option to reschedule once if deduplicated' do
expect(described_class.get_deduplication_options).to include({ if_deduplicated: :reschedule_once })
end
context 'when pipeline workers from a stage are running' do
before do
pipeline_tracker.enqueue!
end
it 'does not enqueue the pipeline workers from the next stage and re-enqueues itself' do
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
expect(logger).to receive(:info).with(hash_including('message' => 'Stage running', 'entity_stage' => 0))
end
expect(BulkImports::PipelineWorker).not_to receive(:perform_async)
expect(described_class).to receive(:perform_in).with(described_class::PERFORM_DELAY, entity.id)
worker.perform(entity.id)
end
end
context 'when there are no pipeline workers from the previous stage running' do
before do
pipeline_tracker.fail_op!
end
it 'enqueues the pipeline workers from the next stage and re-enqueues itself' do
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
expect(logger).to receive(:info).with(hash_including('message' => 'Stage starting', 'entity_stage' => 1))
end
expect(BulkImports::PipelineWorker)
.to receive(:perform_async)
.with(
pipeline_tracker_2.id,
pipeline_tracker_2.stage,
entity.id
)
expect(described_class).to receive(:perform_in).with(described_class::PERFORM_DELAY, entity.id)
worker.perform(entity.id)
end
end
context 'when there are no next stage to run' do
before do
pipeline_tracker.fail_op!
pipeline_tracker_2.fail_op!
end
it 'does not enqueue any pipeline worker and re-enqueues itself' do
expect(BulkImports::PipelineWorker).not_to receive(:perform_async)
expect(described_class).to receive(:perform_in).with(described_class::PERFORM_DELAY, entity.id)
worker.perform(entity.id)
end
end
context 'when entity status is not started' do
let(:entity) { create(:bulk_import_entity, :finished) }
it 'does not re-enqueues itself' do
expect(described_class).not_to receive(:perform_in)
worker.perform(entity.id)
end
end
it 'logs and tracks the raised exceptions' do
exception = StandardError.new('Error!')
expect(BulkImports::PipelineWorker)
.to receive(:perform_async)
.and_raise(exception)
expect(Gitlab::ErrorTracking)
.to receive(:track_exception)
.with(
exception,
hash_including(
bulk_import_entity_id: entity.id,
bulk_import_id: entity.bulk_import_id,
bulk_import_entity_type: entity.source_type,
source_full_path: entity.source_full_path,
source_version: entity.bulk_import.source_version_info.to_s,
importer: 'gitlab_migration'
)
)
worker.perform(entity.id)
expect(entity.reload.failed?).to eq(true)
end
end
|