diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-01-06 15:46:49 +0100 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2017-01-11 11:28:08 -0500 |
commit | c3a2d57415ec32d257c557bd24df8e6aab7fadae (patch) | |
tree | d036a829558c60ec778e45e907c0e8b2790dc3f3 | |
parent | a9cf230d96f024f18ab720178bb7f63bdadd6649 (diff) | |
download | gitlab-ce-c3a2d57415ec32d257c557bd24df8e6aab7fadae.tar.gz |
Fill missing authorized projects rowsfill-authorized-projects
This ensures that the project_authorizations rows exist for all users
for which this data has not yet been populated.
Fixes #26194
-rw-r--r-- | changelogs/unreleased/fill-authorized-projects.yml | 4 | ||||
-rw-r--r-- | db/post_migrate/20170106142508_fill_authorized_projects.rb | 30 | ||||
-rw-r--r-- | spec/migrations/fill_authorized_projects_spec.rb | 18 |
3 files changed, 52 insertions, 0 deletions
diff --git a/changelogs/unreleased/fill-authorized-projects.yml b/changelogs/unreleased/fill-authorized-projects.yml new file mode 100644 index 00000000000..e8e33011a15 --- /dev/null +++ b/changelogs/unreleased/fill-authorized-projects.yml @@ -0,0 +1,4 @@ +--- +title: Fill missing authorized projects rows +merge_request: +author: diff --git a/db/post_migrate/20170106142508_fill_authorized_projects.rb b/db/post_migrate/20170106142508_fill_authorized_projects.rb new file mode 100644 index 00000000000..314c8440c8b --- /dev/null +++ b/db/post_migrate/20170106142508_fill_authorized_projects.rb @@ -0,0 +1,30 @@ +# See http://doc.gitlab.com/ce/development/migration_style_guide.html +# for more information on how to write migrations for GitLab. + +class FillAuthorizedProjects < ActiveRecord::Migration + include Gitlab::Database::MigrationHelpers + + class User < ActiveRecord::Base + self.table_name = 'users' + end + + # Set this constant to true if this migration requires downtime. + DOWNTIME = false + + # We're not inserting any data so we don't need to start a transaction. + disable_ddl_transaction! + + def up + relation = User.select(:id). + where('authorized_projects_populated IS NOT TRUE') + + relation.find_in_batches(batch_size: 1_000) do |rows| + args = rows.map { |row| [row.id] } + + Sidekiq::Client.push_bulk('class' => 'AuthorizedProjectsWorker', 'args' => args) + end + end + + def down + end +end diff --git a/spec/migrations/fill_authorized_projects_spec.rb b/spec/migrations/fill_authorized_projects_spec.rb new file mode 100644 index 00000000000..99dc4195818 --- /dev/null +++ b/spec/migrations/fill_authorized_projects_spec.rb @@ -0,0 +1,18 @@ +require 'spec_helper' +require Rails.root.join('db', 'post_migrate', '20170106142508_fill_authorized_projects.rb') + +describe FillAuthorizedProjects do + describe '#up' do + it 'schedules the jobs in batches' do + user1 = create(:user) + user2 = create(:user) + + expect(Sidekiq::Client).to receive(:push_bulk).with( + 'class' => 'AuthorizedProjectsWorker', + 'args' => [[user1.id], [user2.id]] + ) + + described_class.new.up + end + end +end |