summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Use Gitlab::Git's Popen on that module's codegitlab-git-popenAlejandro Rodríguez2017-09-203-5/+5
| | | | | This allows the current Gitaly migration to depend on less code outside of the Gitlab::Git module
* Merge branch 'rc/use-set-in-api-specs' into 'master'Grzegorz Bizon2017-09-2011-81/+87
|\ | | | | | | | | Add `test-prof` and use `set` in API specs See merge request gitlab-org/gitlab-ce!14370
| * Use set in some API specsrc/use-set-in-api-specsRémy Coutable2017-09-1911-81/+87
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'fix/gitaly-commit-diff-rpc-encoding-error' into 'master'Robert Speicher2017-09-201-0/+4
|\ \ | | | | | | | | | | | | | | | | | | Encode paths properly for Gitaly Commit{Diff,Delta} Closes gitaly#589 See merge request gitlab-org/gitlab-ce!14361
| * | Encode paths properly for Gitaly Commit{Diff,Delta}fix/gitaly-commit-diff-rpc-encoding-errorAhmad Sherif2017-09-191-0/+4
| | | | | | | | | | | | Fixes gitaly#589
* | | Merge branch '37894-handle-if-auto-devops-domain-is-not-set' into 'master'Stan Hu2017-09-192-1/+77
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Handle if Auto DevOps domain is not set" Closes #37894 See merge request gitlab-org/gitlab-ce!14319
| * | | Add specs for warning message shown in Auto DevOps settings37894-handle-if-auto-devops-domain-is-not-setKamil Trzcinski2017-09-192-2/+72
| | | |
| * | | Add warning message if domain or cluster are not configuredbokebilly2017-09-191-1/+7
| | | |
* | | | Merge branch ↵Fatih Acet2017-09-191-182/+100
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '37298-the-prettytime-utils-module-could-use-some-light-maintenance' into 'master' Resolve some tech debt in the prettyTime module Closes #37298 See merge request gitlab-org/gitlab-ce!14058
| * | | | Lightly refactor prettyTime module.37298-the-prettytime-utils-module-could-use-some-light-maintenanceBryce Johnson2017-09-141-182/+100
| | | | |
* | | | | Merge branch 'revert-feature-highlight' into 'master'Jacob Schatz2017-09-193-386/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert feature highlight Closes #38087 See merge request gitlab-org/gitlab-ce!14373
| * | | | | Revert feature highlightClement Ho2017-09-193-386/+0
| | | | | |
* | | | | | Prepare cherry-pick and revert for migration to GitalyJacob Vosmaer (GitLab)2017-09-191-11/+13
| | | | | |
* | | | | | Merge branch 'backstage/gb/jobs-triggering-policy-specifications' into 'master'Kamil Trzciński2017-09-195-117/+289
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement job policy specifications Closes #37280 See merge request gitlab-org/gitlab-ce!14265
| * | | | | Fix closing braces in YAML processor specsbackstage/gb/jobs-triggering-policy-specificationsGrzegorz Bizon2017-09-191-1/+2
| | | | | |
| * | | | | Raise exception when initializing unknown policyGrzegorz Bizon2017-09-181-5/+2
| | | | | |
| * | | | | Use parentheses in CI/CD kubernetes policy specsGrzegorz Bizon2017-09-181-1/+1
| | | | | |
| * | | | | Do not initialize YAML processor with project full pathGrzegorz Bizon2017-09-181-60/+59
| | | | | |
| * | | | | Do not pass project path from YAML processorGrzegorz Bizon2017-09-183-12/+28
| | | | | | | | | | | | | | | | | | | | | | | | Use project full path that can be received from a pipeline object
| * | | | | Add specs for CI/CD job policy refs specificationGrzegorz Bizon2017-09-181-0/+87
| | | | | |
| * | | | | Add specs for CI/CD job policy factory methodGrzegorz Bizon2017-09-181-0/+40
| | | | | |
| * | | | | Fix Rubocop offenses in YAML processor classesGrzegorz Bizon2017-09-181-1/+1
| | | | | |
| * | | | | Remove YAML processor refactoring stubs and fix specsGrzegorz Bizon2017-09-181-53/+55
| | | | | |
| * | | | | Implement CI/CD kubernetes policy specificationGrzegorz Bizon2017-09-181-0/+30
| | | | | |
* | | | | | Merge branch 'acey-flakes' into 'master'Rémy Coutable2017-09-191-10/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix transient user_edits_files_spec failures See merge request gitlab-org/gitlab-ce!14324
| * | | | | | fix transient user_edits_files_spec failuresacey-flakesSimon Knox2017-09-181-10/+6
| | | | | | |
* | | | | | | Merge branch '38060-fix-groups-api-owned-groups' into 'master'Douwe Maan2017-09-191-0/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the groups API endpoint to handle ?owned=true correctly Closes #38060 See merge request gitlab-org/gitlab-ce!14362
| * | | | | | | Fix the groups API endpoint to handle ?owned=true correctlyNick Thomas2017-09-191-0/+3
| | |_|_|_|_|/ | |/| | | | |
* | | | | | | Merge branch 'backstage/gb/steal-stages-statuses-migration' into 'master'Kamil Trzciński2017-09-191-0/+51
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Steal stages statuses migration Closes #37694 See merge request gitlab-org/gitlab-ce!14217
| * | | | | | Make sure all stages are migrated after a cleanupbackstage/gb/steal-stages-statuses-migrationGrzegorz Bizon2017-09-181-0/+16
| | | | | | |
| * | | | | | Steal stages statuses migrationGrzegorz Bizon2017-09-181-0/+35
| | |_|/ / / | |/| | | |
* | | | | | Merge branch '37999-fix-circuit-breaker' into 'master'Douwe Maan2017-09-191-0/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the filesystem shard health check to check all configured shards Closes #37999 See merge request gitlab-org/gitlab-ce!14341
| * | | | | Fix the filesystem shard health check to check all configured shardsNick Thomas2017-09-191-0/+1
| | | | | |
* | | | | | Detect n+1 issues involving GitalyAndrew Newdigate2017-09-192-2/+132
| | | | | |
* | | | | | Merge branch 'rc/improve-users-api-specs' into 'master'Grzegorz Bizon2017-09-193-32/+27
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | Improve the Users API specs by asserting against the response schema See merge request gitlab-org/gitlab-ce!14334
| * | | | | Improve the Users API specs by asserting against the response schemarc/improve-users-api-specsRémy Coutable2017-09-183-32/+27
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | Merge branch 'issue_32215' into 'master'Rémy Coutable2017-09-192-4/+18
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow DEVELOPER role to admin milestones Closes #32215 See merge request gitlab-org/gitlab-ce!14257
| * | | | | | Allow DEVELOPER role to admin milestonesissue_32215Felipe Artur2017-09-182-4/+18
| | | | | | |
* | | | | | | Change all instances of btn-sm to btn-small to keep things consistentJedidiah2017-09-191-1/+1
| | | | | | |
* | | | | | | Merge branch 'zj-update-project-settings' into 'master'Grzegorz Bizon2017-09-191-0/+43
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow updating of project auto devops settings Closes #37893 See merge request gitlab-org/gitlab-ce!14291
| * | | | | | | Fix instance default option being true as valueZeger-Jan van de Weg2017-09-181-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a user selected 'Instance default' as value for the auto devops settings, this was interpreted as a true value. Now we post an empty string in this case, meaning we want to set `NULL` in the database. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/37893#note_40541294
| * | | | | | | Allow updating of project auto devops settingsZeger-Jan van de Weg2017-09-151-0/+31
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updating didn't work, as the project_id was missing to be set for the project_auto_devops model. Fixes gitlab-org/gitlab-ce#37893
* | | | | | | Merge branch '34259-project-denial-of-service-via-gitmodules-fix' into 'master'Robert Speicher2017-09-191-0/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes project denial of service via gitmodules using Extended ASCII. Closes #34259 See merge request gitlab-org/gitlab-ce!14301
| * | | | | | | Fixes project denial of service via gitmodules using Extended ASCII.34259-project-denial-of-service-via-gitmodules-fixTiago Botelho2017-09-151-0/+6
| | | | | | | |
* | | | | | | | Fix MR widget with external CI services/integrations33287-fix-mr-widget-errors-with-external-servicesEric Eastwood2017-09-185-16/+128
| |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/33287 The MR widget was trying to render the pipelines section when there are no GitLab CI pipelines which was throwing some NPE errors.
* | | | | | | fix #35290 Make read-only API for public merge requests available without ↵haseeb2017-09-181-5/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | authentication
* | | | | | | Merge branch 'add_closed_at_attribute' into 'master'Rémy Coutable2017-09-182-1/+11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add 'closed_at' attribute to Issues API Closes #5935 See merge request gitlab-org/gitlab-ce!14316
| * | | | | | | Add 'closed_at' attribute to Issues APIVitaliy @blackst0ne Klachkov2017-09-162-1/+11
| | | | | | | |
* | | | | | | | Merge branch 'sh-optimize-discussion-json' into 'master'Yorick Peterse2017-09-181-0/+32
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Eliminate N+1 queries in loading discussions.json endpoint Closes #37955 See merge request gitlab-org/gitlab-ce!14327
| * | | | | | | | Eliminate N+1 queries in loading discussions.json endpointsh-optimize-discussion-jsonStan Hu2017-09-181-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In #37955,we see that the profile had a number of N+1 queries from repeated access to `cross_reference_not_visible_for?`. This was optimized in previous versions of GitLab by rendering all notes at once, counting the number of visible references, and then using that number to check whether a system note should be fully redacted. There was also another N+1 query calling `ProjectTeam#member?`, which did not take advantage of an optimization in prepare_notes_for_rendering that would preload the maximum access level per project. Closes #37955
View Lorry Controller Status