summaryrefslogtreecommitdiff
path: root/spec/workers
Commit message (Collapse)AuthorAgeFilesLines
* Use hashed storage in the specs40744-hashed-storage-specsNick Thomas2018-02-072-2/+2
|
* Reorder async/sync tasks in BuildFinishedWorker to read traces efficientlyfeature/sm/artifacts-traceShinya Maeda2018-02-061-10/+6
|
* Alessio catches: Test fixShinya Maeda2018-02-061-1/+1
|
* Fix create_trace_artifact_worker_spec.rbShinya Maeda2018-02-061-1/+1
|
* Add essential testsShinya Maeda2018-02-062-0/+31
|
* port of 594e6a0a625^..f74c90f68c6Micaël Bergeron2018-02-011-8/+21
|
* update code based on feedback42327-import-from-gitlab-com-fails-destination-already-exists-and-is-not-an-empty-directory-errorJames Lopez2018-01-261-2/+2
|
* add specJames Lopez2018-01-261-0/+13
|
* Initial work to add notification reason to emailsMario de la Ossa2018-01-162-4/+7
| | | | | | | | | | | Adds `#build_notification_recipients` to `NotificationRecipientService` that returns the `NotificationRecipient` objects in order to be able to access the new attribute `reason`. This new attribute is used in the different notifier methods in order to add the reason as a header: `X-GitLab-NotificationReason`. Only the reason with the most priority gets sent.
* Fork now protects default branch on completion19493-fork-does-not-protect-default-branchTiago Botelho2018-01-151-0/+8
|
* Fix CheckGcpProjectBillingWorker specMatija Čupić2018-01-111-1/+1
|
* Merge branch 'jej/backport-authorized-keys-to-ce' into 'master'Douwe Maan2018-01-091-0/+12
|\ | | | | | | | | | | | | Backport authorized_keys Closes gitlab-ee#3953 See merge request gitlab-org/gitlab-ce!16014
| * Backport authorized_keys_enabled defaults to true'Michael Kozono2018-01-081-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally from branch 'fix-authorized-keys-enabled-default-2738' via merge request https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/2240 Removed background migrations which were intended to fix state after using Gitlab without a default having been set Squashed commits: Locally, if Spring was not restarted, `current_application_settings` was still cached, which prevented the migration from editing the file. This will also ensure that any app server somehow hitting old cache data will properly default this setting regardless. Retroactively fix migration This allows us to identify customers who ran the broken migration. Their `authorized_keys_enabled` column does not have a default at this point. We will fix the column after we fix the `authorized_keys` file. Fix authorized_keys file if needed Add default to authorized_keys_enabled setting Reminder: The original migration was fixed retroactively a few commits ago, so people who did not ever run GitLab 9.3.0 already have a column that defaults to true and disallows nulls. I have tested on PostgreSQL and MySQL that it is safe to run this migration regardless. Affected customers who did run 9.3.0 are the ones who need this migration to fix the authorized_keys_enabled column. The reason for the retroactive fix plus this migration is that it allows us to run a migration in between to fix the authorized_keys file only for those who ran 9.3.0. Tweaks to address feedback Extract work into background migration Move batch-add-logic to background migration Do the work synchronously to avoid multiple workers attempting to add batches of keys at the same time. Also, make the delete portion wait until after adding is done. Do read and delete work in background migration Fix Rubocop offenses Add changelog entry Inform the user of actions taken or not taken Prevent unnecessary `select`s and `remove_key`s Add logs for action taken Fix optimization Reuse `Gitlab::ShellAdapter` Guarantee the earliest key Fix migration spec for MySQL
* | Merge branch 'delay-background-migrations' into 'master'Rémy Coutable2018-01-081-2/+21
|\ \ | | | | | | | | | | | | | | | | | | Run background migrations with a minimum interval Closes #41624 See merge request gitlab-org/gitlab-ce!16230
| * | Run background migrations with a minimum intervaldelay-background-migrationsYorick Peterse2018-01-051-2/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge branch '3968-protected-branch-is-not-set-for-default-branch-on-import' ↵Douwe Maan2018-01-081-0/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Protected branch is now created for default branch on import Closes #3968 See merge request gitlab-org/gitlab-ce!16198
| * | | Protected branch is now created for default branch on import3968-protected-branch-is-not-set-for-default-branch-on-importTiago Botelho2018-01-061-0/+1
| |/ /
* | | Fix check_gcp_project_billing_worker_spec.rbKamil Trzciński2018-01-071-1/+1
| | |
* | | Refactor GCP session token exchange schemeMatija Čupić2018-01-071-2/+2
| | |
* | | Separate let and subject line in specMatija Čupić2018-01-071-0/+1
| | |
* | | Use token from redis in gcp project billing workerMatija Čupić2018-01-061-14/+32
| | |
* | | Merge branch 'master' into ↵Matija Čupić2018-01-051-0/+27
|\ \ \ | | |/ | |/| | | | 39957-redirect-to-gpc-page-if-users-try-to-create-a-cluster-but-the-account-is-not-enabled
| * | Backport 'Rebase' feature from EE to CEjprovazn-rebaseJan Provaznik2018-01-051-0/+27
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | When a project uses fast-forward merging strategy user has to rebase MRs to target branch before it can be merged. Now user can do rebase in UI by clicking 'Rebase' button instead of doing rebase locally. This feature was already present in EE, this is only backport of the feature to CE. Couple of changes: * removed rebase license check * renamed migration (changed timestamp) Closes #40301
* | Return list of billing enabled projectsMatija Čupić2018-01-051-2/+2
| |
* | Merge branch 'master' into ↵Matija Čupić2018-01-051-1/+1
|\ \ | |/ | | | | 39957-redirect-to-gpc-page-if-users-try-to-create-a-cluster-but-the-account-is-not-enabled
| * Replace '.team << [user, role]' with 'add_role(user)' in specs36782-replace-team-user-role-with-add_role-user-in-specsblackst0ne2017-12-221-1/+1
| |
* | Merge branch 'master' into ↵Matija Čupić2017-12-2113-56/+141
|\ \ | |/ | | | | 39957-redirect-to-gpc-page-if-users-try-to-create-a-cluster-but-the-account-is-not-enabled
| * Merge branch ↵Douwe Maan2017-12-203-31/+61
| |\ | | | | | | | | | | | | | | | | | | | | | | | | '39246-fork-and-import-jobs-should-only-be-marked-as-failed-when-the-number-of-retries-was-exhausted' into 'master' Fork and Import jobs only get marked as failed when the number of Sidekiq retries were exhausted Closes #39246 See merge request gitlab-org/gitlab-ce!15844
| | * Fork and Import jobs only get marked as failed when the number of Sidekiq ↵39246-fork-and-import-jobs-should-only-be-marked-as-failed-when-the-number-of-retries-was-exhaustedTiago Botelho2017-12-153-31/+61
| | | | | | | | | | | | retries were exhausted
| * | Merge branch 'sh-add-schedule-pipeline-run-now' into 'master'Kamil Trzciński2017-12-181-0/+39
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | Add button to run scheduled pipeline immediately Closes #38741 See merge request gitlab-org/gitlab-ce!15700
| | * | Address some comments with running a pipeline scheduleStan Hu2017-12-121-6/+9
| | | |
| | * | Add button to run scheduled pipeline immediatelyStan Hu2017-12-121-0/+36
| | |/ | | | | | | | | | Closes #38741
| * | Merge branch 'dm-dedicated-sidekiq-queues' into 'master'Sean McGivern2017-12-147-22/+36
| |\ \ | | | | | | | | | | | | | | | | Use a dedicated queue for each Sidekiq worker See merge request gitlab-org/gitlab-ce!15882
| | * | Add rubocops to ensure Sidekiq workers include ApplicationWorker and don't ↵Douwe Maan2017-12-131-4/+0
| | | | | | | | | | | | | | | | manually set their queue
| | * | Use a dedicated queue for each workerDouwe Maan2017-12-127-18/+36
| | |/
| * | Prevent worker that updates merge requests head pipeline from failing jobsFelipe Artur2017-12-132-3/+5
| |/
* | Expand controller test suite matrixMatija Čupić2017-12-161-3/+3
| |
* | Add lease to CheckGcpProjectBillingWorkerMatija Čupić2017-12-161-9/+27
| |
* | Isolate CheckGcpProjectBillingWorker specreturnsMatija Čupić2017-12-161-4/+5
| |
* | Add CheckGcpProjectBillingWorkerMatija Čupić2017-12-161-0/+23
|/
* Ensure pippeline corresponds with the sha of an MRJarka Kadlecova2017-12-051-0/+38
|
* Merge branch 'zj-multiple-artifacts' into 'master'Grzegorz Bizon2017-12-051-13/+9
|\ | | | | | | | | Multiple artifacts See merge request gitlab-org/gitlab-ce!14367
| * Use `job_artifacts_archive|metadata`Kamil Trzcinski2017-12-031-3/+3
| |
| * Revert "Rename `job_archive|metadata` to `artifacts_archive|metadata`"Kamil Trzcinski2017-12-031-3/+3
| | | | | | | | This reverts commit 714082e65304ae2ec5c5400c59a68ab63e724aa9.
| * Rename `job_archive|metadata` to `artifacts_archive|metadata`Kamil Trzcinski2017-12-031-3/+3
| |
| * Rename Artifact to JobArtifact, split metadata outZeger-Jan van de Weg2017-12-031-13/+9
| | | | | | | | | | | | | | | | | | Two things at ones, as there was no clean way to seperate the commit and give me feedback from the tests. But the model Artifact is now JobArtifact, and the table does not have a type anymore, but the metadata is now its own model: Ci::JobArtifactMetadata.
* | Consistently schedule Sidekiq jobsdm-application-workerDouwe Maan2017-12-053-32/+31
| |
* | Add ApplicationWorker and make every worker include itDouwe Maan2017-12-059-54/+62
| |
* | Fix the fork project functionality for projects with hashed storageNick Thomas2017-12-041-49/+28
|/ | | | Note the dependency on gitlab-shell v5.10.0
* Merge branch 'master' into backstage/gb/build-pipeline-in-a-separate-classGrzegorz Bizon2017-11-3029-91/+1138
|\ | | | | | | * master: (1794 commits)