diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-09-20 14:21:15 +0200 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2017-09-20 14:24:14 +0200 |
| commit | b3566a01049cdfbde2a9221319601d8949c12a5a (patch) | |
| tree | 873d695caaaec68afabf8146af5f83f0983be04e /config/sidekiq_queues.yml | |
| parent | a09d032b2a64c7b6652dcd589de2d9bcba7d9613 (diff) | |
| download | gitlab-ce-b3566a01049cdfbde2a9221319601d8949c12a5a.tar.gz | |
Stop using Sidekiq for updating Key#last_used_atremove-use-key-worker
This makes things simpler as no scheduling is involved. Further we
remove the need for running a SELECT + UPDATE just to get the key and
update it, whereas we only need an UPDATE when setting last_used_at
directly in a request.
The added service class takes care of updating Key#last_used_at without
using Sidekiq. Further it makes sure we only try to obtain a Redis lease
if we're confident that we actually need to do so, instead of always
obtaining it. We also make sure to _only_ update last_used_at instead of
also updating updated_at.
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/36663
Diffstat (limited to 'config/sidekiq_queues.yml')
| -rw-r--r-- | config/sidekiq_queues.yml | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/config/sidekiq_queues.yml b/config/sidekiq_queues.yml index 24c001362c6..d169c38a693 100644 --- a/config/sidekiq_queues.yml +++ b/config/sidekiq_queues.yml @@ -38,7 +38,6 @@ - [invalid_gpg_signature_update, 2] - [create_gpg_signature, 2] - [upload_checksum, 1] - - [use_key, 1] - [repository_fork, 1] - [repository_import, 1] - [project_service, 1] |