summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Merge branch 'sh-fix-source-project-nplus-one' into 'master'47213-epics-page-styling-is-brokenSean McGivern2018-06-041-1/+1
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix N+1 with source_projects in merge requests API See merge request gitlab-org/gitlab-ce!19346
| | * | | | Fix N+1 with source projects in merge requests APIsh-fix-source-project-nplus-oneStan Hu2018-06-031-1/+1
| | |/ / / | | | | | | | | | | | | | | | | | | | | Now that we are checking `MergeRequest#for_fork?`, we also need the source project preloaded for a merge request.
| * | | | Reveert build_relations and simply add a line for creating iidShinya Maeda2018-06-021-13/+7
| | | | |
| * | | | Merge branch 'master' into per-project-pipeline-iidShinya Maeda2018-06-0218-61/+170
| |\ \ \ \ | | |/ / /
| | * | | Use RequestStore to memoize Flipper features so that memoized values are ↵Rémy Coutable2018-06-011-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cleared between requests Signed-off-by: Rémy Coutable <remy@rymai.me>
| | * | | Make http_io honor HTTP(S)_PROXY environment.NLR2018-06-011-1/+1
| | | | |
| | * | | Merge branch '46010-add-more-validations-for-runners-and-runner-type' into ↵Kamil Trzciński2018-06-012-11/+11
| | |\ \ \ | | | |_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | 'master' Improve validations for Ci::Runner#runner_type See merge request gitlab-org/gitlab-ce!18901
| | | * | Improve Runners API validationsKamil Trzciński2018-05-311-1/+1
| | | | |
| | | * | Improve runner registration APIKamil Trzciński2018-05-311-7/+9
| | | | |
| | | * | Improve `Ci::Runner#assign_to` to return a flag whether it succeeded or notKamil Trzciński2018-05-311-3/+1
| | | | |
| | * | | Add "deny disk access" Gitaly feature (tripswitch)Jacob Vosmaer (GitLab)2018-06-018-22/+100
| | | | |
| | * | | Merge branch '46481-preserve-warnings-even-if-passed' into 'master'Douwe Maan2018-06-015-20/+37
| | |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "A lot of constants redefinition warnings" Closes #46481 See merge request gitlab-org/gitlab-ce!19286
| | | * | | Introduce Gitlab::Auth.omniauth_setup_providersLin Jen-Shin2018-06-011-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | Which could extend from EE
| | | * | | Eliminate constants warnings by:Lin Jen-Shin2018-06-014-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Replace `require` or `require_relative` with `require_dependency` * Remove unneeded `autoload`
| | | * | | Preserve warnings even if it passedLin Jen-Shin2018-06-011-10/+10
| | | | |/ | | | |/|
| | * | | Add validation to webhook and service URLs to ensure they are not blocked ↵Francisco Javier López2018-06-011-5/+12
| | |/ / | | | | | | | | | | | | because of SSRF
| * | | Merge branch 'master' into per-project-pipeline-iidShinya Maeda2018-06-0168-5534/+231
| |\ \ \ | | |/ /
| | * | Merge branch 'rails5-active-sup-subscriber' into 'master'Rémy Coutable2018-05-311-1/+1
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make ActiveRecordSubscriber rails 5 compatible Closes #44702 See merge request gitlab-org/gitlab-ce!19276
| | | * | Make ActiveRecordSubscriber rails 5 compatibleJarka Kadlecová2018-05-311-1/+1
| | | |/
| | * | Add merge requests list endpoint for groupsFelipe Artur2018-05-311-17/+32
| | | |
| | * | Merge branch '41587-osw-mr-metrics-migration-take-two' into 'master'Grzegorz Bizon2018-05-312-2/+13
| | |\ \ | | | | | | | | | | | | | | | | | | | | Take two for MR metrics population background migration See merge request gitlab-org/gitlab-ce!19097
| | | * | Take two for MR metrics population background migrationOswaldo Ferreira2018-05-302-2/+13
| | | | |
| | * | | Export assigned issues in iCalendar feedImre Farkas2018-05-315-26/+36
| | | | |
| | * | | Merge branch 'sh-fix-issue-api-perf-n-plus-one' into 'master'Nick Thomas2018-05-311-1/+1
| | |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Eliminate cached N+1 queries for projects in Issue API See merge request gitlab-org/gitlab-ce!19269
| | | * | | Eliminate cached N+1 queries for projects in Issue APIStan Hu2018-05-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In CE, every `Issue` entity is also a `ProjectEntity`, which calls `entity&.project.try(:id)` to show the project ID. In an API request with 100 issues, this would hit the Rails statement cache 100 times for the same project and cause unnecessary overhead as related models would also be loaded. In EE, we call `Issue#supports_weight?` for each issue, which then calls `project&.feature_available?(:issue_weights)`. If the project is not preloaded, this incurs additional overhead, as each individual Project object has to be queried. This can lead to a significant performance hit. In loading the CE project with 100 issues, this contributed to at least 22% of the load time. See https://gitlab.com/gitlab-org/gitlab-ce/issues/47031 for why testing this is a bit tricky.
| | * | | | Merge branch 'rails5-calendar' into 'master'Yorick Peterse2018-05-311-1/+1
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace .having with .where in calendar query Closes #46964 See merge request gitlab-org/gitlab-ce!19230
| | | * | | | Replace .having with .where in calendar queryJan Provaznik2018-05-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the current syntax doesn't work properly in Rails 5, the resulting query looks like: HAVING "events"."project_id" IN (0) instead of: HAVING "events"."project_id" IN (SELECT "projects"."id" FROM... Also we should not use ActiveRecord internal methods. In this case we can filter projects in WHERE clause instead of doing this in HAVING clause. Usage of WHERE should be also more efficient because grouping is then done on much smaller subset of records.
| | * | | | | Merge branch '46445-fix-gitaly-tree-entry-encoding' into 'master'Douwe Maan2018-05-311-1/+1
| | |\ \ \ \ \ | | | |_|_|_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix encoding bug in Gitaly::Commit::TreeEntry Closes #46445 and gitaly#1208 See merge request gitlab-org/gitlab-ce!19243
| | | * | | | Fix encoding error in Gitaly::Commit::TreeEntryKim "BKC" Carlbäcker2018-05-301-1/+1
| | | | |_|/ | | | |/| |
| | * | | | Merge branch 'fj-36819-remove-v3-api' into 'master'Douwe Maan2018-05-3142-5369/+12
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removal of API v3 from the codebase Closes #36819 See merge request gitlab-org/gitlab-ce!18970
| | | * | | | Removed API endpoint and specsfj-36819-remove-v3-apiFrancisco Javier López2018-05-3042-5369/+12
| | | |/ / /
| | * | | | Merge branch 'dm-relative-url-root-in-development' into 'master'Rémy Coutable2018-05-312-1/+6
| | |\ \ \ \ | | | |_|_|/ | | |/| | | | | | | | | | | | | | | Fix various bugs related to relative_url_root in development See merge request gitlab-org/gitlab-ce!19213
| | | * | | Fix various bugs related to relative_url_root in developmentDouwe Maan2018-05-292-1/+6
| | | | | |
| | * | | | Merge branch 'sh-add-usage-ping-api-settings' into 'master'Robert Speicher2018-05-301-0/+1
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing usage_ping_enabled to API settings See merge request gitlab-org/gitlab-ce!19263
| | | * | | | Add missing usage_ping_enabled to API settingsStan Hu2018-05-301-0/+1
| | | | |/ / | | | |/| | | | | | | | | | | | | | Identified while resolving conflicts in https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/5906
| | * | | | Merge branch '46999-line-profiling-modal-width' into 'master'Clement Ho2018-05-301-2/+2
| | |\ \ \ \ | | | |/ / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Line profiling modal width Closes #46999 See merge request gitlab-org/gitlab-ce!19253
| | | * | | Fix UI broken in line profiling modal due to Bootstrap 4Takuya Noguchi2018-05-311-2/+2
| | | | | |
| | * | | | Merge branch 'dm-api-projects-members-preload' into 'master'Robert Speicher2018-05-302-16/+20
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only preload member records for the relevant projects/groups/user in projects API See merge request gitlab-org/gitlab-ce!18945
| | | * | | | Only preload member records for the relevant projects/groups/user in ↵dm-api-projects-members-preloadDouwe Maan2018-05-302-16/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | projects API
| | * | | | | Merge branch 'sh-use-grape-path-helpers' into 'master'Sean McGivern2018-05-301-1/+1
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace grape-route-helpers with our own grape-path-helpers Closes #45718 See merge request gitlab-org/gitlab-ce!19240
| | | * | | | | Replace grape-route-helpers with our own grape-path-helpersStan Hu2018-05-301-1/+1
| | | | |_|_|/ | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gem (https://gitlab.com/gitlab-org/grape-path-helpers) makes a number of changes: 1. Brings in @mdelaossa's changes in https://github.com/reprah/grape-route-helpers/pull/21 2. Fixes some broken specs and code for Grape 1.0+ 3. Optimizes the generation of paths by bringing in @dblessing's HashWithIndifferentAccess changes in https://gitlab.com/gitlab-org/gitlab-ce/issues/45718#note_70123793 Closes #45718
| | * | | | | Merge branch 'brodock-refactor-hashed-storage-task' into 'master'Douwe Maan2018-05-302-86/+90
| | |\ \ \ \ \ | | | |_|/ / / | | |/| | | | | | | | | | | | | | | | | | Extracted auxiliary methods from storage.rake into specific RakeHelper See merge request gitlab-org/gitlab-ce!18962
| | | * | | | Specs covering listing all gitlab:storage:* typesbrodock-refactor-hashed-storage-taskGabriel Mazetto2018-05-291-1/+1
| | | | | | |
| | | * | | | refactor to remove duplicationGabriel Mazetto2018-05-281-14/+12
| | | | | | |
| | | * | | | assign helper full name to variable to reduce code lengthGabriel Mazetto2018-05-281-10/+19
| | | | | | |
| | | * | | | Refactor RakeHelper to reduce complexityGabriel Mazetto2018-05-281-26/+18
| | | | | | |
| | | * | | | Extracted auxiliary methods from storage.rake into specific RakeHelperGabriel Mazetto2018-05-282-86/+91
| | | | | | |
| | * | | | | Merge branch 'fix/bitbucket_import_anonymous' into 'master'Douwe Maan2018-05-302-1/+2
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import bitbucket issues that are reported by an anonymous user Closes #44381 See merge request gitlab-org/gitlab-ce!18199
| | | * | | | | Import bitbucket issues that are reported by an anonymous userBart Libert2018-05-242-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For these kind of issues, the "reporter" field is present but zero. In such a case, "fetch" will not return the default value, but it will return nil. Hence, importing fails, because the "username" field of nil is referenced Fixes issue #44381
| | * | | | | | Merge branch 'mattermost-api-v4' into 'master'Sean McGivern2018-05-303-6/+6
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated Mattermost integration to use Mattermost API v4 Closes #41631 See merge request gitlab-org/gitlab-ce!19043
View Lorry Controller Status