summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | | | | | | | | | | Fix environment automatic on_stop triggerShinya Maeda2019-04-3010-5/+169
| | | |_|_|_|_|_|_|_|_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to the nature of pipelines for merge requests, deployments.ref can be a merge request ref instead of a branch name. We support the environment auto-stop hook for this case
| * | | | | | | | | | | Merge branch 'bw-add-graphql-groups' into 'master'Grzegorz Bizon2019-04-3013-5/+222
| |\ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add basic GraphQL for a Group Closes #60786 See merge request gitlab-org/gitlab-ce!27492
| | * | | | | | | | | | Mention group query in GraphQL documentationbw-add-graphql-groupsBrett Walker2019-04-231-1/+5
| | | | | | | | | | | |
| | * | | | | | | | | | Refactor group query specBrett Walker2019-04-232-107/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and removing unnecessary code
| | * | | | | | | | | | Basic GraphQL for a groupBrett Walker2019-04-2312-4/+296
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add new query for Groups, with new GroupType and NamespaceType
| * | | | | | | | | | | Merge branch 'fix-danger-when-nobody-in-category' into 'master'James Lopez2019-04-301-1/+3
| |\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix reviewer roulette when no-one is in a category See merge request gitlab-org/gitlab-ce!27895
| | * | | | | | | | | | | Fix reviewer roulette when no-one is in a categorySean McGivern2019-04-301-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This would return `people.size` before, when it should return `nil`.
| * | | | | | | | | | | | Merge branch 'pl-fix-k8s-ns-query-variables' into 'master'Grzegorz Bizon2019-04-304-5/+37
| |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use correct k8s namespace in Prometheus queries See merge request gitlab-org/gitlab-ce!27812
| | * | | | | | | | | | | | Also define KubernetesService#namespace_forPeter Leitzen2019-04-291-0/+4
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Remove superfluous navigator operatorPeter Leitzen2019-04-291-1/+1
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Use correct k8s namespace in Prometheus queriesPeter Leitzen2019-04-293-5/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this commit the wrong namespace could have been used in Prometheus queries for group-level installations.
| * | | | | | | | | | | | | Merge branch 'disable-http-for-review-apps' into 'master'Nick Thomas2019-04-301-0/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable HTTP for the nginx-ingress Review App service See merge request gitlab-org/gitlab-ce!27884
| | * | | | | | | | | | | | Disable HTTP for the nginx-ingress Review App serviceRémy Coutable2019-04-301-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | | | | | | | | | | | Merge branch 'remove-disabled-pages-domains-part-2' into 'master'Nick Thomas2019-04-3010-10/+112
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove pages domain scheduled for removal See merge request gitlab-org/gitlab-ce!26227
| | * | | | | | | | | | | | | Remove disabled pages domainsVladimir Shushlin2019-04-3010-10/+112
| |/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Domain will be removed by verification worker after 1 week of being disabled
| * | | | | | | | | | | | | Merge branch 'give-reviewer-roulette-a-header' into 'master'Nick Thomas2019-04-301-5/+20
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Give reviewer roulette its own header in the docs See merge request gitlab-org/gitlab-ce!27883
| | * | | | | | | | | | | | | Give reviewer roulette its own header in the docsSean McGivern2019-04-301-5/+20
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Merge branch 'docs/clarify-issue-board-permissions' into 'master'Marcia Ramos2019-04-301-1/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarify that reporters can manage boards Closes #60794 See merge request gitlab-org/gitlab-ce!27616
| | * | | | | | | | | | | | | | Clarify that reporters can manage boardsSean McGivern2019-04-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original issue didn't specify this, but the code always allowed reporters to do everything here. I think that's correct because 1. Developer permissions normally refer to code, and boards are to do with issues. 2. Reporters can manage epics, as well as labels on issues, which are in a similar space.
| * | | | | | | | | | | | | | | Merge branch 'remove-docs-from-roulette' into 'master'Nick Thomas2019-04-302-15/+14
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs aren't reviewed using roulette Closes #61047 See merge request gitlab-org/gitlab-ce!27876
| | * | | | | | | | | | | | | | | Docs aren't reviewed using rouletteEvan Read2019-04-302-15/+14
| |/ / / / / / / / / / / / / / /
| * | | | | | | | | | | | | | | Merge branch 'mk/check-ooo-in-roulette' into 'master'Sean McGivern2019-04-301-8/+34
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reviewer roulette via Danger should take OOO status into account Closes #57652 See merge request gitlab-org/gitlab-ce!27696
| | * | | | | | | | | | | | | | | Exclude reviewers with OOO in statusMichael Kozono2019-04-291-8/+34
| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Merge branch 'fix_letter_opener' into 'master'Douwe Maan2019-04-300-0/+0
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix letter opener gem error: "undefined method `before_filter' for LetterOpenerWeb::LettersController:Class" See merge request gitlab-org/gitlab-ce!27769
| | * | | | | | | | | | | | | | | | Fix letter opener gem errorFelipe Artur2019-04-262-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Accessing http://localhostrails/letter_opener is thowring an exception, updating the gem fixes it.
| * | | | | | | | | | | | | | | | | Merge branch 'fj-refactor-lfs-import-code' into 'master'James Lopez2019-04-3011-234/+326
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactor LfsImportService and ImportService See merge request gitlab-org/gitlab-ce!27255
| | * | | | | | | | | | | | | | | | | Refactored LfsImportService and ImportServiceFrancisco Javier López2019-04-3011-234/+326
| |/ / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to make `LfsImportService` more reusable, we need to extract the logic inside `ImportService` and encapsulate it into the service.
| * | | | | | | | | | | | | | | | | Merge branch 'sh-disable-batch-load-replace-methods' into 'master'Sean McGivern2019-04-305-4/+18
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable method replacement in avatar loading Closes #60903 See merge request gitlab-org/gitlab-ce!27866
| | * | | | | | | | | | | | | | | | | Disable method replacement in avatar loadingStan Hu2019-04-295-4/+18
| | | |/ / / / / / / / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We've seen a significant performance penalty when using `BatchLoader#__replace_with!`. This defines methods on the batch loader that proxy to the 'real' object using send. The alternative is `method_missing`, which is slower. However, we've noticed that `method_missing` can be faster if: 1. The objects being loaded have a large interface. 2. We don't call too many methods on the loaded object. Avatar uploads meet both criteria above, so let's use the newly-released feature in https://github.com/exAspArk/batch-loader/pull/45. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/60903
| * | | | | | | | | | | | | | | | | Merge branch 'uninstall_cluster_apps' into 'master'Stan Hu2019-04-3042-30/+848
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Uninstall cluster applications (backend services) Closes #60665 See merge request gitlab-org/gitlab-ce!27096
| | * | | | | | | | | | | | | | | | | Add a newline in spec for readabilityStan Hu2019-04-291-0/+1
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Remove unncessary `to_time` in cluster servicesDylan Griffith2019-04-292-2/+2
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Internationalize errors CheckUninstallProgressServiceDylan Griffith2019-04-292-4/+13
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Rename #timeouted -> #timed_outDylan Griffith2019-04-296-11/+11
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Use #public_send instead #method.callDylan Griffith2019-04-293-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These builder methods are using user provided input inside a public_send but this is safe to do in this instance because before they are called we check before calling them that they match an expected application name.
| | * | | | | | | | | | | | | | | | | Minor refactor of prometheus_specDylan Griffith2019-04-291-4/+2
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Minor refactoring in check_uninstall_progress_service_specDylan Griffith2019-04-291-5/+5
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | In Prometheus use update! instead of updateDylan Griffith2019-04-292-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to not miss any errors since we are not checking the return value of update
| | * | | | | | | | | | | | | | | | | CheckUninstallProgressService remove unnecessary beginDylan Griffith2019-04-291-3/+1
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Fix uninstall specs: helm not uninstallableDylan Griffith2019-04-292-9/+9
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Deactivate any prometheus_service upon destroyThong Kuah2019-04-292-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Basically does the reverse of after_transition to :installed.
| | * | | | | | | | | | | | | | | | | Remove xit test for uninstall group cluster appDylan Griffith2019-04-291-60/+0
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | DELETE clusters/:id/:application endpointThong Kuah2019-04-298-0/+240
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add endpoint to delete/uninstall a cluster application
| | * | | | | | | | | | | | | | | | | Teach Prometheus about #uninstall_commandThong Kuah2019-04-292-0/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add specs
| | * | | | | | | | | | | | | | | | | Expose can_uninstall in cluster_status.jsonThong Kuah2019-04-2918-1/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only prometheus can be uninstalled atm, the rest will be dealt with later. Presumption is that new application types will have uninstallation implmemented at the same time.
| | * | | | | | | | | | | | | | | | | Destroy app on successful uninstallationThong Kuah2019-04-295-24/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rescue and put into :uninstall_errored if something goes wrong while destroying, which can happen. I think it is safe to expose the full error message from the destroy error. Remove the :uninstalled state as no longer used.
| | * | | | | | | | | | | | | | | | | Services to uninstall cluster applicationThong Kuah2019-04-297-0/+348
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | + to monitor progress of uninstallation pod
| | * | | | | | | | | | | | | | | | | Teach Helm::Api about #uninstallThong Kuah2019-04-292-0/+29
| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | Define state transitions for uninstalling appsThong Kuah2019-04-293-17/+98
| |/ / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added :uninstalled state as wasn't sure if we should be destroying the cluster apps
| * | | | | | | | | | | | | | | | | Merge branch 'fix-ref-text-of-mr-pipelines' into 'master'Ash McKenzie2019-04-305-15/+103
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix pipelines for merge requests does not show pipeline page when source branch is removed Closes #60432 See merge request gitlab-org/gitlab-ce!27803