summaryrefslogtreecommitdiff
path: root/spec/migrations/normalize_ldap_extern_uids_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Fix interval in NormalizeLdapExternUids specYorick Peterse2018-08-061-3/+3
| | | | | The background migration interval has been reduced from 5 minutes to 2 minutes, so we have to adjust this migration accordingly.
* Replace 'Sidekiq::Testing.inline!' with 'perform_enqueued_jobs'blackst0ne2018-07-231-2/+2
| | | | | | | | | | `perform_enqueued_jobs` is a Sidekiq method. Using this method violates the Dependency inversion principle[0]. This commit replaces `perform_enqueued_jobs` with ActiveJob's abstract method `perform_enqueued_jobs` in specs. [0]: https://en.wikipedia.org/wiki/Dependency_inversion_principle
* Run background migrations with a minimum intervaldelay-background-migrationsYorick Peterse2018-01-051-3/+3
| | | | | | | | | | | | | | | This adds a minimum interval to BackgroundMigrationWorker, ensuring background migrations of the same class only run once every 5 minutes. This prevents a thundering herd problem where scheduled migrations all run at once due to their delays having been expired (e.g. as the result of a queue being paused for a long time). If a job was recently executed it's rescheduled with a delay that equals the remaining time of the job's lease. This means that if the lease expires in two minutes we only need to wait two minutes, instead of five. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/41624
* Add migration specsMichael Kozono2017-10-071-0/+56