summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Allow to make builds soft-archived.disallow-retry-of-old-buildsKamil Trzciński2018-11-058-19/+171
| | | | | The soft-archived builds cannot be run after some deadline time. The intent is to aggressively recycle old builds after sometime.
* Merge branch 'scheduled-manual-jobs-environment-play-buttons' into 'master'Tim Zallmann2018-11-056-42/+239
|\ | | | | | | | | | | | | Add the Play button for delayed jobs in environment page Closes #52129 See merge request gitlab-org/gitlab-ce!22106
| * Add scheduled_actions to JSON schemaWinnie Hellmann2018-10-241-0/+4
| |
| * Merge branch 'master' into scheduled-manual-jobs-environment-play-buttonsWinnie Hellmann2018-10-22569-8366/+143933
| |\ | | | | | | | | | | | | Conflicts: spec/javascripts/environments/environment_actions_spec.js
| * | Add spec for Deployment#scheduled_actionsShinya Maeda2018-10-181-1/+16
| | |
| * | Use regex for the extecped timeShinya Maeda2018-10-121-2/+1
| | |
| * | Improve deployment entity specShinya Maeda2018-10-121-4/+4
| | |
| * | Fix and add specShinya Maeda2018-10-103-2/+67
| | |
| * | Add feature flags for scheduled jobs in environments listWinnie Hellmann2018-10-091-2/+66
| | |
| * | Add component test for scheduled jobs in environments actions componentWinnie Hellmann2018-10-091-34/+88
| | |
* | | Merge branch 'fast_project_blob_path' into 'master'Douwe Maan2018-11-052-7/+47
|\ \ \ | | | | | | | | | | | | | | | | Improve performance of tree rendering in repositories with a lot of items See merge request gitlab-org/gitlab-ce!16511
| * | | Improve performance of tree rendering in repositories with lots of itemsDrew Blessing2018-10-312-7/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails is slow to generate paths dynamically especially when called hundreds/thousands of times. Also, rendering many partials hundreds of times can be slower. This change reduces the number of partials rendered and introduces two fast path methods to speed up path generation.
* | | | Merge branch ↵Stan Hu2018-11-042-5/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '53467-transient-failure-in-spec-controllers-groups-boards_controller_spec-rb' into 'master' Resolve "Transient failure in spec/controllers/groups/boards_controller_spec.rb" Closes #53467 See merge request gitlab-org/gitlab-ce!22785
| * | | | Fix transient error with boards_controller_spec53467-transient-failure-in-spec-controllers-groups-boards_controller_spec-rbBrett Walker2018-11-032-5/+7
| | | | |
* | | | | Merge branch 'sh-optimize-mr-commit-sha-lookup' into 'master'Douwe Maan2018-11-032-0/+47
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Optimize merge request refresh by using the database to check commit SHAs See merge request gitlab-org/gitlab-ce!22731
| * | | | Optimize merge request refresh by using the database to check commit SHAssh-optimize-mr-commit-sha-lookupStan Hu2018-11-022-0/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously for a given merge request, we would: 1. Create the array of commit SHAs involved in the push (A) 2. Request all merge request commits and map the SHA (B) 3. Reload the diff if there were any common commits between A and B We can avoid additional database querying and overhead by checking with the database whether the merge request contains any of the commit SHAs. Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/53213
* | | | | Merge branch 'use-our-own-docker-image-for-helm-install-pods' into 'master'Kamil Trzciński2018-11-022-7/+1
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Use our own docker image for helm install pods See merge request gitlab-org/gitlab-ce!22778
| * | | | Use our own docker image for helm install podsuse-our-own-docker-image-for-helm-install-podsDylan Griffith2018-11-022-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will reduce dependencies and failure points during installation. It will also reduce security risks from untrusted dependencies being able to effect all our users
* | | | | Add email for milestone changeChantal Rollison2018-11-023-3/+251
|/ / / /
* | | | Incorporates Kubernetes Namespace into Cluster's flowThong Kuah2018-11-0218-295/+616
| | | |
* | | | Merge branch '51620-cannot-add-label-to-issue-from-board' into 'master'Douwe Maan2018-11-022-0/+46
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cannot add label to issue from board Closes #51620 See merge request gitlab-org/gitlab-ce!22475
| * | | | Rename to assignable_labels_endpoint51620-cannot-add-label-to-issue-from-boardBrett Walker2018-10-311-1/+1
| | | | |
| * | | | Dynamically store the valid label endpointBrett Walker2018-10-312-0/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to ensure we have the right endpoint to query for an issue's possible valid labels, we store that url in the issue object that gets passed to the frontend.
* | | | | Merge branch 'top_level_clusters_controller' into 'master'Kamil Trzciński2018-11-026-19/+163
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Top level clusters controller See merge request gitlab-org/gitlab-ce!22438
| * | | | | Remove method as using nested resource routesThong Kuah2018-11-021-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We introduced hidden_clusterable_fields in this MR but it is no longer needed as we have gone back to nested resource routes which should provide the params such as `project_id`.
| * | | | | Add missing specs for PresenterThong Kuah2018-11-021-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | Also add missing interface in abstract class.
| * | | | | Resolve controller sharing concernThong Kuah2018-11-014-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use ClustersController as base while having Projects::ClustersController to inform what `clusterable` is. Thanks @ayufan for the great suggestion ! - View changes to work with new approach - Fix javascript for new approach - Fix feature specs for new approach - Fix QA
| * | | | | Move view and path concerns to presentersThong Kuah2018-11-013-1/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Move show path for cluster to ClusterPresenter - Create ClusterablePresenter to encapsulate logic. Consolidates scattered methods from BaseController and ClustersHelper into an object.
| * | | | | Abstract out project out of ClustersControllerThong Kuah2018-11-012-7/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To the extent possible swap out `project` with `clusterable` - Abstract paths for showing cluster or clusters. This will allow us to swap in alternative paths for group level cluster - Push :project_id and :namespace_id params from the URL to the POST body. - Create a nice helper for to generate links for the destroy action For some reason, spec :project_id and :namespace_id param are not going through `to_param` for a JSON format. Manually call `to_param` to fix specs. - Move :layout to BaseController
| * | | | | Convert clusters to use a top-level controllerThong Kuah2018-11-014-18/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation so that we can create both cluster attached to project and cluster attached to group. - Move ClustersController to top level - Move Clusters::ApplicationsController to top-level too - Creates a Clusters::BaseController to share common functions - Do not rely on @project ivar. Anything could set the ivar. - Fix Vue page components due to new data-page value Because of the controller change we have gone from `projects:clusters:new` to `clusters:new`, so we need to update the file location of the page components. There is somewhere a function that will convert data-page to a file location. On that note, projects/clusters/gcp/new/, translate to Projects::Clusters::Gcp#new doesn't exist so replace that with clusters/create_gcp/ and clusters/create_user/
* | | | | | Merge branch '28249-add-pagination' into 'master'Kamil Trzciński2018-11-023-0/+25
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds pagination to pipelines table in merge request & commit page Closes #28249 See merge request gitlab-org/gitlab-ce!22696
| * | | | | | Adds pagination to pipelines table in merge request page28249-add-paginationFilipa Lacerda2018-10-313-0/+25
| | | | | | |
* | | | | | | Add scheduled flag to job entityShinya Maeda2018-11-022-0/+5
| | | | | | |
* | | | | | | Remove ci_enable_scheduled_build feature flagShinya Maeda2018-11-023-52/+8
| | | | | | |
* | | | | | | Merge branch 'sh-optimize-reload-diffs-service' into 'master'Sean McGivern2018-11-023-8/+19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Significantly cut memory and SQL queries when reloading diffs See merge request gitlab-org/gitlab-ce!22725
| * | | | | | | Fix merge request specs that expect diff_files to be calledsh-optimize-reload-diffs-serviceStan Hu2018-11-012-8/+8
| | | | | | | |
| * | | | | | | Significantly cut memory usage and SQL queries when reloading diffsStan Hu2018-11-011-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By preloading certain models with the diff, we can eliminate many N+1 queries. For a push to the staging GitLab.com www-gitlab-com repository, this eliminates over 3000 SQL queries and appears to bring down the RSS usage by several gigabytes. Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/49703
* | | | | | | | Merge branch '52548-links-in-tabs-of-the-labels-index-pages-ends-with-html' ↵Sean McGivern2018-11-021-0/+25
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Resolve "Links in tabs of the labels index pages ends with `.html`" Closes #52548 See merge request gitlab-org/gitlab-ce!22716
| * | | | | | | | Fix typo in labels_helper_spec.rb52548-links-in-tabs-of-the-labels-index-pages-ends-with-htmlDmitriy Zaporozhets2018-11-011-1/+1
| | | | | | | | |
| * | | | | | | | Fix bug when links in tabs of the labels index pages ends with .htmlDmitriy Zaporozhets2018-10-311-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | | | | add related merge request endpointHelmut Januschka2018-11-011-0/+68
| | | | | | | | |
* | | | | | | | | Merge branch 'engwan/gitlab-ce-44012-filter-reactions-none-any'Sean McGivern2018-11-015-10/+72
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix testsHeinrich Lee Yu2018-11-014-10/+30
| | | | | | | | | |
| * | | | | | | | | Add testsHeinrich Lee Yu2018-11-013-4/+46
| | |_|_|_|/ / / / | |/| | | | | | |
* | | | | | | | | Merge branch '52382-filter-milestone-api-none-any' into 'master'Sean McGivern2018-11-012-4/+16
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Filter by `None`/`Any` for milestone in issues/mrs API" Closes #52382 See merge request gitlab-org/gitlab-ce!22637
| * | | | | | | | | Update API to accept None and AnyHeinrich Lee Yu2018-11-012-4/+16
| |/ / / / / / / /
* | | | | | | | | Merge branch 'gt-update-typos-in-comments-and-specs' into 'master'Achilleas Pipinellis2018-11-0162-85/+85
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix typos in comments and specs See merge request gitlab-org/gitlab-ce!22683
| * | | | | | | | | Fix typos in comments and specsGeorge Tsiolis2018-11-0162-85/+85
| | | | | | | | | |
* | | | | | | | | | Merge branch 'sh-optimize-merge-request-project-lookup' into 'master'Sean McGivern2018-11-012-0/+18
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce SQL queries needed to load open merge requests See merge request gitlab-org/gitlab-ce!22709
| * | | | | | | | | Reduce SQL queries needed to load open merge requestssh-optimize-merge-request-project-lookupStan Hu2018-10-302-0/+18
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The SQL queries and memory allocation in MergeRequests::RefreshService is dominated by queries for Project and Route loads. On staging, the absence of an inverse relationship caused Rails to make over 1100 extraneous SQL queries for the www-gitlab-com repository. Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/49703