summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Improve variables supportimprove-variables-supportKamil Trzciński2018-11-192-2/+6
| | | | | | This ensures that variables accept only string, alongside also improves kubernetes_namespace, improving validation and default value being set.
* Merge branch 'osw-fallback-on-blank-refs' into 'master'Grzegorz Bizon2018-11-151-1/+9
|\ | | | | | | | | | | | | Avoid Gitaly RPC errors when fetching diff stats Closes gitaly#1393 See merge request gitlab-org/gitlab-ce!22995
| * Avoid Gitaly RPC errors when fetching diff statsosw-fallback-on-blank-refsOswaldo Ferreira2018-11-141-1/+9
| |
* | Merge branch 'auto_devops_kubernetes_active' into 'master'Grzegorz Bizon2018-11-151-10/+16
|\ \ | | | | | | | | | | | | | | | | | | Switch kubernetes:active with checking $KUBECONFIG Closes gitlab-ee#8311 See merge request gitlab-org/gitlab-ce!22929
| * | Use except:variables to fix multiple variable jobsauto_devops_kubernetes_activeThong Kuah2018-11-091-20/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | As `only:variables` has the effect of being satisfied by any one of variable expression - this will create a situation where e.g. the dast job will be active even if there's no cluster. So use `except:variables which requires each and every variable expression to be satisfied.
| * | Switch kubernetes:active with checking $KUBECONFIGThong Kuah2018-11-091-10/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | We have found an issue with `kubernetes:active` where it doesn't work with multiple clusters. Switch with the similar $KUBECONFIG variable check but this check actually gets the variable from the cluster matching the environment name. Fixes https://gitlab.com/gitlab-org/gitlab-ee/issues/8311
* | | Track Kubernetes errors using SentryDylan Griffith2018-11-141-3/+11
| | |
* | | Merge branch 'patch-31' into 'master'Rémy Coutable2018-11-141-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Update comment in Auto-DevOps.gitlab-ci.yml See merge request gitlab-org/gitlab-ce!23013
| * | | Update Auto-DevOps.gitlab-ci.ymlTripple Moon2018-11-131-1/+1
| | | | | | | | | | | | Typo in comment INCREMENTAL_ROLLOUT_TYPE => INCREMENTAL_ROLLOUT_MODE
* | | | Show HTTP response code for Kubernetes errorsThong Kuah2018-11-132-16/+6
| | | |
* | | | Merge branch 'zj-remove-broken-storage' into 'master'Douwe Maan2018-11-131-37/+0
|\ \ \ \ | |/ / / |/| | | | | | | | | | | Remove obsolete gitlab_shell checks See merge request gitlab-org/gitlab-ce!22417
| * | | Migrate storage checks to GitalyZeger-Jan van de Weg2018-11-091-37/+0
| |/ / | | | | | | | | | | | | | | | | | | | | | Most of these checks are not valuable in the current setup, thus removed and replaced by Gitaly checking if its read and writeable. Depends on: 1. https://gitlab.com/gitlab-org/gitaly/merge_requests/675
* | | Merge branch 'ashmckenzie/hmac-token-decode-and-tests' into 'master'Stan Hu2018-11-132-2/+35
|\ \ \ | | | | | | | | | | | | | | | | Relocate JSONWebToken::HMACToken from EE See merge request gitlab-org/gitlab-ce!22906
| * | | Relocate JSONWebToken::HMACToken from EEashmckenzie/hmac-token-decode-and-testsAsh McKenzie2018-11-132-2/+35
| | | |
* | | | Merge branch 'ce-3777-promote-to-epic' into 'master'Rémy Coutable2018-11-124-19/+30
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Refactoring Issues::MoveService (port of promote epics) See merge request gitlab-org/gitlab-ce!22766
| * | | | Extract code from Issues::MoveServicece-3777-promote-to-epicJarka Košanová2018-11-124-19/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extract code to make it easier reusable - introduce AttributesRewriter and ContentRewriter - support group entites when rewriting content - make Uploader copy_to working for Namespaces
* | | | | Merge branch 'sh-53180-append-path' into 'master'Rémy Coutable2018-11-125-22/+10
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | Make sure there's only one slash as path separator Closes #53180 See merge request gitlab-org/gitlab-ce!22954
| * | | | Make sure there's only one slash as path separatorsh-53180-append-pathStan Hu2018-11-095-22/+10
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Ruby 2.4, `URI.join("http://test//", "a").to_s` will remove the double slash, however it's not the case in Ruby 2.5. Using chomp should work better for the intention, as we're not trying to allow things like ../ or / paths resolution. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53180
* | | | Merge branch 'osw-comment-on-any-line-on-diffs-w-feature-flag' into 'master'Douwe Maan2018-11-124-4/+275
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Comment on any expanded diff line on MRs (with feature-flag) Closes #13950 See merge request gitlab-org/gitlab-ce!22914
| * | | | Comment on any expanded diff line on MRsosw-comment-on-any-line-on-diffs-w-feature-flagOswaldo Ferreira2018-11-094-4/+275
| | |/ / | |/| |
* | | | Merge branch 'patch-31' into 'master'Stan Hu2018-11-121-1/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set an empty entrypoint for license_management job Closes #53786 See merge request gitlab-org/gitlab-ce!22955
| * | | | Set an empty entrypoint for license_management jobPhilippe Lafoucrière2018-11-101-1/+3
| | |/ / | |/| |
* | | | Merge branch 'blackst0ne-add-discord-service' into 'master'Sean McGivern2018-11-121-0/+9
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | Add Discord integration Closes #21635 See merge request gitlab-org/gitlab-ce!22684
| * | | Add Discord integrationblackst0ne2018-10-301-0/+9
| | | |
* | | | Remove circular dependency on Redactable in migrationStan Hu2018-11-092-17/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gitlab::BackgroundMigration::RedactLinks was prepending EE::Gitlab::BackgroundMigration::RedactLinks, while EE::Gitlab::BackgroundMigration::RedactLinks was trying to include Gitlab::BackgroundMigration::RedactLinks::Redactable. Ruby 2.5.3 failed with an uninitialized constant (https://gitlab.com/gitlab-org/gitlab-ee/-/jobs/118388511).
* | | | Merge branch 'rs-cherry-pick-api' into 'master'Nick Thomas2018-11-091-3/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve possible cherry pick API race condition Closes #53773 See merge request gitlab-org/gitlab-ce!22946
| * | | | Resolve possible cherry pick API race conditionrs-cherry-pick-apiRobert Speicher2018-11-091-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, we just fetched the latest commit for the given branch when presenting the resulting commit, but because something could have been committed to that branch between the time we cherry-picked and the time we render the result, the wrong commit could have been presented. Now, we fetch the commit object with the commit SHA returned by the commit change service, which should always be the correct commit. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53773
* | | | | Merge branch 'frozen-string-lib-gitlab-more' into 'master'Rémy Coutable2018-11-09101-5/+209
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Enable even more frozen string in lib/gitlab See merge request gitlab-org/gitlab-ce!22863
| * | | | | Enable even more frozen string in lib/gitlabgfyoung2018-11-06101-5/+209
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enables frozens string for the following: * lib/gitlab/conflict/**/*.rb * lib/gitlab/cross_project_access/**/*.rb * lib/gitlab/cycle_analytics/**/*.rb * lib/gitlab/data_builder/**/*.rb * lib/gitlab/database/**/*.rb * lib/gitlab/dependency_linker/**/*.rb * lib/gitlab/diff/**/*.rb * lib/gitlab/downtime_check/**/*.rb * lib/gitlab/email/**/*.rb * lib/gitlab/etag_caching/**/*.rb Partially addresses gitlab-org/gitlab-ce#47424.
* | | | | | Merge branch 'rs-revert-api' into 'master'Nick Thomas2018-11-091-0/+34
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add revert to commits API Closes gitlab-org/release/framework#48 See merge request gitlab-org/gitlab-ce!22919
| * | | | | Add revert to commits APIRobert Speicher2018-11-091-0/+34
| | |_|_|/ | |/| | |
* | | | | Improve Debugging Capabilities For Helm Application InstallsDylan Griffith2018-11-094-7/+7
|/ / / /
* | | | Limit parallel to 100limit-parallel-to-100Kamil Trzciński2018-11-081-1/+2
| |_|/ |/| | | | | | | | This prevents some of the abusive behaviors, of someone putting 100000 and creating out of memory condition easily
* | | Revert "Merge branch 'osw-comment-on-any-line-on-diffs' into 'master'"osw-revert-comment-in-any-diff-lineOswaldo Ferreira2018-11-074-275/+4
| | | | | | | | | | | | | | | This reverts commit a82a595728d54bdc12e51dfcfb22e9eddc449143, reversing changes made to e7df959b8f99875edd246c7ac7779c3203e8755e.
* | | Merge branch 'sh-paginate-bitbucket-server-imports' into 'master'Douglas Barbosa Alexandre2018-11-073-10/+49
|\ \ \ | | | | | | | | | | | | | | | | Paginate Bitbucket Server importer projects See merge request gitlab-org/gitlab-ce!22825
| * | | Paginate Bitbucket Server importer projectsStan Hu2018-11-073-10/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | To prevent delays in loading the page and reduce memory usage, limit the number of projects shown at 25 per page. Part of https://gitlab.com/gitlab-org/gitlab-ce/issues/50021
* | | | Merge branch 'jira-ping-differentiate-cloud' into 'master'Sean McGivern2018-11-071-1/+17
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usage ping - Differentiate Jira Server and Cloud Closes #51043 See merge request gitlab-org/gitlab-ce!22791
| * | | | Usage ping - Differentiate Jira Server and Cloudjira-ping-differentiate-cloudMario de la Ossa2018-11-071-1/+17
| | | | |
* | | | | Allow limiting quick actions to executeBob Van Landuyt2018-11-071-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes we don't want to trigger any quick actions that cause side effects. For example when building a record to validate. This allows listing the quick actions that need to be performed.
* | | | | Apply patches when creating MR via emailBob Van Landuyt2018-11-076-2/+155
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows users to add patches as attachments to merge request created via email. When an email to create a merge request is sent, all the attachments ending in `.patch` will be applied to the branch specified in the subject of the email. If the branch did not exist, it will be created from the HEAD of the repository. When the patches could not be applied, the error message will be replied to the user. The patches can have a maximum combined size of 2MB for now.
* | | | | Merge branch 'fj-41213-api-update-submodule-commit' into 'master'Sean McGivern2018-11-074-0/+88
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add endpoint to update a git submodule reference Closes #41213 See merge request gitlab-org/gitlab-ce!20949
| * | | | | Add submodule update API endpointFrancisco Javier López2018-11-074-0/+88
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new endpoint allow users to update a submodule's reference. The MR involves adding a new operation RPC operation in gitaly-proto (see gitlab-org/gitaly-proto!233) and change Gitaly to use this new version (see gitlab-org/gitaly!936). See gitlab-org/gitlab-ce!20949
* | | | | | Merge branch 'max_retries_when' into 'master'Grzegorz Bizon2018-11-073-8/+115
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow to configure when to retry builds Closes gitlab-runner#3515 See merge request gitlab-org/gitlab-ce!21758
| * | | | | | use Ci::Build instead of CommitStatus as per feedbackMarkus Doits2018-11-071-1/+1
| | | | | | |
| * | | | | | small fixes to doc and remove on whitespace noiseMarkus Doits2018-11-071-2/+0
| | | | | | |
| * | | | | | refactoring after latest feedbackMarkus Doits2018-11-071-1/+15
| | | | | | |
| * | | | | | remove now unneeded validationMarkus Doits2018-11-071-1/+0
| | | | | | |
| * | | | | | fix wrong retry error messagesMarkus Doits2018-11-071-0/+12
| | | | | | |
| * | | | | | refactor validations to a Entry::Retry classMarkus Doits2018-11-073-64/+78
| | | | | | |
| * | | | | | one more try to reduce cognitive overheadMarkus Doits2018-11-071-9/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before: - Method `validate_retry_max` has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. - Method `validate_retry_when` has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.