summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Implement OperationService.UserAddBranch Gitaly RPCgitaly-add-branchAlejandro Rodríguez2017-09-302-7/+47
|
* Populate `Gitlay::Repository`'s `gl_repository` fieldAlejandro Rodríguez2017-09-302-2/+3
|
* Create repositories via GitalyJacob Vosmaer2017-09-294-8/+22
|
* Merge branch ↵Rémy Coutable2017-09-291-5/+27
|\ | | | | | | | | | | | | | | | | '36631-activerecord-statementinvalid-pg-querycanceled-error-canceling-statement-due-to-statement-timeout' into 'master' Insert at most 1,000 rows at once in MR diff background migration Closes #36631 et #37505 See merge request gitlab-org/gitlab-ce!13661
| * Make MR diff background migration less likely to time outSean McGivern2017-09-291-9/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This version does not use transactions, but individual statements. As we have unique constraints on the target tables for the inserts, we can just ignore uniqueness violations there (as long as we always insert the same batch size, in the same order). This means the spec now must use truncation, not a transaction, as the uniqueness violation means that the whole transaction for that spec would be invalid, which isn't what we'd want. In real-world use, this isn't run in a transaction anyway. This commit also wraps unhandled exceptions, for easier finding in Sentry, and logs with a consistent format, for easier searching.
| * Insert at most 1,000 rows at once in MR diff background migrationSean McGivern2017-09-291-3/+9
| | | | | | | | | | | | We were hitting the statement timeout for very large MR diffs. Now we insert at most 1,000 rows to `merge_request_diff_commits` in a single statement, or 100 rows to `merge_request_diff_files`.
* | Make Repository#has_visible_content more efficientJacob Vosmaer (GitLab)2017-09-292-1/+33
| |
* | Fix notes type created from importLin Jen-Shin (godfat)2017-09-295-37/+55
|/
* Migrate Gitlab::Git::Repository#add_tag to Gitalyfeature/migrate-repository-add-tag-to-gitalyAhmad Sherif2017-09-294-30/+70
| | | | Closes gitaly#601
* Merge branch 'feature/migrate-repository-rm-tag-to-gitaly' into 'master'Robert Speicher2017-09-283-1/+45
|\ | | | | | | | | | | | | Migrate Git::Repository#rm_tag to Gitaly Closes gitaly#562 See merge request gitlab-org/gitlab-ce!14388
| * Migrate Git::Repository#rm_tag to GitalyAhmad Sherif2017-09-283-1/+45
| | | | | | | | Closes gitaly#562
* | Merge branch 'hashed-storage-migration-path' into 'master'Douwe Maan2017-09-282-1/+91
|\ \ | | | | | | | | | | | | | | | | | | Hashed storage migration path Closes gitlab-ee#3118 See merge request gitlab-org/gitlab-ce!14067
| * | Add support to migrate existing projects to Hashed Storage asynchashed-storage-migration-pathGabriel Mazetto2017-09-281-0/+85
| | |
| * | [Backported from EE] Readonly flag for ProjectsGabriel Mazetto2017-09-281-1/+6
| | | | | | | | | | | | | | | This is used in EE for the storage migration, and we want to use this in CE as well to be able to migrate projects to hashed_storage.
* | | Support custom attributes on usersMarkus Koller2017-09-283-0/+84
| | |
* | | Merge branch 'dm-api-unauthorized' into 'master'Sean McGivern2017-09-282-16/+38
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Make sure API responds with 401 when invalid authentication info is provided Closes #38438 See merge request gitlab-org/gitlab-ce!14525
| * | | Make sure API responds with 401 when invalid authentication info is provideddm-api-unauthorizedDouwe Maan2017-09-282-16/+38
| | | |
* | | | Handle error when fetching ref for MR with deleted source branch38319-nomethoderror-undefined-method-sha-for-nil-nilclassSean McGivern2017-09-281-2/+7
| |/ / |/| | | | | | | | | | | | | | If the ref doesn't exist, and the source branch is deleted, we can't get it back easily. Previously, we ignored this error by shelling out, so replicate that behaviour.
* | | Merge branch 'zj-repo-gitaly' into 'master'Sean McGivern2017-09-281-2/+12
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | RepositoryExists is always called with #gitaly_migration Closes #38333 See merge request gitlab-org/gitlab-ce!14513
| * | | Rolling back change to n+1 detectionan-repo-gitalyAndrew Newdigate2017-09-271-7/+1
| | | |
| * | | RepositoryExists is always called with #gitaly_migrationZeger-Jan van de Weg2017-09-261-2/+12
| | | |
| * | | Don't enforce gitaly request limits for distinct callsAlejandro Rodríguez2017-09-261-1/+7
| | | |
* | | | refactor emails servicerefactor-servicesJames Lopez2017-09-281-4/+4
| | | |
* | | | refactor users update serviceJames Lopez2017-09-285-5/+5
| | | |
* | | | refactor services to match EE signatureJames Lopez2017-09-285-9/+9
| |_|/ |/| |
* | | Merge branch 'backstage/gb/refactor-pipeline-create-service' into 'master'Kamil Trzciński2017-09-2710-141/+412
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | Refactor a service responsible for creating a pipeline Closes #37563 and #34415 See merge request gitlab-org/gitlab-ce!14482
| * | Add some minor improvements to pipeline creation chainbackstage/gb/refactor-pipeline-create-serviceGrzegorz Bizon2017-09-272-5/+2
| | |
| * | Add specs for builder chain step that skipps pipelinesGrzegorz Bizon2017-09-261-1/+1
| | |
| * | Fix coding style offenses in pipeline chain classesGrzegorz Bizon2017-09-262-4/+2
| | |
| * | Move pipeline builder validation chain to a moduleGrzegorz Bizon2017-09-266-115/+121
| | |
| * | Split pipeline chain builder validation classGrzegorz Bizon2017-09-266-102/+143
| | |
| * | Add class that handles pipeline creation sequenceGrzegorz Bizon2017-09-261-0/+37
| | |
| * | Extract pipeline persistence to a separate chain classGrzegorz Bizon2017-09-261-0/+29
| | |
| * | Fix code style offenses in pipeline create servicesGrzegorz Bizon2017-09-261-1/+1
| | |
| * | Use tag/branch methods to check if pipeline ref existsGrzegorz Bizon2017-09-251-8/+2
| | |
| * | Extract pipeline chain builder classes from serviceGrzegorz Bizon2017-09-253-0/+167
| | |
| * | Move related pipeline class to new pipeline moduleGrzegorz Bizon2017-09-252-141/+143
| | |
* | | Merge branch ↵Rémy Coutable2017-09-271-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '37467-helper-method-from-users-endpoint-overrides-api-helper-method' into 'master' find_user users helper method no longer overrides find_user API helper method. Closes #37467 See merge request gitlab-org/gitlab-ce!14418
| * | | find_user users helper method no longer overrides find_user API helper method.37467-helper-method-from-users-endpoint-overrides-api-helper-methodTiago Botelho2017-09-261-2/+2
| | | |
* | | | Add a spec for Github::Client and revert an `if !` to `unless`38198-fetch-github-api-per-100Rémy Coutable2017-09-261-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Retrieve PR comments only when we know there are anyRémy Coutable2017-09-262-20/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Pull Request resource doesn't include the `comments` field, while the Issue resource does. And since we're looping through all issues anyway, we can freely check if the issue is a PR and has comments and in this case only fetch comments for it. That means if you have 1000 PRs but only 200 with comments, you will do 200 API requests instead of 1000. :notbad: Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Fetch 100 results when calling the GitHub API in Github::ImportRémy Coutable2017-09-261-6/+6
| |_|/ |/| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch 'dm-simple-project-avatar-url' into 'master'Rémy Coutable2017-09-261-5/+6
|\ \ \ | | | | | | | | | | | | | | | | Expose avatar_url when requesting list of projects from API with simple=true See merge request gitlab-org/gitlab-ce!14498
| * | | Expose avatar_url when requesting list of projects from API with simple=truedm-simple-project-avatar-urlDouwe Maan2017-09-261-5/+6
| | | |
* | | | Merge branch 'dm-bitbucket-import-truncated-shas' into 'master'Robert Speicher2017-09-263-10/+32
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug that caused merge requests with diff notes imported from Bitbucket to raise errors Closes #38100 See merge request gitlab-org/gitlab-ce!14438
| * | | | Remove unnecessary commentsDouwe Maan2017-09-251-2/+2
| | | | |
| * | | | Properly compare diff refs and diff positions when shas are truncatedDouwe Maan2017-09-252-8/+25
| | | | |
| * | | | Expand truncated Bitbucket pull request source/target SHA if possibleDouwe Maan2017-09-251-2/+7
| | |/ / | |/| |
* | | | Expose last pipeline details in API response when getting a single commitMehdi Lahmam2017-09-261-0/+1
| |/ / |/| | | | | | | | Closes #35692.
* | | Remove dead `Gitlab::Markdown::Pipeline` classrs-remove-dead-markdown-pipelineRobert Speicher2017-09-251-32/+0
| |/ |/|
View Lorry Controller Status