summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Collapse)AuthorAgeFilesLines
* Forking a project to a namespace with lower visibility.bvl-fork-public-project-to-private-namespaceBob Van Landuyt2017-12-291-7/+7
| | | | | | | In this case the project will get the minimum between both visibilities. If that visibility is restricted, then a lower level will be picked.
* Extend Cluster Applications to allow installation of PrometheusMayra Cabrera2017-12-221-1/+1
|
* Skip projects filter on merge requests searchjprovazn-searchJan Provaznik2017-12-212-1/+5
| | | | | | | | | | | | | | | When searching for merge requests, an additional subquery is added which by default filters only merge requests which belong to source or target project user has permission for. This filter is not needed because more restrictive filter which checks if user has permission for target project is used in the query. So unless a custom projects filter is used by user, it's possible to skip the default projects filter and speed up the final query. Related to #40540
* Merge branch '38356-add-last_commit_sha-to-the-commit-api' into 'master'Douwe Maan2017-12-214-20/+38
|\ | | | | | | | | | | | | Add new last_commit_id param for create commit endpoint Closes #38356 See merge request gitlab-org/gitlab-ce!15922
| * Updates from last code review:38356-add-last_commit_sha-to-the-commit-apiRubén Dávila2017-12-204-27/+33
| | | | | | | | | | | | - Apply some refactoring for code reuse - Add file status validation for Files::DeleteService - Write additional specs
| * Check if file has been modified for each action provided.Rubén Dávila2017-12-201-0/+12
| | | | | | | | | | | | When commiting multiple files we're now checking if any of those files has been modified by another commit and we're rejecting the new commit in this case.
* | Merge branch 'rs-blocks-json-serialization' into 'master'Douwe Maan2017-12-201-2/+2
|\ \ | | | | | | | | | | | | | | | | | | Add BlocksJsonSerialization model concern and include it in User Closes #37947 See merge request gitlab-org/gitlab-ce!14350
| * | Only include the user's ID in the time_spent command's update hashRobert Speicher2017-12-191-2/+2
| |/ | | | | | | | | | | | | | | | | Previously, this would include the entire User record in the update hash, which was rendered in the response using `to_json`, erroneously exposing every attribute of that record, including their (now removed) private token. Now we only include the user ID, and perform the lookup on-demand.
* | Don't link LFS-objects multiple times.bvl-fix-unlinking-with-lfs-objectsBob Van Landuyt2017-12-191-1/+1
|/ | | | | | If Unlinking a fork would fail somewhere after this, the LFS objects might still be linked. Which would cause issues when trying to destroy a project.
* Merge branch ↵Douwe Maan2017-12-183-7/+19
|\ | | | | | | | | | | | | | | | | '40871-todo-notification-count-shows-notification-without-having-a-todo' into 'master' Resolve "Todo notification count shows notification without having a todo" Closes #40871 See merge request gitlab-org/gitlab-ce!15807
| * Reset todo counters when the target is deleted40871-todo-notification-count-shows-notification-without-having-a-todoSean McGivern2017-12-183-7/+19
| | | | | | | | | | | | When the target is deleted, todos are destroyed, but we did not reset the todo cache for users with todos on the deleted target. This would only update after the next time the todo cache was updated for that user.
* | Merge remote-tracking branch 'upstream/master' into no-ivar-in-modulesLin Jen-Shin2017-12-1527-95/+326
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (671 commits) Make rubocop happy Use guard clause Improve language Prettify Use temp branch Pass info about who started the job and which job triggered it Docs: add indexes for monitoring and performance monitoring clearer-documentation-on-inline-diffs Add docs for commit diff discussion in merge requests sorting for tags api Clear BatchLoader after each spec to prevent holding onto records longer than necessary Include project in BatchLoader key to prevent returning blobs for the wrong project moved lfs_blob_ids method into ExtractsPath module Converted JS modules into exported modules spec fixes Bump gitlab-shell version to 5.10.3 Clear caches before updating MR diffs Use new Ruby version 2.4 in GitLab QA images moved lfs blob fetch from extractspath file Update GitLab QA dependencies ...
| * \ Merge branch 'issue_41021' into 'master'Sean McGivern2017-12-131-1/+1
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | Prevent worker that updates merge requests head pipeline from failing jobs Closes #41021 See merge request gitlab-org/gitlab-ce!15870
| | * | Prevent worker that updates merge requests head pipeline from failing jobsFelipe Artur2017-12-131-1/+1
| | | |
| * | | Merge branch 'fix-create-mr-from-issue-with-template' into 'master'Rémy Coutable2017-12-131-2/+4
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Execute quick actions when creating MR from issue Closes gitlab-ee#4277 See merge request gitlab-org/gitlab-ce!15810
| | * | | Execute quick actions when creating MR from issuefix-create-mr-from-issue-with-templateSean McGivern2017-12-131-2/+4
| | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In CE, this does nothing - the `MergeRequests::BuildService` will, at the time of writing, never return a description for this case. In EE, a project can have a default MR template, which will be returned by the service. Previously we were only using the description passed in the params, ignoring any already set on the object. Now we fall back to the one set on the object if there was none in the params, allowing quick actions to be executed from default MR templates when creating an MR from an issue.
| * | | Merge branch ↵Robert Speicher2017-12-122-2/+20
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'tmlee/gitlab-ce-28004-consider-refactoring-member-view-by-using-presenter' into 'master' Refactor member view by using presenter Closes #28004 See merge request gitlab-org/gitlab-ce!15715
| | * | | Refactor member view by using presenterTM Lee2017-12-112-2/+20
| | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Create MemberPresenter alongside with GroupMemberPresenter and ProjectMemberPresenter - Make Member model Presentable - Move action_member_permission from MembersHelper into the MemberPresenter - Added rspec using double, separate specs for GroupMemberPresenter and ProjectMemberPresenter Fixes #28004. Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | ActiveRecord::StaleObjectError: Attempted to update a stale object: Ci::Build41023-activerecord-staleobjecterror--attempted-to-update-a-stale-object--ci-buildShinya Maeda2017-12-121-8/+9
| |/ /
| * | Merge branch 'fix/gb/fix-pipeline-build-chain-tag-evaluation' into 'master'Douwe Maan2017-12-081-12/+13
| |\ \ | | |/ | |/| | | | | | | | | | | | | Fix invalid pipeline build chain tag evaluation Closes #40944 See merge request gitlab-org/gitlab-ce!15805
| | * Implement and use Gitlab::Ci::Pipeline::Chain::CommandKamil Trzcinski2017-12-081-12/+13
| | |
| * | Merge branch 'feature/sm/34834-missing-dependency-should-fail-job-2' into ↵Kamil Trzciński2017-12-071-0/+3
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Dependency validator Closes #34834 See merge request gitlab-org/gitlab-ce!14009
| | * | Use Class.new(StandardError) instead of custom extended error class. Bring ↵Shinya Maeda2017-12-061-1/+1
| | | | | | | | | | | | | | | | back specified_dependencies?.
| | * | Fail jobs if its dependency is missingShinya Maeda2017-12-061-0/+3
| | | |
| * | | Merge branch 'pawel/update-prometheus_gem_to_highly_optimized_version' into ↵Stan Hu2017-12-071-1/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Update prometheus-client-mmap gem to highly optimized version See merge request gitlab-org/gitlab-ce!15796
| | * | | Update prometheus-client-mmap gem to highly optimized versionPawel Chojnacki2017-12-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | + change string concatenation to help with GC pressure. + fix metric producing incompatible label sets
| * | | | Merge branch ↵Kamil Trzciński2017-12-071-6/+4
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '40761-remove-the-checkbox-for-starting-a-new-pipeline-in-auto-devops-settings' into 'master' Resolve "Remove the checkbox for starting a new pipeline in Auto DevOps settings" Closes #40761 See merge request gitlab-org/gitlab-ce!15714
| | * | | | Remove Auto DevOps checkboxes - backendbikebilly2017-12-041-6/+4
| | | | | |
| * | | | | add support for the commit reference filtermicael.bergeron2017-12-071-11/+3
| | | | | |
| * | | | | corrects the url buildingmicael.bergeron2017-12-071-4/+3
| | | | | |
| * | | | | diff notes created in merge request on a commit have the right contextmicael.bergeron2017-12-072-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | add a spec for commit merge request diff notes
| * | | | | Allow commenting on individual commits inside an MRDouwe Maan2017-12-071-3/+12
| | | | | |
| * | | | | Merge branch 'jej/per-user-protected-branches-api-ce' into 'master'Sean McGivern2017-12-072-0/+57
| |\ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | CE backport of "ProtectedBranches API handles per user/group granularity" See merge request gitlab-org/gitlab-ce!15747
| | * | | | CE backport of ProtectedBranches API changesJames Edwards-Jones2017-12-072-0/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In EE we now allow individual users/groups to be set on POST, which required some refactoring. See https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/3516
| * | | | | The API isn't using the appropriate services for managing forksFrancisco Javier López2017-12-071-7/+29
| | |_|/ / | |/| | |
| * | | | Merge branch ↵Tim Zallmann2017-12-061-3/+18
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '22680-unlabel-slash-command-limit-autocomplete-to-applied-labels' into 'master' Resolve "/unlabel quick action - limit autocomplete to applied labels" Closes #22680 and #27319 See merge request gitlab-org/gitlab-ce!11110
| | * | | | Limit autocomplete menu to applied labelsVitaliy @blackst0ne Klachkov2017-11-281-3/+18
| | | | | |
| * | | | | Merge branch 'ee-backport-issue_40654' into 'master'Sean McGivern2017-12-061-4/+5
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | [CE-Backport] Fix board filter when milestone is predefined See merge request gitlab-org/gitlab-ce!15750
| | * | | | | [CE-Backport] Fix board filter when milestone is predefinedee-backport-issue_40654Felipe Artur2017-12-051-4/+5
| | | | | | |
| * | | | | | Merge branch 'list-multiple-clusters' into 'master'Grzegorz Bizon2017-12-061-0/+6
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | List multiple clusters See merge request gitlab-org/gitlab-ce!15403
| | * \ \ \ \ \ Merge remote-tracking branch 'origin/master' into list-multiple-clustersMatija Čupić2017-12-054-3/+17
| | |\ \ \ \ \ \ | | | | |_|_|_|/ | | | |/| | | |
| | * | | | | | Replace Exception with ArgumentErrorMatija Čupić2017-12-051-1/+1
| | | | | | | |
| | * | | | | | Remove redundant return in Clusters::CreateServiceMatija Čupić2017-12-051-1/+1
| | | | | | | |
| | * | | | | | Merge remote-tracking branch 'origin/master' into list-multiple-clustersKamil Trzcinski2017-12-052-2/+12
| | |\ \ \ \ \ \
| | * \ \ \ \ \ \ Merge branch '35616-move-k8-to-cluster-page' into ↵Kamil Trzcinski2017-12-059-14/+108
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cluster-page-with-list-clusters
| | | * \ \ \ \ \ \ Merge branch 'master' into 35616-move-k8-to-cluster-pageShinya Maeda2017-12-058-13/+107
| | | |\ \ \ \ \ \ \
| | | * | | | | | | | Optimize Clusters::CreateService#executeShinya Maeda2017-12-041-1/+1
| | | | |_|_|_|_|_|/ | | | |/| | | | | |
| | * | | | | | | | Merge remote-tracking branch 'origin/list-multiple-clusters' into ↵Kamil Trzcinski2017-12-031-0/+6
| | |\ \ \ \ \ \ \ \ | | | |/ / / / / / / | | |/| | | | | | | | | | | | | | | | | cluster-page-with-list-clusters
| | | * | | | | | | Raise error when trying to create second clusterMatija Čupić2017-12-031-1/+1
| | | | | | | | | |
| | | * | | | | | | Merge branch 'list-multiple-clusters' of ↵Filipa Lacerda2017-11-271-5/+1
| | | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce into list-multiple-clusters * 'list-multiple-clusters' of https://gitlab.com/gitlab-org/gitlab-ce: Simplify Clusters::CreateService cluster check