diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2017-07-21 07:20:35 +0000 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2017-07-21 07:20:35 +0000 |
commit | 5d90de1cb24160b39c423f8cec9f381f0b1213bb (patch) | |
tree | dbcd015390d6d0e40ac8b7edeef26df4fd4c5b5a | |
parent | eaceee0fb9e23982571c9c2a878d662ba241240e (diff) | |
download | gitlab-ce-5d90de1cb24160b39c423f8cec9f381f0b1213bb.tar.gz |
Update sidekiq_style_guide.md with removing or renaming queues instructionsdz-sidekiq-remove-queue
-rw-r--r-- | doc/development/sidekiq_style_guide.md | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/doc/development/sidekiq_style_guide.md b/doc/development/sidekiq_style_guide.md index e3a20f29a09..734bce963e5 100644 --- a/doc/development/sidekiq_style_guide.md +++ b/doc/development/sidekiq_style_guide.md @@ -36,3 +36,10 @@ slow jobs blocking work (even for different jobs) on the shared queue. Each Sidekiq worker must be tested using RSpec, just like any other class. These tests should be placed in `spec/workers`. + +## Removing or renaming queues + +Try to avoid renaming or removing queues in minor and patch releases. +During online update instance can have pending jobs and removing the queue can +lead to those jobs being stuck forever. If you can't write migration for those +sidekiq jobs, please consider doing rename or remove queue in major release only.
\ No newline at end of file |