summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGrzegorz Bizon <grzesiek.bizon@gmail.com>2018-11-23 13:37:32 +0100
committerGrzegorz Bizon <grzesiek.bizon@gmail.com>2018-11-23 13:37:32 +0100
commit3dfbfa4e4f2ce962660dc534ac7a8c670049b506 (patch)
treef912819b85c0d680a1f42913576cb7bb642ea38b
parent1143411ae848fade4050d3d8319ba5ed88a1b1da (diff)
downloadgitlab-ce-3dfbfa4e4f2ce962660dc534ac7a8c670049b506.tar.gz
Adjust batch size for migrating runners token
-rw-r--r--db/post_migrate/20181121111200_schedule_runners_token_encryption.rb4
-rw-r--r--spec/migrations/schedule_runners_token_encryption_spec.rb14
2 files changed, 9 insertions, 9 deletions
diff --git a/db/post_migrate/20181121111200_schedule_runners_token_encryption.rb b/db/post_migrate/20181121111200_schedule_runners_token_encryption.rb
index 3a59217f07a..753e052f7a7 100644
--- a/db/post_migrate/20181121111200_schedule_runners_token_encryption.rb
+++ b/db/post_migrate/20181121111200_schedule_runners_token_encryption.rb
@@ -5,7 +5,7 @@ class ScheduleRunnersTokenEncryption < ActiveRecord::Migration
DOWNTIME = false
BATCH_SIZE = 10000
- RANGE_SIZE = 100
+ RANGE_SIZE = 2000
MIGRATION = 'EncryptRunnersTokens'
MODELS = [
@@ -20,7 +20,7 @@ class ScheduleRunnersTokenEncryption < ActiveRecord::Migration
def up
MODELS.each do |model|
model.each_batch(of: BATCH_SIZE) do |relation, index|
- delay = index * 2.minutes
+ delay = index * 4.minutes
relation.each_batch(of: RANGE_SIZE) do |relation|
range = relation.pluck('MIN(id)', 'MAX(id)').first
diff --git a/spec/migrations/schedule_runners_token_encryption_spec.rb b/spec/migrations/schedule_runners_token_encryption_spec.rb
index a1e4d36de1c..376d2795277 100644
--- a/spec/migrations/schedule_runners_token_encryption_spec.rb
+++ b/spec/migrations/schedule_runners_token_encryption_spec.rb
@@ -24,13 +24,13 @@ describe ScheduleRunnersTokenEncryption, :migration do
Timecop.freeze do
migrate!
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(2.minutes, 'settings', 1, 1)
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(2.minutes, 'namespace', 11, 11)
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'namespace', 12, 12)
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(2.minutes, 'project', 111, 111)
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'project', 114, 114)
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(2.minutes, 'runner', 201, 201)
- expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'runner', 202, 202)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'settings', 1, 1)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'namespace', 11, 11)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(8.minutes, 'namespace', 12, 12)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'project', 111, 111)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(8.minutes, 'project', 114, 114)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(4.minutes, 'runner', 201, 201)
+ expect(described_class::MIGRATION).to be_scheduled_delayed_migration(8.minutes, 'runner', 202, 202)
expect(BackgroundMigrationWorker.jobs.size).to eq 7
end
end