summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Don't fail the job if the Review App doesn't deploy properlyqa-fix-review-app-pingRémy Coutable2019-04-171-2/+2
| | | | | | | | | | | | It seems the deploy function causes the job to fail if it doesn't succeed. That wasn't the intent as we want to curl the Review App after the deploy finished (even if it failed) because sometimes the Review App is just a bit long to be ready. This change wraps the Review App deployment with "set +e"/"set -e" to ensure that the job doesn't fail right away if the deploy fails. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'filipa-master-patch-09545' into 'master'Phil Hughes2019-04-171-31/+11
|\ | | | | | | | | Removes ifEE example for JS code. See merge request gitlab-org/gitlab-ce!27408
| * Removes ifEE example for JS codefilipa-master-patch-09545Filipa Lacerda2019-04-161-31/+11
| |
* | Merge branch '59275-cluster-form-hints' into 'master'Phil Hughes2019-04-174-15/+39
|\ \ | | | | | | | | | | | | | | | | | | Add help texts to K8 form fields Closes #59275 See merge request gitlab-org/gitlab-ce!27274
| * | Add help texts to K8 form fieldsEnrique Alcántara2019-04-174-15/+39
|/ /
* | Merge branch 'fix-logic-in-43263-git-push-option-to-create-mr-test' into ↵11-11-auto-deploy-0012345Lin Jen-Shin2019-04-171-4/+2
|\ \ | | | | | | | | | | | | | | | | | | 'master' Fix logic in Api::Internal test See merge request gitlab-org/gitlab-ce!27193
| * | Fix logic in Api::Internal testfix-logic-in-43263-git-push-option-to-create-mr-testLuke Duncalfe2019-04-171-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The intention of this test is to ensure that the service class MergeRequests::PushOptionsHandlerService does not run when the :mr_push_options feature flag is disabled. This test was passing, however was not testing what it was supposed to be! For one, setting Feature.disable(:feature) in the test does not disable the feature, as rspec config in spec_helper stubs Feature to make all features enabled: https://gitlab.com/gitlab-org/gitlab-ce/commit/3ee48e422defaedd69946c607bd8d3672e510375 So the feature was still enabled in the test. But this test wasn't failing because unfortunately I had put: ``` expect(MergeRequests::PushOptionsHandlerService).to receive(:new) ``` instead of not_to! This meant that the `.new` method was being stubbed, so the service class did not create a MergeRequest, which satisfied the second expectation. ``` expect(MergeRequests::PushOptionsHandlerService).to receive(:new) ```
* | | Merge branch 'tortuetorche/gitlab-ce-patch-auto-deploy-extra-values' into ↵Grzegorz Bizon2019-04-173-0/+9
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 'master' Allow extra args for helm in Auto-DevOps See merge request gitlab-org/gitlab-ce!26838
| * | | Rename HELM_EXTRA_ARGS to differentiate between helm invocationstortuetorche/gitlab-ce-patch-auto-deploy-extra-valuesJames Fargher2019-04-172-4/+4
| | | |
| * | | Apply suggestion to doc/topics/autodevops/index.mdJames Fargher2019-04-111-1/+1
| | | |
| * | | Apply suggestion to doc/topics/autodevops/index.mdJames Fargher2019-04-101-1/+1
| | | |
| * | | Add HELM_EXTRA_ARGS to all Auto-DevOps helm invocationsJames Fargher2019-04-083-0/+7
| | | |
| * | | Allows extra arguments in helm commands for auto-deploying applicationTortue Torche2019-04-081-0/+2
| | | |
* | | | Merge branch 'fix-sidebar-spec-master' into 'master'Michael Kozono2019-04-171-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | [CE backport] Fix no assignee text in sidebar_spec See merge request gitlab-org/gitlab-ce!27444
| * | | | Fix no assignee text in sidebar_specfix-sidebar-spec-masterLuke Bennett2019-04-171-1/+1
|/ / / /
* | | | Merge branch '58361-issue-create-system-note-timestamp' into 'master'Stan Hu2019-04-164-2/+16
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set correct timestamps when creating past issues Closes #58361 See merge request gitlab-org/gitlab-ce!27406
| * | | | Set correct timestamps when creating past issues58361-issue-create-system-note-timestampHeinrich Lee Yu2019-04-164-2/+16
| | | | | | | | | | | | | | | | | | | | Sets `system_note_timestamp` from `created_at`
* | | | | Merge branch 'none-for-assignees' into 'master'Annabel Dunstone Gray2019-04-164-13/+13
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `None` for assignees Closes #60599 See merge request gitlab-org/gitlab-ce!27419
| * | | | | `None` for assigneesVictor Wu2019-04-164-13/+13
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update assignees_spec.js Update sidebar_spec.rb Update issues_spec.rb
* | | | | Merge branch 'zj-git-2.21-tests' into 'master'Douglas Barbosa Alexandre2019-04-166-9/+14
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Test using Git 2.21 See merge request gitlab-org/gitlab-ce!27418
| * | | | | Update preflight check to require 2.21zj-git-2.21-testsZeger-Jan van de Weg2019-04-161-1/+1
| | | | | |
| * | | | | Update source install docs for Git 2.21Zeger-Jan van de Weg2019-04-161-4/+4
| | | | | |
| * | | | | Test using Git 2.21Zeger-Jan van de Weg2019-04-164-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Given Rails doesn't use the Git binary anymore, this shouldn't be a problem.
* | | | | | Merge branch '60605-karma-failing' into 'master'Michael Kozono2019-04-161-18/+32
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Karma failing with MrWidgetAlertMessage when helpPath is provided Maximum call stack size exceeded" Closes #60605 See merge request gitlab-org/gitlab-ce!27429
| * | | | | | Add sync: false to fix Karma tests60605-karma-failingNathan Friend2019-04-161-18/+32
|/ / / / / /
* | | | | | Merge branch 'docs-cicd-workflow-illustration' into 'master'Achilleas Pipinellis2019-04-162-16/+45
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs: CI/CD intro - add new illustration + description See merge request gitlab-org/gitlab-ce!27278
| * | | | | | Docs: CI/CD intro - add new illustration + descriptionMarcia Ramos2019-04-162-16/+45
|/ / / / / /
* | | | | | Merge branch 'resource-label-events-api-link' into 'master'Achilleas Pipinellis2019-04-162-0/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Link to resource label events API See merge request gitlab-org/gitlab-ce!27412
| * | | | | | Link to resource label events APIVictor Wu2019-04-162-0/+4
|/ / / / / /
* | | | | | Merge branch 'winh-frontend-faq-docs' into 'master'Marcia Ramos2019-04-162-0/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create frontend FAQ (documentation) See merge request gitlab-org/gitlab-ce!26372
| * | | | | | Create frontend FAQ (documentation)Winnie Hellmann2019-04-162-0/+31
|/ / / / / /
* | | | | | Merge branch '59034-external-link-button' into 'master'Tim Zallmann2019-04-162-1/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaced "View file" button with doc-text icon Closes #59034 See merge request gitlab-org/gitlab-ce!27365
| * | | | | | Replaced external link button with doc-text icon59034-external-link-buttonDenys Mishunov2019-04-152-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/59034
* | | | | | | Merge branch 'nfriend-update-merge-request-widget-for-post-merge-pipelines' ↵Filipa Lacerda2019-04-1613-5/+270
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Update merge request widget to accommodate post-merge pipelines See merge request gitlab-org/gitlab-ce!25983
| * | | | | | | Add two warning messages to the MR widgetnfriend-update-merge-request-widget-for-post-merge-pipelinesNathan Friend2019-04-1513-5/+270
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds two new warning messages to the MR widget that handle cases involving merge request pipelines.
* | | | | | | | Merge branch 'docs-fix-ldap-typo' into 'master'Rémy Coutable2019-04-161-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix minor typo in LDAP docs Closes #60413 See merge request gitlab-org/gitlab-ce!27415
| * | | | | | | | Fix minor typo in LDAP docsdocs-fix-ldap-typoMark Fletcher2019-04-161-1/+1
| | | | | | | | |
* | | | | | | | | Merge branch 'gt-fix-styling-for-pages-status' into 'master'Filipa Lacerda2019-04-1616-24/+24
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix styling for `app/assets/stylesheets/pages/status.scss` Closes #59929 See merge request gitlab-org/gitlab-ce!26875
| * | | | | | | | | Rename CI related selectorsGeorge Tsiolis2019-04-1616-24/+24
| | |_|_|_|_|_|_|/ | |/| | | | | | |
* | | | | | | | | Merge branch '48431-width' into 'master'Kushal Pandya2019-04-163-1/+24
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hide overflow in scoped labels Closes #48431 See merge request gitlab-org/gitlab-ce!27362
| * | | | | | | | Hide overflow in scoped labelsRajat Jain2019-04-163-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Scoped labels overflew in all the places because of lack of restriction on `max-width`. This commit introduces the same limits on width we have inplace for regular labels
* | | | | | | | | Merge branch 'mc/feature/custom-metrics-ce' into 'master'Kamil Trzciński2019-04-169-28/+58
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Backport metrics report type See merge request gitlab-org/gitlab-ce!26798
| * | | | | | | | | Backport changes from EEMatija Čupić2019-04-169-28/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This backports the changes from https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/10452
* | | | | | | | | | Merge branch 'update-migration-docs' into 'master'Marcia Ramos2019-04-161-3/+23
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update migration docs See merge request gitlab-org/gitlab-ce!27261
| * | | | | | | | | | Update migration docsMichael Leopard2019-04-161-3/+23
|/ / / / / / / / / /
* | | | | | | | | | Merge branch 'patch-38' into 'master'Marcia Ramos2019-04-161-2/+2
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs: Complete "Repository storage" directions See merge request gitlab-org/gitlab-ce!27346
| * | | | | | | | | | Complete "Repository storage" directionsKatrin Leinweber2019-04-131-2/+2
| | |/ / / / / / / / | |/| | | | | | | |
* | | | | | | | | | Merge branch '60533-tooltip-for-view-file-shows-raw-html' into 'master'Kushal Pandya2019-04-162-3/+4
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Tooltip for "View file" shows raw HTML" Closes #60533 See merge request gitlab-org/gitlab-ce!27398
| * | | | | | | | | | Fix showing HTML content in view file tooltip60533-tooltip-for-view-file-shows-raw-htmlFatih Acet2019-04-162-3/+4
| | |_|_|_|_|_|_|_|/ | |/| | | | | | | |
* | | | | | | | | | Merge branch '59570-move-quick-action' into 'master'Sean McGivern2019-04-163-120/+92
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extract move quick action spec to shared example Closes #59570 See merge request gitlab-org/gitlab-ce!26907