summaryrefslogtreecommitdiff
path: root/spec/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch '46600-fix-gitlab-revision-when-not-in-git-repo' into 'master'Rémy Coutable2018-05-251-0/+60
|\ | | | | | | | | | | | | Resolve "gitlab 10.8 assumes that installation is a git working copy." Closes #46600 See merge request gitlab-org/gitlab-ce!19125
| * Replace Gitlab::REVISION with Gitlab.revision and handle installations ↵46600-fix-gitlab-revision-when-not-in-git-repoRémy Coutable2018-05-241-0/+60
| | | | | | | | | | | | without a .git directory Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'migrate-restore-repo-to-gitaly' into 'master'Douwe Maan2018-05-241-26/+3
|\ \ | | | | | | | | | | | | | | | | | | Use gitaly repository_service.CreateRepositoryFromBundle to restore (rake task) Closes charts/gitlab#390 See merge request gitlab-org/gitlab-ce!18644
| * | Fix failing specsAhmad Hassan2018-05-161-26/+3
| | |
* | | Fix backup creation and restore for specific Rake tasksStan Hu2018-05-242-6/+7
| |/ |/|
* | Simplify Gitlab::CurrentSettings now that the logic is in CacheableAttributesRémy Coutable2018-05-231-50/+51
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'issue-25256' into 'master'Douwe Maan2018-05-232-2/+30
|\ \ | | | | | | | | | | | | skip email trim when email is creating new issue See merge request gitlab-org/gitlab-ce!18865
| * | skip email trim when email is creating new issueCameron Crockett2018-05-222-2/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates from MR discussion 1. Added test for ReplyParser 2. Changed param to trim_reply with default set as true Removed keyword param in favor of normal options param updates for MR discussion Resolutions for code review comments more code review fixes
* | | Resolve "Deprecate Gemnasium project service"Olivier Gonzalez2018-05-231-20/+0
|/ /
* | Merge branch 'ab-42194-keyset-pagination' into 'master'Yorick Peterse2018-05-221-8/+204
|\ \ | | | | | | | | | | | | | | | | | | API: Keyset pagination support Closes #45756 See merge request gitlab-org/gitlab-ce!18584
| * | Add keyset pagination for API calls.Andreas Brandl2018-05-211-8/+204
| | | | | | | | | | | | Closes #45756.
* | | Fix `expose_url` helper does not include custom base url if it is setFrancisco Javier López2018-05-221-2/+14
|/ /
* | Merge branch 'zj-workhorse-archive-mandatory' into 'master'Grzegorz Bizon2018-05-212-15/+1
|\ \ | | | | | | | | | | | | Move git archives downloading to Gitaly See merge request gitlab-org/gitlab-ce!18990
| * | Move git archives downloading to GitalyZeger-Jan van de Weg2018-05-172-15/+1
| | |
* | | Merge branch 'xeodon/gitlab-ce-fix/45743-master-fix-gitaly-delete-refs' into ↵Grzegorz Bizon2018-05-191-0/+4
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Fix error when deleting an empty list of refs Closes #45743 See merge request gitlab-org/gitlab-ce!19053
| * | | Fix error when deleting an empty list of refsxeodon/gitlab-ce-fix/45743-master-fix-gitaly-delete-refsSergey Sinev2018-05-181-0/+4
| | | | | | | | | | | | | | | | Closes #45743
* | | | Fix double brackets being linkified in wiki markdownAlex2018-05-181-0/+6
| | | |
* | | | Merge branch 'fix/handle-large-commit-tag-message' into 'master'Robert Speicher2018-05-182-3/+105
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Add handling for commit/tags with big messages See merge request gitlab-org/gitlab-ce!17892
| * | | | Add handling for commit/tags with big messagesAhmad Sherif2018-05-182-3/+105
| | | | |
* | | | | Merge branch '5913-extract-ee-specific-lines-for-lib-gitlab-auth' into 'master'Robert Speicher2018-05-183-4/+50
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | [CE] Resolve "Extract EE specific files/lines for lib/gitlab/auth" See merge request gitlab-org/gitlab-ce!19037
| * | | | Minimize CE/EE difference in Gitlab::Auth::LDAP::User5913-extract-ee-specific-lines-for-lib-gitlab-authRémy Coutable2018-05-181-2/+3
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | | Minimize CE/EE difference in Gitlab::Auth::LDAP::ConfigRémy Coutable2018-05-181-1/+35
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | | Minimize CE/EE difference in Gitlab::Auth::LDAP::AccessRémy Coutable2018-05-181-1/+12
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | Merge branch 'zj-calculate-checksum-mandator' into 'master'Nick Thomas2018-05-181-48/+24
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calculating repository checksums executed by Gitaly Closes gitaly#1105 and #46293 See merge request gitlab-org/gitlab-ce!18907
| * | | | | Calculating repository checksums executed by GitalyZeger-Jan van de Weg2018-05-181-48/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OPT_OUT status has been removed, and alternative implementation removed. Also checks if the repository exists before executing the checksum RPC to guard against NotFound errors. Closes gitlab-org/gitaly#1105
* | | | | | Merge branch 'memoize-database-version' into 'master'Rémy Coutable2018-05-181-0/+14
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | Memoize Gitlab::Database.version See merge request gitlab-org/gitlab-ce!19021
| * | | | | Memoize Gitlab::Database.versionmemoize-database-versionYorick Peterse2018-05-171-0/+14
| | |/ / / | |/| | | | | | | | | | | | | | | | | | This removes the need for running a database query every time we want to check the database version.
* | | | | Merge branch 'feature/gb/add-regexp-variables-expression' into 'master'Kamil Trzciński2018-05-188-48/+294
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for variables expression regexp syntax Closes #43601 See merge request gitlab-org/gitlab-ce!18902
| * | | | | Simplify untrusted regexp factory methodGrzegorz Bizon2018-05-182-12/+8
| | | | | |
| * | | | | Update variables expressions statement specsGrzegorz Bizon2018-05-171-1/+1
| | | | | |
| * | | | | Add variables expression pattern validation supportGrzegorz Bizon2018-05-172-1/+16
| | | | | |
| * | | | | Add anti-corruption layer above expressions pattern matchingGrzegorz Bizon2018-05-171-0/+28
| | | | | |
| * | | | | Add extended /regexp/ scheme support to untrusted regexpGrzegorz Bizon2018-05-171-1/+48
| | | | | |
| * | | | | Make variables expression pattern case-sensitivity explicitGrzegorz Bizon2018-05-152-0/+14
| | | | | |
| * | | | | Use parameterized RSpec to improve variables expressions specsGrzegorz Bizon2018-05-151-43/+44
| | | | | |
| * | | | | Add support for pattern matching in variables expressionsGrzegorz Bizon2018-05-153-4/+18
| | | | | |
| * | | | | Reorganize exceptions in pipeline expressions moduleGrzegorz Bizon2018-05-153-7/+7
| | | | | |
| * | | | | Merge branch 'master' into feature/gb/add-regexp-variables-expressionGrzegorz Bizon2018-05-156-37/+185
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (76 commits) Conflicts: lib/gitlab/untrusted_regexp.rb
| * | | | | | Add pattern matching variables expression lexemeGrzegorz Bizon2018-05-141-0/+57
| | | | | | |
| * | | | | | Raise variables statement exception if pattern is invalidGrzegorz Bizon2018-05-143-4/+11
| | | | | | |
| * | | | | | Do not raise if variable expression can not be evaluatedGrzegorz Bizon2018-05-141-2/+14
| | | | | | |
| * | | | | | Implement variables expression untrusted pattern lexemeGrzegorz Bizon2018-05-111-0/+55
| | | | | | |
* | | | | | | Merge branch 'ccr/regexp_anchor' into 'master'Kamil Trzciński2018-05-181-0/+4
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | Add anchor for incoming email regex, closes 44989 See merge request gitlab-org/gitlab-ce!18917
| * | | | | | Add anchor for incoming email regex, closes 44989ccr/regexp_anchorChantal Rollison2018-05-111-0/+4
| | | | | | |
* | | | | | | Preload pipeline data for project pipelinesYorick Peterse2018-05-171-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When displaying the pipelines of a project we now preload the following data: 1. Authors of the commits that belong to these pipelines 2. The number of warnings per pipeline, which is used by Ci::Pipeline#has_warnings? == Commit Authors Previously this data was queried for every Commit separately, leading to 20 SQL queries being executed in the worst case. With an average of 3 to 5 milliseconds per SQL query this could result in 100 milliseconds being spent in _just_ getting Commit authors. To preload this data Commit#author now uses BatchLoader (through Commit#lazy_author), and a separate module Gitlab::Ci::Pipeline::Preloader is used to ensure all authors are loaded before they are used. == Number of warnings This changes Ci::Pipeline#has_warnings? so it supports preloading of the number of warnings per pipeline. This removes the need for executing a COUNT(*) query for every pipeline just to see if it has any warnings or not.
* | | | | | | Merge branch 'zj-workhorse-commit-patch-diff' into 'master'Grzegorz Bizon2018-05-171-14/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Workhorse to send raw diff and patch for commits Closes gitaly#1196 See merge request gitlab-org/gitlab-ce!18974
| * | | | | | | Workhorse to send raw diff and patch for commitsZeger-Jan van de Weg2018-05-161-14/+0
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this change, this was done through unicorn. In theory this could time out. Workhorse has been sending these raw patches and diffs for a long time and is stable in doing so. Added bonus is the fact that `Commit#to_patch` can be removed. `Commit#to_diff` too, which closes https://gitlab.com/gitlab-org/gitaly/issues/324 Closes https://gitlab.com/gitlab-org/gitaly/issues/1196
* | | | | | | Merge branch 'sh-fast-admin-counts' into 'master'Yorick Peterse2018-05-161-0/+62
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Error 500 viewing admin page due to statement timeouts Closes #46255 See merge request gitlab-org/gitlab-ce!18982
| * | | | | | Fix Error 500 viewing admin page due to statement timeoutsStan Hu2018-05-161-0/+62
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Uses PostgreSQL tuple estimates to provide a much faster yet approximate count. See https://wiki.postgresql.org/wiki/Slow_Counting for more details. We only use this fast method if the table has been analyzed or vacuumed within the last hour. Closes #46255
* | | | | | Fix GPM content types for DoorkeeperYorick Peterse2018-05-161-3/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Certain controllers (e.g. Doorkeeper::TokensController) don't expose the method "request_format". This commit changes Gitlab::Metrics::WebTransaction so we don't rely on this method, instead using the underlying code this method uses. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/46412