summaryrefslogtreecommitdiff
path: root/changelogs
Commit message (Collapse)AuthorAgeFilesLines
* Move Contribution Analytics related spec in ↵Imre Farkas2019-04-051-0/+5
| | | | spec/features/groups/group_page_with_external_authorization_service_spec to EE
* Merge branch '57493-add-limit-to-user-name' into 'master'Andreas Brandl2019-04-051-0/+5
|\ | | | | | | | | Add a length limit of 128 char to the user name field See merge request gitlab-org/gitlab-ce!26146
| * Add limit of 128 characters to users name57493-add-limit-to-user-nameBrandon Labuschagne2019-04-051-0/+5
| | | | | | | | | | Truncate existing users names which exceed 128 characters Include test for truncating users names
* | Merge branch 'require-all-templates-to-include-default-stages' into 'master'Sean McGivern2019-04-051-0/+5
|\ \ | | | | | | | | | | | | | | | | | | Require all templates to use default stages Closes #59992 See merge request gitlab-org/gitlab-ce!26954
| * | Require all templates to use default stagesKamil Trzciński2019-04-051-0/+5
| | | | | | | | | | | | | | | | | | Our templates are free to override the 'stages:'. However, they should follow the convention to always include 'build', 'test', 'deploy' which are defaults when not defined.
* | | Merge branch 'sh-update-rails-5.0.7.2' into 'master'Robert Speicher2019-04-051-0/+5
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Update Rails to 5.0.7.2 Closes #58963 See merge request gitlab-org/gitlab-ce!27022
| * | | Update Rails to 5.0.7.2Stan Hu2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a number of issues as described in https://weblog.rubyonrails.org/releases/. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/58963
* | | | Upgrade Gitaly to v1.32.0GitalyBot2019-04-051-0/+5
| | | |
* | | | Merge branch 'bump_kubernetes_1_11_9' into 'master'Douwe Maan2019-04-051-0/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Bump Helm to 2.13.1 and kubectl to 1.11.9 See merge request gitlab-org/gitlab-ce!26991
| * | | | Bump Helm to 2.13.1 and kubectl to 1.11.9bump_kubernetes_1_11_9Thong Kuah2019-04-051-0/+5
| | | | |
* | | | | Merge branch '58717-checkbox-cannot-be-checked-if-a-blockquote-is-above' ↵Sean McGivern2019-04-051-0/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Checkbox cannot be checked if preceded by a blockquote Closes #58717 See merge request gitlab-org/gitlab-ce!26937
| * | | | | Fenced blockquotes to not change source line pos58717-checkbox-cannot-be-checked-if-a-blockquote-is-aboveBrett Walker2019-04-041-0/+5
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | Replaces blockquote fences with \n, keeping the line numbering intact.
* | | | | Merge branch '58375-api-controller' into 'master'Sean McGivern2019-04-051-0/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a prometheus proxy API per environment Closes #58375 See merge request gitlab-org/gitlab-ce!26841
| * | | | | Add a Prometheus API per environmentrpereira22019-04-051-0/+5
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | The api will proxy requests to the environment's prometheus server. The Prometheus::ProxyService class can be reused when we add support for group prometheus servers.
* | | | | Merge branch 'ce-proj-settings-ok-avatar-only' into 'master'Kushal Pandya2019-04-051-0/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | CE Improve project avatar settings See merge request gitlab-org/gitlab-ce!26589
| * | | | | Improve project avatar settingsce-proj-settings-ok-avatar-onlyLuke Bennett2019-03-281-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | Prioritize and simplify project settings content.
* | | | | | Restore multiple series supportAdriel Santiago2019-04-051-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Corrects an issue whereby queries which return multiple series results were not rendering as multiple data series on the metrics dashboard
* | | | | | Merge branch 'fix-merge-request-relations-with-pipeline-on-mwps' into 'master'Kamil Trzciński2019-04-051-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix MWPS doesn't work for merge request pipelines See merge request gitlab-org/gitlab-ce!26906
| * | | | | | Fix merge requst relationships with pipeline in MWPSServicefix-merge-request-relations-with-pipeline-on-mwpsShinya Maeda2019-04-051-0/+5
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | MWPSService currently uses the old pipeline lookup method. It searches related merge requests with pipeline.ref, however, this doesn't work for attached/detached merge request pipelines.
* | | | | | Merge branch 'extend-cte-optimisations-to-projects' into 'master'Dmitriy Zaporozhets2019-04-051-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend CTE search optimisation to projects Closes #55170 See merge request gitlab-org/gitlab-ce!26908
| * | | | | | Extend CTE search optimisation to projectsextend-cte-optimisations-to-projectsSean McGivern2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we use the `search` param on an `IssuableFinder`, we can run into issues. We have trigram indexes to support these searches. On GitLab.com, we often see Postgres's optimiser prioritise the (global) trigram indexes over the index on `project_id`. For group and project searches, we know that it will be quicker to filter by `project_id` first, as it returns fewer rows in most cases. For group issues search, we ran into this issue previously, and went through the following iterations: 1. Use a CTE on the project IDs as an optimisation fence. This prevents the planner from disregarding the index on `project_id`. Unfortunately it breaks some types of sorting, like priority and popularity, as they sort on a joined table. 2. Use a subquery for listing issues, and a CTE for counts. The subquery - in the case of group lists - didn't help as much as the CTE, but was faster than not including it. We can safely use a CTE for counts as they don't have sorting. Now, however, we're seeing the same issue in a project context. The subquery doesn't help at all there (it would only return one row, after all). In an attempt to keep total code complexity under control, this commit removes the subquery optimisation and applies the CTE optimisation only for sorts we know that are safe. This means that for more complicated sorts (like priority and popularity), the search will continue to be very slow. If this is a high-priority issue, we can consider introducing further optimisations, but this finder is already very complicated and additional complexity has a cost. The group CTE optimisation is controlled by the same feature flag as before, `attempt_group_search_optimizations`, which is enabled by default. The new project CTE optimisation is controlled by a new feature flag, `attempt_project_search_optimizations`, which is disabled by default.
* | | | | | | Fix old image diff swipe viewftab2019-04-051-0/+5
| | | | | | |
* | | | | | | Fix IDE detecting MR from fork branchPaul Slaughter2019-04-051-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | **Why?** Currently the IDE loads a merge request based on only the `source_branch` name. This means it loads MR's from forks that have the same branch name (not good). - This required updating the BE API to accept `source_project_id`
* | | | | | | Merge branch '54506-show-error-when-namespace-svc-missing' into 'master'Grzegorz Bizon2019-04-051-0/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Show error when namespace/svc account missing Closes #54506 See merge request gitlab-org/gitlab-ce!26362
| * | | | | | | Display error for unmet prerequisitesjerasmus2019-04-021-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added the ability to display an error for unmet prerequisites
* | | | | | | | fix(gitlab-ci-config): allow strings in the 'include' keywordPaul 🐻2019-04-051-0/+5
| |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fix is a followup to !24098 which introduced a validation of the `include:` keyword of a gitlab-ci configuration file when triggered from /ci/lint API calls. However, there was a test case missing: the case of a single string as value. I have added a test case for that which shows that the code was not validating it correctly. This commit fixes that to allow all `include:` valid inputs.
* | | | | | | Merge branch ↵Clement Ho2019-04-041-0/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '57319-hide-kubernetes-cluster-warning-if-project-has-cluster-related' into 'master' Resolve "Hide Kubernetes cluster warning if project has cluster related" See merge request gitlab-org/gitlab-ce!26749
| * | | | | | | Hide kubernetes cluster warning when a cluster existsMike Greiling2019-04-041-0/+5
| | |_|_|/ / / | |/| | | | |
* | | | | | | Merge branch 'sh-fix-ref-name-caching' into 'master'Douglas Barbosa Alexandre2019-04-041-0/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix and expand Gitaly FindCommit caching See merge request gitlab-org/gitlab-ce!27018
| * | | | | | | Fix and expand Gitaly FindCommit cachingsh-fix-ref-name-cachingStan Hu2019-04-041-0/+5
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/26248 added support for deduplicating FindCommit requests using Gitaly ref name caching. However, not all endpoints were covered, and in one case the Gitaly wrapper wasn't actually surrounding the serialization step. We can safely cache ref names between FindCommit calls for #index and #show endpoints for merge requests and pipelines. This can significantly reduce the number of FindCommit requests.
* | | | | | | Fix UI anchor links after docs refactorEvan Read2019-04-041-0/+5
| | | | | | |
* | | | | | | Merge branch '57602-create-cluster-validations' into 'master'Mike Greiling2019-04-041-0/+5
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Display cluster form validation error messages inline Closes #57602 See merge request gitlab-org/gitlab-ce!26502
| * | | | | | Changelog entry addedEnrique Alcantara2019-04-031-0/+5
| | | | | | |
* | | | | | | Merge branch 'drop-usage-of-leagcy-artifacts' into 'master'Kamil Trzciński2019-04-041-0/+5
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | Drop legacy artifacts usage as there are no leftovers See merge request gitlab-org/gitlab-ce!24294
| * | | | | | Drop legacy artifacts usagedrop-usage-of-leagcy-artifactsShinya Maeda2019-04-041-0/+5
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Legacy artifacts have been correctly migrated to new place - ci_job_artifacts. Now it's time to remove the related code, but before that we should ensure it doesn't break anything by using feature flag.
* | | | | | Allow reactive caching to be used in servicesReuben Pereira2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for defining a reactive_cache_worker_finder function that will be used by the reactive_caching_worker to generate/initialize the calling object. This allows reactive caching to work with Services where the object cannot be obtained from DB but a new object can be initialized.
* | | | | | Allow to sort wiki pages by date and titleIgor2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add controls for sorting by title and date - Execute Gitaly call which now accepts sorting params for wikis
* | | | | | Merge branch 'update-gitlab-shell' into 'master'Douglas Barbosa Alexandre2019-04-041-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update GitLab Shell to v9.0.0 See merge request gitlab-org/gitlab-ce!27002
| * | | | | | Update GitLab Shell to v9.0.0Nick Thomas2019-04-041-0/+5
| | | | | | |
* | | | | | | Merge branch '58405-basic-limiting-complexity-of-graphql-queries' into 'master'Rémy Coutable2019-04-041-0/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Basic limiting complexity of GraphQL queries Closes #58405 See merge request gitlab-org/gitlab-ce!26629
| * | | | | | | Initial field and query complexity limits58405-basic-limiting-complexity-of-graphql-queriesBrett Walker2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It makes all Types::BaseField default to a complexity of 1. Queries themselves now have limited complexity, scaled to the type of user: no user, authenticated user, or an admin user.
* | | | | | | | Automatically set Prometheus step intervalPeter Leitzen2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By computing the step interval passed to the query_range Prometheus API call we improve the performance on the Prometheus server and GitLab by reducing the amount of data points sent back and prevent Prometheus from sending errors when requesting longer intervals.
* | | | | | | | Merge branch 'xanf/gitlab-ce-transfer-disables-js' into 'master'Sean McGivern2019-04-041-0/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Failing group transfer disables expandable sections" Closes #45511 See merge request gitlab-org/gitlab-ce!26837
| * | | | | | | | Redirect to edit page on group transfer failurexanf/gitlab-ce-transfer-disables-jsIllya Klymov2019-04-031-0/+5
| | | | | | | | |
* | | | | | | | | Allow to use untrusted Regexp via feature flagKamil Trzciński2019-04-041-0/+5
| |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This brings support for untrusted regexp for 'only:refs:' when enabled via feature flag: alllow_unsafe_ruby_regexp. This is by default disabled, and should not be used in production
* | | | | | | | Add cr remarksGosia Ksionek2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Chnage method used in model to make it more efficient database-wise Add additional spec
* | | | | | | | Merge branch 'revert-4e045cd5' into 'master'error-pipelines-for-blocked-usersAnnabel Dunstone Gray2019-04-041-5/+0
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Merge branch 'fix/missing-border' into 'master'" See merge request gitlab-org/gitlab-ce!26980
| * | | | | | | | Revert "Merge branch 'fix/missing-border' into 'master'"revert-4e045cd5Phil Hughes2019-04-041-5/+0
| | |_|/ / / / / | |/| | | | | | | | | | | | | | This reverts merge request !26242
* | | | | | | | Monitor GraphQL with PrometheusJan Provaznik2019-04-041-0/+5
| |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extends graphql's platform tracing class to observe duration of graphql methods. In graphql 1.8.11 is added prometheus class but it's not very useful for us because it uses prometheus_exporter to export results.
* | | | | | | Fixed test specsOswaldo Ferreira2019-04-041-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - added suggestions to mock data - fixed props to be not required