summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | GraphQL mutations for managing NotesLuke Duncalfe2019-07-107-1/+376
| |_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/issues/62826
* | | | | | | | Merge branch 'fix-unicorn-sampler-workers-count' into 'master'Stan Hu2019-07-091-10/+23
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make unicorn_workers to return meaningful results Closes #63659 See merge request gitlab-org/gitlab-ce!30506
| * | | | | | | | Make unicorn_workers to return meaningful resultsfix-unicorn-sampler-workers-countKamil Trzciński2019-07-091-10/+23
| | |_|/ / / / / | |/| | | | | |
* | | | | | | | Merge branch 'project_api' into 'master'Douglas Barbosa Alexandre2019-07-093-5/+105
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve Project API Closes #49938, #40892, and #35992 See merge request gitlab-org/gitlab-ce!28327
| * | | | | | | Merge branch 'master' into sathieu/gitlab-ce-project_apiDouglas Barbosa Alexandre2019-07-09109-3112/+4862
| |\ \ \ \ \ \ \
| * | | | | | | | Add auto_devops_* to project APIMathieu Parent2019-07-032-2/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The auto_devops object sometimes doesn't exists. We may need to create it.
| * | | | | | | | Add build_git_strategy attribute to project APIMathieu Parent2019-07-032-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We map the boolean to the string 'fetch' or 'clone', to be more explicit.
| * | | | | | | | Add *_access_level to project APIMathieu Parent2019-07-032-5/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - issues_access_level - repository_access_level - merge_requests_access_level - builds_access_level - wiki_access_level - snippets_access_level
* | | | | | | | | Merge branch 'jc-disable-can-use-disk-temp' into 'master'Stan Hu2019-07-091-0/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disabling can_use_disk? temporarily See merge request gitlab-org/gitlab-ce!30526
| * | | | | | | | | Disabling can_use_disk? temporarilyJohn Cai2019-07-091-0/+2
| | |_|_|/ / / / / | |/| | | | | | |
* | | | | | | | | Merge branch 'osw-bypass-push-rules-for-merge-to-ref' into 'master'Stan Hu2019-07-092-0/+26
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | [CE port] Bypass push rules for merge to ref service See merge request gitlab-org/gitlab-ce!30426
| * | | | | | | | [CE port] Bypass push rules for merge to ref serviceosw-bypass-push-rules-for-merge-to-refOswaldo Ferreira2019-07-092-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Turns out push rules to validate commit message does not apply in the context of automatic merge to the refs/merge-requests/:iid/merge. Mainly because if it fails to merge to it, we currently can't give enough preemptive feedback to the user and it'll turn the merge request unmergeable (given we automatically mark it as unmergeable if we can't merge to the ref). In general, it's a systematic operation, which already bypasses user authorization and git hooks. Therefore, this commit makes it bypass the push rules at EE as well.
* | | | | | | | | Merge branch 'sh-redact-redis-perf-bar' into 'master'Sean McGivern2019-07-091-0/+36
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Perform more redactions in Redis performance bar traces Closes #64309 See merge request gitlab-org/gitlab-ce!30514
| * | | | | | | | | Perform more redactions in Redis performance bar tracesStan Hu2019-07-091-0/+36
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | HMSET and AUTH commands were not properly redacted. This commit does that and adds a test. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/64309
* | | | | | | | | Skip spam check for task list updatesFelipe Artur2019-07-091-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Task list updates should not mark users action as spam on akismet.
* | | | | | | | | Merge branch 'dropdown-styling-fixes' into 'master'Mike Greiling2019-07-091-34/+0
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix dropdown styling issues Closes #64250 See merge request gitlab-org/gitlab-ce!30477
| * | | | | | | | Fix dropdown styling issuesPhil Hughes2019-07-091-34/+0
| | | | | | | | |
* | | | | | | | | Implemented frontend suggestionsNick Kipling2019-07-092-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Converted storage details into a small partial Reworded the storage details summary as suggested Updated pot file
* | | | | | | | | Remove `:graphql` feature flagcharlie ablett2019-07-092-33/+0
| |_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove `FeatureConstrainer` call wrapping api endpoint - Remove `Feature.enabled?(:graphql)` conditionals in back and frontend - Modify graphql test to be graphql flag agnostic - Remove api routing spec - Remove frontend feature flag via `gon`
* | | | | | | | Merge branch 'fix-median-counting-for-cycle-analytics' into 'master'Andreas Brandl2019-07-091-0/+34
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add additional condition to count median properly See merge request gitlab-org/gitlab-ce!30229
| * | | | | | | | Modify specs according to reviewfix-median-counting-for-cycle-analyticsMałgorzata Ksionek2019-07-041-9/+8
| | | | | | | | |
| * | | | | | | | Rearrange spec for readabilityMałgorzata Ksionek2019-07-041-22/+23
| | | | | | | | |
| * | | | | | | | Fix rubocop offenceMałgorzata Ksionek2019-07-041-1/+1
| | | | | | | | |
| * | | | | | | | Add where condition to count in median classMałgorzata Ksionek2019-07-041-0/+34
| | |_|_|_|_|/ / | |/| | | | | |
* | | | | | | | CE Port of Allow bulk update for group issuesFatih Acet2019-07-092-240/+268
| |_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a port MR for CE Original MR: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14141
* | | | | | | Fix subgroup url in search drop downFrancisco Javier López2019-07-091-62/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Inside a subgroup scope, when the user clicks on the search drop down in the header, some options are displayed. Some of this links direct the user to the issues and merge requests linked to the subgroup. Nevertheless, the url was broken because they doesn't include the complete group namespace
* | | | | | | Updates on success of an MR the count on top and in other tabsTim Zallmann2019-07-095-0/+187
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | New API endpoint for merge request count Updates all open tabs at the same time with one call Restructured API response API response changed to 401 if no current_user Added API + JS specs Fix for Static Check Updated Count on Open/Close, Assign/Unassign of MR's Checking if MR Count is refreshed Added # frozen_string_literal: true to spec Added Changelog
* | | | | | Merge branch 'pb-fix-confidential-mr-dropdown-typo' into 'master'Phil Hughes2019-07-091-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix typo on confidential MR dropdown See merge request gitlab-org/gitlab-ce!30499
| * | | | | | Fix typo on confidential MR dropdownpb-fix-confidential-mr-dropdown-typoPatrick Bajao2019-07-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change issues to issue's
* | | | | | | Fix WorkhorseHelpers module is missingShinya Maeda2019-07-091-0/+4
|/ / / / / / | | | | | | | | | | | | | | | | | | This interferes spec execution on local environment.
* | | | | | Fixed editing formNatalia Tepluhina2019-07-092-1/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - applied functional component patch - fixed border radius for list - fixed Karma test to take a correct li amount
* | | | | | Merge branch 'allow-reactive-caching-of-nil' into 'master'Stan Hu2019-07-092-28/+30
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow ReactiveCaching to support nil value See merge request gitlab-org/gitlab-ce!30456
| * | | | | | Allow ReactiveCaching to support nil valueFabio Pitino2019-07-082-28/+30
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When :calculate_reactive_caching returns a nil value this caused ReactiveCaching to schedule a worker every time the code using :with_reactive_cache was called. This issue caused an increasing amount of Sidekiq jobs being created continuously. Implementing this fix behind feature flag :reactive_caching_check_key_exists
* | | | | | Schedule namespace aggregation in other contextsMayra Cabrera2019-07-082-0/+57
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Schedules a Namespace::AggregationSchedule worker if some of the project statistics are refreshed. The worker is only executed if the feature flag is enabled.
* | | | | Merge branch 'run-pipeline-for-merge-train-at-train-ref-ce' into 'master'Kamil Trzciński2019-07-082-0/+39
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | CE Port: Fix race condition of `refs/merge` competing overwrite See merge request gitlab-org/gitlab-ce!30308
| * | | | | Fix race condition on merge train ref generationrun-pipeline-for-merge-train-at-train-ref-ceShinya Maeda2019-07-082-0/+39
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Today, Pipelines for merge train run on `refs/merge`, however, this causes a race condition that it can be overwritten by CheckMergeabilityService. This patch fixes the problem by generating `refs/train` for those pipelines.
* | | | | Merge branch 'jc-detect-nfs-for-rugged' into 'master'Dmitriy Zaporozhets2019-07-083-1/+99
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Use Rugged if we detect storage is NFS and we can access the disk See merge request gitlab-org/gitlab-ce!29725
| * | | | | Use Rugged if we detect storage is NFS and we can access the diskjc-detect-nfs-for-ruggedJohn Cai2019-07-053-1/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a module we use as a singleton to determine whether or not rails is able to access the disk
* | | | | | Merge branch '63475-fix-n-1' into 'master'Lin Jen-Shin2019-07-086-199/+177
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce N+1 queries in MergeRequestsController#ci_environment_status See merge request gitlab-org/gitlab-ce!30224
| * | | | | | Add retry:0 for controller specs n+163475-fix-n-1Thong Kuah2019-07-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | As a workaround for https://gitlab.com/gitlab-org/gitlab-ce/issues/64116
| * | | | | | Restore fallback to deployment_platform_clusterThong Kuah2019-07-082-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In 12.2 we will remove this fallback.
| * | | | | | Remove un-used methodThong Kuah2019-07-081-28/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We stopped calling the fallback so we can remove this now
| * | | | | | Could not address last 5 queriesThong Kuah2019-07-081-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Split into followup issue
| * | | | | | Extract deployment_metrics into own objectThong Kuah2019-07-083-155/+140
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can now share project so that we don't have to load project twice. Also, this extracts non-relevant logic out of Deployment. Update DeploymentsController accordingly
| * | | | | | Share project object in EnvironmentStatusThong Kuah2019-07-082-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise, each EnvironmentStatus object instantiates its own project when really they are the same. Improves query count performance
| * | | | | | Remove fallback to project.deployment_platformThong Kuah2019-07-081-12/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This improves query performance of MergeRequestsController#ci_environments_status a lot. However this means old deployments that deployed to kubernetes clusters with prometheus installations will no longer show performance metrics as we cannot backfill cluster_id from deployment_platform with certainty (clusters may be edited/added/deleted, which changes the results of deployment_platform).
| * | | | | | Add failing test showing N+1Thong Kuah2019-07-081-0/+12
| | |/ / / / | |/| | | | | | | | | | | | | | | | We have an N+1 problem where N is environments.
* | | | | | Merge branch '40379-CJK-search-min-chars' into 'master'Andreas Brandl2019-07-082-0/+22
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove minimum required characters for fuzzy search if no trigram is used See merge request gitlab-org/gitlab-ce!29810
| * | | | | | Ignore min_chars_for_partial_matching unles trigrm40379-CJK-search-min-charsMario de la Ossa2019-07-042-0/+22
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | If we're not using a trigram index, then ignore the min_chars_for_partial_matching setting
* | | | | | Merge branch '60213-fix-is-ee-webpack-checks' into 'master'Filipa Lacerda2019-07-081-2/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace runtime isEE check with compile time one Closes #60213 See merge request gitlab-org/gitlab-ce!30351