Skip to content
GitLab
Menu
Projects
Groups
Snippets
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
Menu
Open sidebar
heptapod
heptapod
Commits
7faf18017b9c
Commit
3190e812
authored
Mar 07, 2018
by
Shinya Maeda
Browse files
Move update_head_pipeline_for_merge_request queue to pipeline_processing namespace
parent
839d866a7adc
Changes
4
Hide whitespace changes
Inline
Side-by-side
app/workers/all_queues.yml
View file @
7faf1801
...
...
@@ -48,7 +48,6 @@
-
pipeline_default:build_trace_sections
-
pipeline_default:pipeline_metrics
-
pipeline_default:pipeline_notification
-
pipeline_default:update_head_pipeline_for_merge_request
-
pipeline_hooks:build_hooks
-
pipeline_hooks:pipeline_hooks
-
pipeline_processing:build_finished
...
...
@@ -58,6 +57,7 @@
-
pipeline_processing:pipeline_success
-
pipeline_processing:pipeline_update
-
pipeline_processing:stage_update
-
pipeline_processing:update_head_pipeline_for_merge_request
-
repository_check:repository_check_clear
-
repository_check:repository_check_single_repository
...
...
db/post_migrate/20180307012445_migrate_update_head_pipeline_for_merge_request_sidekiq_queue.rb
0 → 100644
View file @
7faf1801
class
MigrateUpdateHeadPipelineForMergeRequestSidekiqQueue
<
ActiveRecord
::
Migration
include
Gitlab
::
Database
::
MigrationHelpers
DOWNTIME
=
false
def
up
sidekiq_queue_migrate
'pipeline_default:update_head_pipeline_for_merge_request'
,
to:
'pipeline_processing:update_head_pipeline_for_merge_request'
end
def
down
sidekiq_queue_migrate
'pipeline_processing:update_head_pipeline_for_merge_request'
,
to:
'pipeline_default:update_head_pipeline_for_merge_request'
end
end
db/schema.rb
View file @
7faf1801
...
...
@@ -11,7 +11,7 @@
#
# It's strongly recommended that you check this file into your version control system.
ActiveRecord
::
Schema
.
define
(
version:
2018030
60740
45
)
do
ActiveRecord
::
Schema
.
define
(
version:
2018030
70124
45
)
do
# These are extensions that must be enabled in order to support this database
enable_extension
"plpgsql"
...
...
spec/migrations/migrate_update_head_pipeline_for_merge_request_sidekiq_queue_spec.rb
0 → 100644
View file @
7faf1801
require
'spec_helper'
require
Rails
.
root
.
join
(
'db'
,
'post_migrate'
,
'20180307012445_migrate_update_head_pipeline_for_merge_request_sidekiq_queue.rb'
)
describe
MigrateUpdateHeadPipelineForMergeRequestSidekiqQueue
,
:sidekiq
,
:redis
do
include
Gitlab
::
Database
::
MigrationHelpers
context
'when there are jobs in the queues'
do
it
'correctly migrates queue when migrating up'
do
Sidekiq
::
Testing
.
disable!
do
stubbed_worker
(
queue:
'pipeline_default:update_head_pipeline_for_merge_request'
).
perform_async
(
'Something'
,
[
1
])
stubbed_worker
(
queue:
'pipeline_processing:update_head_pipeline_for_merge_request'
).
perform_async
(
'Something'
,
[
1
])
described_class
.
new
.
up
expect
(
sidekiq_queue_length
(
'pipeline_default:update_head_pipeline_for_merge_request'
)).
to
eq
0
expect
(
sidekiq_queue_length
(
'pipeline_processing:update_head_pipeline_for_merge_request'
)).
to
eq
2
end
end
it
'does not affect other queues under the same namespace'
do
Sidekiq
::
Testing
.
disable!
do
stubbed_worker
(
queue:
'pipeline_default:build_coverage'
).
perform_async
(
'Something'
,
[
1
])
stubbed_worker
(
queue:
'pipeline_default:build_trace_sections'
).
perform_async
(
'Something'
,
[
1
])
stubbed_worker
(
queue:
'pipeline_default:pipeline_metrics'
).
perform_async
(
'Something'
,
[
1
])
stubbed_worker
(
queue:
'pipeline_default:pipeline_notification'
).
perform_async
(
'Something'
,
[
1
])
described_class
.
new
.
up
expect
(
sidekiq_queue_length
(
'pipeline_default:build_coverage'
)).
to
eq
1
expect
(
sidekiq_queue_length
(
'pipeline_default:build_trace_sections'
)).
to
eq
1
expect
(
sidekiq_queue_length
(
'pipeline_default:pipeline_metrics'
)).
to
eq
1
expect
(
sidekiq_queue_length
(
'pipeline_default:pipeline_notification'
)).
to
eq
1
end
end
it
'correctly migrates queue when migrating down'
do
Sidekiq
::
Testing
.
disable!
do
stubbed_worker
(
queue:
'pipeline_processing:update_head_pipeline_for_merge_request'
).
perform_async
(
'Something'
,
[
1
])
described_class
.
new
.
down
expect
(
sidekiq_queue_length
(
'pipeline_default:update_head_pipeline_for_merge_request'
)).
to
eq
1
expect
(
sidekiq_queue_length
(
'pipeline_processing:update_head_pipeline_for_merge_request'
)).
to
eq
0
end
end
end
context
'when there are no jobs in the queues'
do
it
'does not raise error when migrating up'
do
expect
{
described_class
.
new
.
up
}.
not_to
raise_error
end
it
'does not raise error when migrating down'
do
expect
{
described_class
.
new
.
down
}.
not_to
raise_error
end
end
def
stubbed_worker
(
queue
:)
Class
.
new
do
include
Sidekiq
::
Worker
sidekiq_options
queue:
queue
end
end
end
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment