summaryrefslogtreecommitdiff
path: root/lib/gitlab
Commit message (Collapse)AuthorAgeFilesLines
* Replace old GH importer with the parallel importergithub-importer-refactorYorick Peterse2017-11-071-1/+1
|
* Rewrite the GitHub importer from scratchYorick Peterse2017-11-0753-118/+2445
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this MR there were two GitHub related importers: * Github::Import: the main importer used for GitHub projects * Gitlab::GithubImport: importer that's somewhat confusingly used for importing Gitea projects (apparently they have a compatible API) This MR renames the Gitea importer to Gitlab::LegacyGithubImport and introduces a new GitHub importer in the Gitlab::GithubImport namespace. This new GitHub importer uses Sidekiq for importing multiple resources in parallel, though it also has the ability to import data sequentially should this be necessary. The new code is spread across the following directories: * lib/gitlab/github_import: this directory contains most of the importer code such as the classes used for importing resources. * app/workers/gitlab/github_import: this directory contains the Sidekiq workers, most of which simply use the code from the directory above. * app/workers/concerns/gitlab/github_import: this directory provides a few modules that are included in every GitHub importer worker. == Stages The import work is divided into separate stages, with each stage importing a specific set of data. Stages will schedule the work that needs to be performed, followed by scheduling a job for the "AdvanceStageWorker" worker. This worker will periodically check if all work is completed and schedule the next stage if this is the case. If work is not yet completed this worker will reschedule itself. Using this approach we don't have to block threads by calling `sleep()`, as doing so for large projects could block the thread from doing any work for many hours. == Retrying Work Workers will reschedule themselves whenever necessary. For example, hitting the GitHub API's rate limit will result in jobs rescheduling themselves. These jobs are not processed until the rate limit has been reset. == User Lookups Part of the importing process involves looking up user details in the GitHub API so we can map them to GitLab users. The old importer used an in-memory cache, but this obviously doesn't work when the work is spread across different threads. The new importer uses a Redis cache and makes sure we only perform API/database calls if absolutely necessary. Frequently used keys are refreshed, and lookup misses are also cached; removing the need for performing API/database calls if we know we don't have the data we're looking for. == Performance & Models The new importer in various places uses raw INSERT statements (as generated by `Gitlab::Database.bulk_insert`) instead of using Rails models. This allows us to bypass any validations and callbacks, drastically reducing the number of SQL queries and Gitaly RPC calls necessary to import projects. To ensure the code produces valid data the corresponding tests check if the produced rows are valid according to the model validation rules.
* Add returning IDs to Gitlab::Database.bulk_insertYorick Peterse2017-11-071-2/+23
| | | | | | | This adds the keyword argument "return_ids" to Gitlab::Database.bulk_insert. When set to `true` (and PostgreSQL is used) this method will return an Array of the IDs of the inserted rows, otherwise it will return an empty Array.
* Improve GitLab Import rake task to work with Hashed Storage and SubgroupsJames Lopez2017-11-073-97/+143
|
* Merge branch '38464-k8s-apps' of https://gitlab.com/gitlab-org/gitlab-ce ↵Shinya Maeda2017-11-081-1/+1
|\ | | | | | | into 38464-k8s-apps
| * Review Gitlab::Kubernetes::NamespaceAlessio Caiazza2017-11-071-1/+1
| |
* | Merge branch 'master' into 38464-k8s-appsShinya Maeda2017-11-083-13/+21
|\ \ | |/ |/|
| * Merge branch 'fix/import-export-arguments' into 'master'Douwe Maan2017-11-071-1/+1
| |\ | | | | | | | | | | | | | | | | | | Fix arguments error on Import/Export fetch_ref method Closes #39541 See merge request gitlab-org/gitlab-ce!15241
| | * fix specsfix/import-export-argumentsJames Lopez2017-11-071-1/+1
| | |
| | * Fix arguments error on Import/Export fetch_ref methodJames Lopez2017-11-071-1/+1
| | | | | | | | | | | | Added unit test and updated integration spec to test for this as well.
| * | Merge branch '39831-warning-icon-in-jobs-dropdown-has-incorrect-style' into ↵Clement Ho2017-11-071-1/+1
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Resolve "Warning icon in jobs dropdown has incorrect style" Closes #39831 See merge request gitlab-org/gitlab-ce!15221
| | * | Fixed Status Warning Icon39831-warning-icon-in-jobs-dropdown-has-incorrect-styleTim Zallmann2017-11-061-1/+1
| | | |
| * | | Merge branch 'fix-mysql-grant-check' into 'master'Rémy Coutable2017-11-071-11/+19
| |\ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | Fix TRIGGER checks for MySQL Closes #38372 See merge request gitlab-org/gitlab-ce!15226
| | * | Fix TRIGGER checks for MySQLfix-mysql-grant-checkYorick Peterse2017-11-061-11/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ensures we can check if the user has TRIGGER permissions without querying restricted tables. Thanks to Steve Norman (https://gitlab.com/stevenorman) for helping out with this merge request. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/38372
* | | | Add Helm InstallCommandAlessio Caiazza2017-11-071-35/+42
| | | |
* | | | Add k8s namespace implementation and testsAlessio Caiazza2017-11-072-25/+35
| | | |
* | | | Merge branch '38464-k8s-apps' of https://gitlab.com/gitlab-org/gitlab-ce ↵Shinya Maeda2017-11-072-4/+0
|\ \ \ \ | | | | | | | | | | | | | | | into 38464-k8s-apps
| * | | | Remove Project#clusters exportAlessio Caiazza2017-11-072-4/+0
| | | | |
* | | | | Merge branch 'master' into 38464-k8s-appsShinya Maeda2017-11-072-1/+3
|\ \ \ \ \ | |/ / / / |/| / / / | |/ / /
| * | | Merge branch 'feature/custom-attributes-on-projects-and-groups' into 'master'Rémy Coutable2017-11-072-1/+3
| |\ \ \ | | |/ / | |/| | | | | | | | | | Support custom attributes on groups and projects See merge request gitlab-org/gitlab-ce!14593
| | * | Support custom attributes on projectsMarkus Koller2017-11-062-1/+3
| | | |
* | | | Merge branch 'master' into 38464-k8s-appsEric Eastwood2017-11-0629-381/+763
|\ \ \ \ | |/ / / | | | | | | | | | | | | Conflicts: db/schema.rb
| * | | Merge branch 'refactor-clusters' into 'master'Grzegorz Bizon2017-11-063-18/+5
| |\ \ \ | | | | | | | | | | | | | | | | | | | | Refactor Clusters to be consisted from GcpProvider and KubernetesPlatform See merge request gitlab-org/gitlab-ce!14879
| * \ \ \ Merge branch 'pawel/metrics-to-prometheus-33643' into 'master'Grzegorz Bizon2017-11-0621-350/+684
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add all InfluxDB metrics to prometheus See merge request gitlab-org/gitlab-ce!13891
| | * | | | Add missing mutex guard to method call metricspawel/metrics-to-prometheus-33643Pawel Chojnacki2017-11-031-12/+21
| | | | | |
| | * | | | Make subscriber tests pass, after refactoring metricsPawel Chojnacki2017-11-022-4/+4
| | | | | |
| | * | | | Use Mutex to guard metrics creation in transaction. Switch action view to ↵Pawel Chojnacki2017-11-022-28/+49
| | | | | | | | | | | | | | | | | | | | | | | | threadsafe instance variables
| | * | | | move metrics for ActiveRecord, RailsCache and queue duration to instance ↵Pawel Chojnacki2017-11-023-38/+40
| | | | | | | | | | | | | | | | | | | | | | | | variables
| | * | | | rename BackgroundMigration to SidekiqMigrationPawel Chojnacki2017-11-023-2/+2
| | | | | |
| | * | | | Fix sidekiq middleware testsPawel Chojnacki2017-11-023-6/+5
| | | | | |
| | * | | | Remove unnecessary namespace usePawel Chojnacki2017-11-022-7/+7
| | | | | |
| | * | | | Tests for Web transaction and remove simple transactonPawel Chojnacki2017-11-025-264/+33
| | | | | |
| | * | | | Web/Sidekiq transaction splitPawel Chojnacki2017-11-023-1/+262
| | | | | |
| | * | | | Move labels to be initialized in constructorPawel Chojnacki2017-11-021-5/+2
| | | | | |
| | * | | | Avoid using Send (but is it better?) + small rubocop cleanupPawel Chojnacki2017-11-022-10/+8
| | | | | |
| | * | | | Cleanup metrics names and removing unnecessary cache read totalPawel Chojnacki2017-11-021-15/+5
| | | | | |
| | * | | | Adjust test to changed metrics methods.Pawel Chojnacki2017-11-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | + remove deprecated test suite
| | * | | | Convert rails_queue_duration to metric_rails_queue_duration_secondsPawel Chojnacki2017-11-021-1/+12
| | | | | |
| | * | | | Add info about prometheus bucketsPawel Chojnacki2017-11-022-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | + fix cpu time
| | * | | | Guard metrics creation with a mutexPawel Chojnacki2017-11-021-7/+23
| | | | | |
| | * | | | Move labels tests from Metrics rack spec to Transaction specPawel Chojnacki2017-11-022-19/+19
| | | | | |
| | * | | | Fix rspec errors, and add more tests to MethodCall and ActionViewPawel Chojnacki2017-11-022-5/+21
| | | | | |
| | * | | | Fix Active record and transaction specsPawel Chojnacki2017-11-021-1/+1
| | | | | |
| | * | | | Fix rubocop warningsPawel Chojnacki2017-11-024-4/+3
| | | | | |
| | * | | | Differentiatie between class and instance methodsPawel Chojnacki2017-11-021-4/+6
| | | | | |
| | * | | | Split call name to module and method namePawel Chojnacki2017-11-024-10/+27
| | | | | |
| | * | | | More parsable labels in method performance measurementsPawel Chojnacki2017-11-022-7/+7
| | | | | |
| | * | | | Make transaction labels more readablePawel Chojnacki2017-11-023-34/+48
| | | | | |
| | * | | | Fix measure codde to work with secondsPawel Chojnacki2017-11-021-2/+2
| | | | | |
| | * | | | Rails cache metrics name alignmentPawel Chojnacki2017-11-021-8/+8
| | | | | |
View Lorry Controller Status