summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | Add tests for Release Link APIShinya Maeda2019-01-074-9/+433
| |_|_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add tests for the API and add a couple of tests Add revert revert
* | | | | | | | | | Move settings operations controller from EE to CEmove-settings-oprations-to-cePeter Leitzen2019-01-062-0/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit prepares the structure for the upcoming feature error tracking.
* | | | | | | | | | Merge branch 'kamil-refactor-ci-builds-v5' into 'master'Grzegorz Bizon2019-01-0621-212/+411
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use BuildMetadata to store build configuration in JSONB form See merge request gitlab-org/gitlab-ce!21499
| * | | | | | | | | | Add config_options|variables to BuildMetadatakamil-refactor-ci-builds-v5Kamil Trzciński2019-01-0421-212/+411
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are data columns that store runtime configuration of build needed to execute it on runner and within pipeline. The definition of this data is that once used, and when no longer needed (due to retry capability) they can be freely removed. They use `jsonb` on PostgreSQL, and `text` on MySQL (due to lacking support for json datatype on old enough version).
* | | | | | | | | | | Merge branch '52446-hide-ado-project-banner-for-ci-file-or-ci-disabled' into ↵Kamil Trzciński2019-01-051-0/+99
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Hide Auto DevOps banner for projects with CI file or CI disabled Closes #52446 See merge request gitlab-org/gitlab-ce!24067
| * | | | | | | | | | | Hides ADO banner for certain projects52446-hide-ado-project-banner-for-ci-file-or-ci-disabledMayra Cabrera2019-01-041-0/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ADO banner is being hidden if: - Project has no CI configuration - Project has CI disabled Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52446
* | | | | | | | | | | | Merge branch 'sh-fix-multipart-uploads-failure' into 'master'Kamil Trzciński2019-01-051-6/+18
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix multipart attachments not uploading Closes gitlab-ee#9035 See merge request gitlab-org/gitlab-ce!24170
| * | | | | | | | | | | | Fix multipart attachments not uploadingsh-fix-multipart-uploads-failureStan Hu2019-01-051-6/+18
| | |_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mixing and matching the use of Rack::Request and ActionDispatch::Request in Rails 5 is bad, particularly if you have middleware that manipulates or accesses environment variables. `Gitlab::Middleware::Multipart` attempts to rewrite request parameters to the proper values (e.g. replacing `data_file` with `UploadedFile`). It does this by calling `Rack::Request#update_params`, which essentially updates `env['rack.request.form_hash']`. By changing to `ActionDispatch::Request`, the Go middleware was causing the request parameters to be stored inside `env['action_dispatch.request.request_parameters']`. Later calls to `Rack::Request#update_params` would not have any effect because it would attempt to update `env['rack.request.form_has']` instead of `env['action_dispatch.request.request_parameters']`. As a result, the controller still saw the old parameters. Since the Go middleware appears to be using `ActionDispatch::Request` for authorization methods, we can switch the multipart middleware to use it too. Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/9035
* | | | | | | | | | | | Merge branch '40473-api-support-for-kubernetes-integration' into 'master'Kamil Trzciński2019-01-051-0/+450
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add API Support for Kubernetes integration Closes #40473 See merge request gitlab-org/gitlab-ce!23922
| * | | | | | | | | | | | Include CRUD endpoints for Cluster API40473-api-support-for-kubernetes-integrationMayra Cabrera2019-01-041-0/+450
| | |/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds the following initial CRUD endpoints for Clusters API: - GET list of clusters - GET specific cluster - POST add existing cluster (mimic of "Add cluster") - PUT update cluser - DELETE destroy cluster Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/40473
* | | | | | | | | | | | Merge branch 'knative-rbac-check' into 'master'Kamil Trzciński2019-01-055-2/+42
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Require Knative to be installed only on an RBAC kubernetes cluster See merge request gitlab-org/gitlab-ce!23807
| * | | | | | | | | | | Require Knative to be installed only on an RBAC kubernetes clusterChris Baumbauer2019-01-045-2/+42
| | |_|_|_|_|/ / / / / | |/| | | | | | | | |
* | | | | | | | | | | Merge branch 'mk/fix-broken-master-api-variables' into 'master'Stan Hu2019-01-051-1/+3
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix broken master api_variables test See merge request gitlab-org/gitlab-ce!24165
| * | | | | | | | | | Fix broken master api_variables testMichael Kozono2019-01-041-1/+3
| | |_|_|_|_|_|_|_|/ | |/| | | | | | | |
* | | | | | | | | | Support new report syntax for common vulnerabilitiesKamil Trzciński2019-01-0410-2386/+3987
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This MR also removes other reports and uses one common set of fixtures for all test types.
* | | | | | | | | Merge branch 'feature/gb/expose-ci-api-url-variable' into 'master'Douwe Maan2019-01-044-1/+71
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Expose `CI_API_V4_URL` CI/CD variable Closes #54621 See merge request gitlab-org/gitlab-ce!23936
| * | | | | | | | Improve readablity of CI_API_V4_URL related codeGrzegorz Bizon2019-01-041-1/+1
| | | | | | | | |
| * | | | | | | | Minor improvements to CI_API_V4_URL variable exposurefeature/gb/expose-ci-api-url-variableGrzegorz Bizon2019-01-031-1/+1
| | | | | | | | |
| * | | | | | | | Expose CI_API_V4_URL predefined CI/CD variableGrzegorz Bizon2019-01-031-0/+1
| | | | | | | | |
| * | | | | | | | Add API::Helpers::Version and expose API root URLGrzegorz Bizon2019-01-033-17/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commits adds a new class that is supposed to represent Grape API version, like `v3` or `v4`.
| * | | | | | | | Expose method that returns GitLab API pathsGrzegorz Bizon2019-01-031-0/+36
| | | | | | | | |
* | | | | | | | | Merge branch 'include-templates' into 'master'Kamil Trzciński2019-01-047-43/+287
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include templates Closes #53445 See merge request gitlab-org/gitlab-ce!23495
| * | | | | | | | Allow to include templatesKamil Trzciński2019-01-047-43/+287
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This rewrites a syntax to allow include of templates. This also normalises the syntax used by include: feature
* | | | | | | | | Merge branch 'contacted_at' into 'master'Stan Hu2019-01-042-0/+45
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Guarantee visually correct order of runners on the "Admin Area > Runners" page Closes #49791 See merge request gitlab-org/gitlab-ce!21627
| * | | | | | | | | Take contacted_at value from the DB when it is about to be displayed in the ↵Alexander Kutelev2019-01-042-0/+45
| | |_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | Web interface with sorting enabled.
* | | | | | | | | Add sequence to release link factoryShinya Maeda2019-01-041-2/+2
| | | | | | | | |
* | | | | | | | | Add frozen_string_literal: trueShinya Maeda2019-01-043-0/+6
| | | | | | | | |
* | | | | | | | | Add unique constraint to release and urlShinya Maeda2019-01-043-99/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix coding style Improve coding style Decouple UPDATE and DELETE operations of asset links Rename links_attributes to assets:links Rename exposed param and updated spec
* | | | | | | | | Add spec for duplicate link name creationShinya Maeda2019-01-041-22/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix spec for mysql unique validation failure
* | | | | | | | | Add spec for Releases::Source and Link modelsShinya Maeda2019-01-044-0/+133
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Releases::Source and Releases::Link are covered by tests
* | | | | | | | | Update safe model attributes for import/exportShinya Maeda2019-01-041-0/+7
| | | | | | | | |
* | | | | | | | | Support CURD operation for release asset linksShinya Maeda2019-01-044-0/+229
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add Releases::Links model - Expose it in release API - Add integration tests
* | | | | | | | | Merge branch 'ee1979-gl-modal-vuex' into 'master'Kushal Pandya2019-01-043-0/+231
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create shared gl-modal-vuex component and module See merge request gitlab-org/gitlab-ce!24140
| * | | | | | | | | Create shared gl-modal-vuex component and moduleee1979-gl-modal-vuexPaul Slaughter2019-01-033-0/+231
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | **Why?** It is significantly easier to manage the visibility of the modal in Vuex. The module contains the state and mutations to manage this. The component wraps GlModal and syncs the visibility with the module.
* | | | | | | | | Merge branch '53696-make-rbac-default' into 'master'Kamil Trzciński2019-01-0411-108/+70
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make RBAC enabled by default for new clusters Closes #53696 See merge request gitlab-org/gitlab-ce!24119
| * | | | | | | | | Make RBAC enabled default for new clustersDylan Griffith2019-01-0411-108/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many changes were also made to tests that expected this to default to false.
* | | | | | | | | | Merge branch 'tz-make-user-popover-spec-more-consistent' into 'master'Kushal Pandya2019-01-041-6/+10
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Made user_popovers_spec more stable See merge request gitlab-org/gitlab-ce!24145
| * | | | | | | | | | Made user_popovers_spec more stable on the value sidetz-make-user-popover-spec-more-consistentTim Zallmann2019-01-041-6/+10
| | |_|_|_|_|_|_|/ / | |/| | | | | | | |
* | | | | | | | | | Merge branch 'ce-revert-revert-gold-trial-mrs' into 'master'Kamil Trzciński2019-01-044-18/+53
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring back reverted Gold Trial MR (CE backport) See merge request gitlab-org/gitlab-ce!23621
| * | | | | | | | | Port GitLab.com gold trial callout changes to CEce-revert-revert-gold-trial-mrsLuke Bennett2019-01-034-18/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/8731
* | | | | | | | | | Merge branch 'sh-skip-validation-visibility-changed' into 'master'Douwe Maan2019-01-041-0/+25
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only validate project visibility when it has changed Closes #55881 See merge request gitlab-org/gitlab-ce!24142
| * | | | | | | | | Only validate project visibility when it has changedsh-skip-validation-visibility-changedStan Hu2019-01-031-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On GitLab.com, there are hundreds of projects that have visibility levels that are inconsistent with the fork or group settings. Most likely, this happened during a GitLab project import because the validation was bypassed. Attempting to migrate these projects to hashed storage fails even though the migration doesn't touch the visibility settings. Skipping the visibility validation allows the migration to go through. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/55881
* | | | | | | | | | Merge branch '29951-issue-creation-by-email-without-subaddressing' into 'master'Sean McGivern2019-01-0421-56/+234
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support new issue creation by email without subaddressing Closes #29951 See merge request gitlab-org/gitlab-ce!23523
| * | | | | | | | | | Refactoring and review commentsBrett Walker2019-01-033-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | including verifying the project_slug
| * | | | | | | | | | Use new issue email address formatBrett Walker2019-01-0310-30/+164
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now use `-issue` in order to support catch all email addresses
| * | | | | | | | | | Use new merge request email address formatBrett Walker2019-01-0312-26/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now use `-merge-request` instead of `+merge-request+` in order to support catch all email addresses
| * | | | | | | | | | Use new unsubscribe linkBrett Walker2019-01-032-2/+10
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now use `-unsubscribe` instead of `+unsubscribe` in order to support catch all email addresses
* | | | | | | | | | Merge branch 'dm-git-access-any-ce' into 'master'Sean McGivern2019-01-046-22/+40
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | [CE] Don't run checks for changed refs when specific changes are unknown See merge request gitlab-org/gitlab-ce!23990
| * | | | | | | | | Remove unnecessary double caching (in hash and request store)dm-git-access-any-ceDouwe Maan2019-01-022-8/+1
| | | | | | | | | |
| * | | | | | | | | Consider all matching MRs to determine if user can pushDouwe Maan2019-01-022-1/+11
| | | | | | | | | |
View Lorry Controller Status